Giuseppe Bilotta <giuseppe.bilotta@xxxxxxxxx> writes: > foreach my $line (@$log) { > - if ($line =~ m/^ *(signed[ \-]off[ \-]by[ :]|acked[ \-]by[ :]|cc[ :])/i) { > + if ($line =~ m/^ *(signed[ \-]off[ \-]by[ :]|(?:trivially[ \-])?acked[ \-]by[ :]|cc[ :])/i) { Wouldn't it make more sense to do something like: if ($line =~ m/^[-a-z]+: (.*)$/i && looks_like_a_name_and_email($1)) { ... do the avatar thing ... } and limit this processing only to the last run of no-blank lines? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html