Subject: Re: [PATCH] .mailmap: record canonical email address for Sven Strickroth

From: Jonathan Nieder <[email-masked]>
Date: Tue, 20 Nov 2018 13:58:03 -0800

Hi,

Sven Strickroth wrote:

> Subject: .mailmap: record canonical email address for Sven Strickroth
>
> Signed-off-by: Sven Strickroth <[email-masked]>
> ---
>  .mailmap | 2 ++
>  1 file changed, 2 insertions(+)

Thanks for taking care of it.

[...]
> --- a/.mailmap
> +++ b/.mailmap
> @@ -235,6 +235,8 @@ Steven Grimm <[email-masked]> <[email-masked]>
>  Steven Grimm <[email-masked]> [email-masked]
>  Steven Walter <[email-masked]> <[email-masked]>
>  Steven Walter <[email-masked]> <[email-masked]>
> +Sven Strickroth <[email-masked]> <[email-masked]>

Is the above line needed?  It's not clear to me from the commit message
what it does.

> +Sven Strickroth <[email-masked]> <[email-masked]>

This line looks good.

>  Sven Verdoolaege <[email-masked]> <[email-masked]>
>  Sven Verdoolaege <[email-masked]> <[email-masked]>
>  SZEDER G??bor <[email-masked]> <[email-masked]>

This line does not seem to have survived mailing.  Fortunately it's a
context line, but thought I should mention it for the future.

Thanks and hope that helps,
Jonathan