This terminology makes it clearer what's going on at the API layer, and paves the way for future changesets that offer more nuanced decryption policy. --- lib/index.cc | 2 +- lib/indexopts.c | 10 +++++----- lib/notmuch.h | 8 ++++---- notmuch-client.h | 4 ++-- notmuch.c | 12 ++++++------ 5 files changed, 18 insertions(+), 18 deletions(-)
diff --git a/lib/index.cc b/lib/index.cc index 6eb60f30..ff14e408 100644 --- a/lib/index.cc +++ b/lib/index.cc @@ -525,7 +525,7 @@ _index_encrypted_mime_part (notmuch_message_t *message, notmuch_database_t * notmuch = NULL; GMimeObject *clear = NULL; - if (!indexopts || !notmuch_indexopts_get_try_decrypt (indexopts)) + if (!indexopts || !notmuch_indexopts_get_decrypt_policy (indexopts)) return; notmuch = _notmuch_message_database (message); diff --git a/lib/indexopts.c b/lib/indexopts.c index ca6bf6c9..0f65b97c 100644 --- a/lib/indexopts.c +++ b/lib/indexopts.c @@ -36,24 +36,24 @@ notmuch_database_get_default_indexopts (notmuch_database_t *db) ((!(strcasecmp(decrypt, "true"))) || (!(strcasecmp(decrypt, "yes"))) || (!(strcasecmp(decrypt, "1"))))) - notmuch_indexopts_set_try_decrypt (ret, true); + notmuch_indexopts_set_decrypt_policy (ret, true); free (decrypt); return ret; } notmuch_status_t -notmuch_indexopts_set_try_decrypt (notmuch_indexopts_t *indexopts, - notmuch_bool_t try_decrypt) +notmuch_indexopts_set_decrypt_policy (notmuch_indexopts_t *indexopts, + notmuch_bool_t decrypt_policy) { if (!indexopts) return NOTMUCH_STATUS_NULL_POINTER; - indexopts->crypto.decrypt = try_decrypt; + indexopts->crypto.decrypt = decrypt_policy; return NOTMUCH_STATUS_SUCCESS; } notmuch_bool_t -notmuch_indexopts_get_try_decrypt (const notmuch_indexopts_t *indexopts) +notmuch_indexopts_get_decrypt_policy (const notmuch_indexopts_t *indexopts) { if (!indexopts) return false; diff --git a/lib/notmuch.h b/lib/notmuch.h index 2c5dcab5..ef463090 100644 --- a/lib/notmuch.h +++ b/lib/notmuch.h @@ -2244,17 +2244,17 @@ notmuch_database_get_default_indexopts (notmuch_database_t *db); * @since libnotmuch 5.1 (notmuch 0.26) */ notmuch_status_t -notmuch_indexopts_set_try_decrypt (notmuch_indexopts_t *indexopts, - notmuch_bool_t try_decrypt); +notmuch_indexopts_set_decrypt_policy (notmuch_indexopts_t *indexopts, + notmuch_bool_t decrypt_policy); /** * Return whether to decrypt encrypted parts while indexing. - * see notmuch_indexopts_set_try_decrypt. + * see notmuch_indexopts_set_decrypt_policy. * * @since libnotmuch 5.1 (notmuch 0.26) */ notmuch_bool_t -notmuch_indexopts_get_try_decrypt (const notmuch_indexopts_t *indexopts); +notmuch_indexopts_get_decrypt_policy (const notmuch_indexopts_t *indexopts); /** * Destroy a notmuch_indexopts_t object. diff --git a/notmuch-client.h b/notmuch-client.h index f7524e59..fc981459 100644 --- a/notmuch-client.h +++ b/notmuch-client.h @@ -500,8 +500,8 @@ int notmuch_minimal_options (const char* subcommand_name, /* the state chosen by the user invoking one of the notmuch * subcommands that does indexing */ struct _notmuch_client_indexing_cli_choices { - bool try_decrypt; - bool try_decrypt_set; + bool decrypt_policy; + bool decrypt_policy_set; notmuch_indexopts_t * opts; }; extern struct _notmuch_client_indexing_cli_choices indexing_cli_choices; diff --git a/notmuch.c b/notmuch.c index 7ee3ad0b..89d5e77b 100644 --- a/notmuch.c +++ b/notmuch.c @@ -99,8 +99,8 @@ int notmuch_minimal_options (const char *subcommand_name, struct _notmuch_client_indexing_cli_choices indexing_cli_choices = { }; const notmuch_opt_desc_t notmuch_shared_indexing_options [] = { - { .opt_bool = &indexing_cli_choices.try_decrypt, - .present = &indexing_cli_choices.try_decrypt_set, + { .opt_bool = &indexing_cli_choices.decrypt_policy, + .present = &indexing_cli_choices.decrypt_policy_set, .name = "decrypt" }, { } }; @@ -111,21 +111,21 @@ notmuch_process_shared_indexing_options (notmuch_database_t *notmuch, g_mime_3_u { if (indexing_cli_choices.opts == NULL) indexing_cli_choices.opts = notmuch_database_get_default_indexopts (notmuch); - if (indexing_cli_choices.try_decrypt_set) { + if (indexing_cli_choices.decrypt_policy_set) { notmuch_status_t status; if (indexing_cli_choices.opts == NULL) return NOTMUCH_STATUS_OUT_OF_MEMORY; - status = notmuch_indexopts_set_try_decrypt (indexing_cli_choices.opts, indexing_cli_choices.try_decrypt); + status = notmuch_indexopts_set_decrypt_policy (indexing_cli_choices.opts, indexing_cli_choices.decrypt_policy); if (status != NOTMUCH_STATUS_SUCCESS) { fprintf (stderr, "Error: Failed to set index decryption policy to %s. (%s)\n", - indexing_cli_choices.try_decrypt ? "True" : "False", notmuch_status_to_string (status)); + indexing_cli_choices.decrypt_policy ? "True" : "False", notmuch_status_to_string (status)); notmuch_indexopts_destroy (indexing_cli_choices.opts); indexing_cli_choices.opts = NULL; return status; } } #if (GMIME_MAJOR_VERSION < 3) - if (indexing_cli_choices.opts && notmuch_indexopts_get_try_decrypt (indexing_cli_choices.opts)) { + if (indexing_cli_choices.opts && notmuch_indexopts_get_decrypt_policy (indexing_cli_choices.opts)) { const char* gpg_path = notmuch_config_get_crypto_gpg_path (config); if (gpg_path && strcmp(gpg_path, "gpg")) fprintf (stderr, "Warning: deprecated crypto.gpg_path is set to '%s'\n" -- 2.15.0 _______________________________________________ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch