CLI: drop notmuch_config_t from subcommand interface.

At this point it is unused in all subcommands.
This commit is contained in:
David Bremner 2021-02-27 10:07:45 -04:00
parent ec4b7efbe4
commit 1c747a501c
15 changed files with 37 additions and 69 deletions

View file

@ -171,60 +171,46 @@ void
notmuch_exit_if_unsupported_format (void); notmuch_exit_if_unsupported_format (void);
int int
notmuch_count_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_count_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_dump_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_dump_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_new_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_new_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_insert_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_insert_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_reindex_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_reindex_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_reply_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_reply_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_restore_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_restore_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_search_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_search_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_address_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_address_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_setup_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_setup_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_show_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_show_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_tag_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_tag_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_config_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_config_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
int int
notmuch_compact_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_compact_command (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
const char * const char *
notmuch_time_relative_date (const void *ctx, time_t then); notmuch_time_relative_date (const void *ctx, time_t then);

View file

@ -27,8 +27,7 @@ status_update_cb (const char *msg, unused (void *closure))
} }
int int
notmuch_compact_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_compact_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
const char *backup_path = NULL; const char *backup_path = NULL;
notmuch_status_t ret; notmuch_status_t ret;

View file

@ -578,7 +578,7 @@ _set_db_config (notmuch_database_t *notmuch, const char *key, int argc, char **a
} }
static int static int
notmuch_config_command_set (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_config_command_set (notmuch_database_t *notmuch,
int argc, char *argv[]) int argc, char *argv[])
{ {
char *group, *key; char *group, *key;
@ -691,8 +691,7 @@ notmuch_config_command_list (notmuch_database_t *notmuch)
} }
int int
notmuch_config_command (notmuch_config_t *config, notmuch_database_t *notmuch, notmuch_config_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
int ret; int ret;
int opt_index; int opt_index;
@ -722,7 +721,7 @@ notmuch_config_command (notmuch_config_t *config, notmuch_database_t *notmuch,
} }
ret = notmuch_config_command_get (notmuch, argv[1]); ret = notmuch_config_command_get (notmuch, argv[1]);
} else if (strcmp (argv[0], "set") == 0) { } else if (strcmp (argv[0], "set") == 0) {
ret = notmuch_config_command_set (config, notmuch, argc, argv); ret = notmuch_config_command_set (notmuch, argc, argv);
} else if (strcmp (argv[0], "list") == 0) { } else if (strcmp (argv[0], "list") == 0) {
ret = notmuch_config_command_list (notmuch); ret = notmuch_config_command_list (notmuch);
} else { } else {

View file

@ -151,8 +151,7 @@ count_file (notmuch_database_t *notmuch, FILE *input, notmuch_config_values_t *e
} }
int int
notmuch_count_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_count_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
char *query_str; char *query_str;
int opt_index; int opt_index;

View file

@ -361,8 +361,7 @@ notmuch_database_dump (notmuch_database_t *notmuch,
} }
int int
notmuch_dump_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_dump_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
const char *query_str = NULL; const char *query_str = NULL;
int ret; int ret;

View file

@ -444,8 +444,7 @@ add_file (notmuch_database_t *notmuch, const char *path, tag_op_list_t *tag_ops,
} }
int int
notmuch_insert_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_insert_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
notmuch_status_t status, close_status; notmuch_status_t status, close_status;
struct sigaction action; struct sigaction action;

View file

@ -1104,8 +1104,7 @@ _maybe_upgrade (notmuch_database_t *notmuch, add_files_state_t *state)
} }
int int
notmuch_new_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_new_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
add_files_state_t add_files_state = { add_files_state_t add_files_state = {
.verbosity = VERBOSITY_NORMAL, .verbosity = VERBOSITY_NORMAL,

View file

@ -83,8 +83,7 @@ reindex_query (notmuch_database_t *notmuch, const char *query_string,
} }
int int
notmuch_reindex_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_reindex_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
char *query_string = NULL; char *query_string = NULL;
struct sigaction action; struct sigaction action;

View file

@ -705,8 +705,7 @@ do_reply (notmuch_database_t *notmuch,
} }
int int
notmuch_reply_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_reply_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
notmuch_query_t *query; notmuch_query_t *query;
char *query_string; char *query_string;

View file

@ -219,8 +219,7 @@ parse_sup_line (void *ctx, char *line,
} }
int int
notmuch_restore_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_restore_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
bool accumulate = false; bool accumulate = false;
tag_op_flag_t flags = 0; tag_op_flag_t flags = 0;

View file

