Re: [PATCH 2/2] pretty: add '%aA' to show domain-part of email addresses

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 4, 2023, 01:54 UTC
Message-ID: <xmqqy1fegu5i.fsf@gitster.g>
In-Reply-To: <4e3a5661-cba0-4849-8340-06d8d4094f48@gmail.com>

Andy Koppe <andy.koppe@gmail.com> writes:

> I'm not sure that this is the right way to handle a missing '@' here
> actually, because %al already returns the whole email field in that
> case, which makes sense as the likes of the 'mail' command would
> interpret it as a local username.

We could expand "%am" to \C-h (\010) so that "%al@%am" would end up displaying the same as "%al" but that would be way too cute for its own worth ;-)

It is unfortunate that "%al@%am" cannot be the same as "%ae" for local-only address, but giving an empty string for "%am" if "%ae" is local-only would be the best we could do for our users, and certainly much better than giving the same as "%ae", as you said above.

Thanks.