Jay Soffian wrote: > On Fri, Feb 6, 2009 at 8:03 AM, Johannes Schindelin > <Johannes.Schindelin@xxxxxx> wrote: >> On Fri, 6 Feb 2009, Jakub Narebski wrote: >> >>> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl >>> index f27dbb6..5dcc108 100755 >>> --- a/gitweb/gitweb.perl >>> +++ b/gitweb/gitweb.perl >>> @@ -1364,7 +1364,7 @@ sub format_log_line_html { >>> my $line = shift; >>> >>> $line = esc_html($line, -nbsp=>1); >>> - if ($line =~ m/([0-9a-fA-F]{8,40})/) { >>> + if ($line =~ m/\b([0-9a-fA-F]{8,40})\b(!?\@)/) { >> >> Looks good to me! > > I wonder if just matching lower-case a-f would be sufficient as well? Well... On one hand side git generates always lower-case a-f for SHA-1. On the other hand git _accepts_ upper-case A-F for SHA-1 of object. -- Jakub Narebski Poland -- 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