@ -794,8 +794,7 @@ static const notmuch_opt_desc_t common_options[] = {
}; };
int int
notmuch_search_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_search_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
search_context_t *ctx = &search_context; search_context_t *ctx = &search_context;
int opt_index, ret; int opt_index, ret;
@ -862,8 +861,7 @@ notmuch_search_command (unused(notmuch_config_t *config), notmuch_database_t *no
} }
int int
notmuch_address_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_address_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
search_context_t *ctx = &search_context; search_context_t *ctx = &search_context;
int opt_index, ret; int opt_index, ret;

View file

@ -124,8 +124,7 @@ parse_tag_list (void *ctx, char *response)
} }
int int
notmuch_setup_command (unused(notmuch_config_t *config), notmuch_setup_command (notmuch_database_t *notmuch,
notmuch_database_t *notmuch,
int argc, char *argv[]) int argc, char *argv[])
{ {
char *response = NULL; char *response = NULL;

View file

@ -1225,8 +1225,7 @@ static const notmuch_show_format_t *formatters[] = {
}; };
int int
notmuch_show_command (unused (notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_show_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
notmuch_query_t *query; notmuch_query_t *query;
char *query_string; char *query_string;

View file

@ -187,8 +187,7 @@ tag_file (void *ctx, notmuch_database_t *notmuch, tag_op_flag_t flags,
} }
int int
notmuch_tag_command (unused(notmuch_config_t *config), notmuch_database_t *notmuch, notmuch_tag_command (notmuch_database_t *notmuch, int argc, char *argv[])
int argc, char *argv[])
{ {
tag_op_list_t *tag_ops = NULL; tag_op_list_t *tag_ops = NULL;
char *query_string = NULL; char *query_string = NULL;

View file

@ -31,8 +31,7 @@
* Each subcommand should be passed either a config object, or an open * Each subcommand should be passed either a config object, or an open
* database * database
*/ */
typedef int (*command_function_t) (notmuch_config_t *config, notmuch_database_t *notmuch, typedef int (*command_function_t) (notmuch_database_t *notmuch, int argc, char *argv[]);
int argc, char *argv[]);
typedef struct command { typedef struct command {
const char *name; const char *name;
@ -42,10 +41,10 @@ typedef struct command {
} command_t; } command_t;
static int static int
notmuch_help_command (notmuch_config_t *config, notmuch_database_t *notmuch, int argc, char *argv[]); notmuch_help_command (notmuch_database_t *notmuch, int argc, char *argv[]);
static int static int
notmuch_command (notmuch_config_t *config, notmuch_database_t *notmuch, int argc, char *argv[]); notmuch_command (notmuch_database_t *notmuch, int argc, char *argv[]);
static int static int
_help_for (const char *topic); _help_for (const char *topic);
@ -349,8 +348,7 @@ _help_for (const char *topic_name)
} }
static int static int
notmuch_help_command (unused (notmuch_config_t *config), unused(notmuch_database_t *notmuch), int notmuch_help_command (unused(notmuch_database_t *notmuch), int argc, char *argv[])
argc, char *argv[])
{ {
int opt_index; int opt_index;
@ -374,8 +372,7 @@ notmuch_help_command (unused (notmuch_config_t *config), unused(notmuch_database
* to be more clever about this in the future. * to be more clever about this in the future.
*/ */
static int static int
notmuch_command (unused(notmuch_config_t *config), notmuch_command (notmuch_database_t *notmuch,
notmuch_database_t *notmuch,
unused(int argc), unused(char **argv)) unused(int argc), unused(char **argv))
{ {
@ -391,7 +388,7 @@ notmuch_command (unused(notmuch_config_t *config),
strerror (errno)); strerror (errno));
return EXIT_FAILURE; return EXIT_FAILURE;
} else { } else {
return notmuch_setup_command (NULL, notmuch, 0, NULL); return notmuch_setup_command (notmuch, 0, NULL);
} }
} }
@ -457,7 +454,6 @@ main (int argc, char *argv[])
const char *command_name = NULL; const char *command_name = NULL;
command_t *command; command_t *command;
const char *config_file_name = NULL; const char *config_file_name = NULL;
notmuch_config_t *config = NULL;
notmuch_database_t *notmuch = NULL; notmuch_database_t *notmuch = NULL;
int opt_index; int opt_index;
int ret = EXIT_SUCCESS; int ret = EXIT_SUCCESS;
@ -589,7 +585,7 @@ main (int argc, char *argv[])
} }
ret = (command->function)(config, notmuch, argc - opt_index, argv + opt_index); ret = (command->function)(notmuch, argc - opt_index, argv + opt_index);
DONE: DONE:
talloc_report = getenv ("NOTMUCH_TALLOC_REPORT"); talloc_report = getenv ("NOTMUCH_TALLOC_REPORT");