Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:
> This way we're only choosing a default in one place.
> ---
> notmuch-config.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/notmuch-config.c b/notmuch-config.c
> index d252bb2..7cd19a7 100644
> --- a/notmuch-config.c
> +++ b/notmuch-config.c
> @@ -418,7 +418,8 @@ notmuch_config_open (void *ctx,
> }
>
> if (notmuch_config_get_crypto_gpg_path (config) == NULL) {
> - notmuch_config_set_crypto_gpg_path (config, "gpg");
> + _notmuch_crypto_t crypto = { .gpg_path = NULL };
> + notmuch_config_set_crypto_gpg_path (config, _notmuch_crypto_get_gpg_path (&crypto));
> }
I'm not sure if it's worth it, but this particular usage would be
cleaner if _notmuch_crypto_get_path allowed a NULL argument.
d