From: David Bremner <david@tethera.net>
To: Daniel Kahn Gillmor <dkg@fifthhorseman.net>,
 Notmuch Mail <notmuch@notmuchmail.org>
Subject: Re: [PATCH v3 05/16] Use a blank _notmuch_crypto to choose the
 default gpg_path
In-Reply-To: <1454272801-23623-6-git-send-email-dkg@fifthhorseman.net>
References: <1454272801-23623-1-git-send-email-dkg@fifthhorseman.net>
 <1454272801-23623-6-git-send-email-dkg@fifthhorseman.net>
User-Agent: Notmuch/0.21+5~gca076ce (http://notmuchmail.org) Emacs/24.5.1
 (x86_64-pc-linux-gnu)
Date: Wed, 10 Feb 2016 07:49:54 -0400
Message-ID: <87oabovl7h.fsf@maritornes.cs.unb.ca>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: list

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