Subject: Re: [PATCH v2 2/2] gpg-interface: remove an unnecessary NULL initialization

From: Junio C Hamano <[email-masked]>
Date: Thu, 05 Feb 2026 17:59:13 -0800

Collin Funk <[email-masked]> writes:

> We assign this variable unconditionally, so we do not need to assign it
> to NULL where it is declared.

The latter "assign" is better phrased as "initialize", but other
than that this looks great.

Thanks.

>
> Signed-off-by: Collin Funk <[email-masked]>
> ---
>  gpg-interface.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gpg-interface.c b/gpg-interface.c
> index 377c0cf49f..87fb6605fb 100644
> --- a/gpg-interface.c
> +++ b/gpg-interface.c
> @@ -398,7 +398,7 @@ static void parse_ssh_output(struct signature_check *sigc)
>  {
>  	const char *line, *principal, *search;
>  	char *to_free;
> -	const char *key = NULL;
> +	const char *key;
>  
>  	/*
>  	 * ssh-keygen output should be: