On Wed, Oct 10 2018, Rasmus Villemoes wrote:

> -                             next if $suppress_cc{'sob'} and $what =~ 
> /Signed-off-by/i;
> +                             next if $suppress_cc{'sob'} and $what =~ 
> /^Signed-off-by$/i;
> +                             next if $suppress_cc{'misc-by'}
> +                                     and $what =~ /-by$/i and $what !~ 
> /^Signed-off-by$/i;
>                               next if $suppress_cc{'bodycc'} and $what =~ 
> /Cc/i;

Looks good, FWIW I was curious if this could be:

    next if $suppress_cc{'misc-by'} and $what =~ /(?<!^Signed-off)-by$/;

But found that as soon as you add a /i Perl will barf on it, and in any
case makes sense to be less clever about regex features.

Reply via email to