"Michael S. Tsirkin" <m...@redhat.com> writes:
> When cccmd is used, old-style suppress-from filter
> is applied by the newer suppress-cc=self isn't.
> Fix this up.
>
> Signed-off-by: Michael S. Tsirkin <m...@redhat.com>
> ---
> git-send-email.perl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/git-send-email.perl b/git-send-email.perl
> index bd13cc8..a138615 100755
> --- a/git-send-email.perl
> +++ b/git-send-email.perl
> @@ -1462,7 +1462,7 @@ sub recipients_cmd {
> $address =~ s/^\s*//g;
> $address =~ s/\s*$//g;
> $address = sanitize_address($address);
> - next if ($address eq $sanitized_sender and $suppress_from);
> + next if ($address eq $sender and $suppress_cc{'self'});
If $suppress_from is defined, $suppress_cc{'self'} gets its value,
so the latter half of this change is very understandable.
The original comparison uses $address that is "sanitized" (whose
definition is roughly "mangled/quoted with rfc2047 etc to fit on
To/Cc headers") against $sanitized_sender, which means for a
$sender = 'Michael S. Tsirkin <m...@redhat.com>';
we used
$sanitized_sender = '"Michael S. Tsirkin" <m...@redhat.com>';
for comparison, but the new code does not quote the $recipient_name
part inside dq for the single dot after the middle name. Is this a
desirable change?
> push @addresses, $address;
> printf("($prefix) Adding %s: %s from: '%s'\n",
> $what, $address, $cmd) unless $quiet;
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
