From: Rafael Garcia-Suarez <rgarciasuarez@gmail.com>
Date: Fri, 06 Feb 2009 10:31:39 GMT
Subject: Re: [PATCH] gitweb: Better regexp for SHA-1 committag match
Message-ID: <b77c1dce0902060231u358587d5o940eb322fde52a68@mail.gmail.com>
In-Reply-To: <200902061126.18418.jnareb@gmail.com>

2009/2/6 Jakub Narebski <jnareb@gmail.com>:
> Dnia piątek 6. lutego 2009 10:49, Rafael Garcia-Suarez napisał:
>> 2009/2/6 Jakub Narebski <jnareb@gmail.com>:
>
>> > Make SHA-1 regexp to be turned into hyperlink (the SHA-1 committag)
>> > to match word boundary at the beginning and the end.  This way we
>> > reduce number of false matches, for example we now don't match
>> > 0x74a5cd01 which is hex decimal (for example memory address),
>> > but is not SHA-1.
>>
>> Further suggestion: you could also turn the final \b into (\b|\@),
>
> You meant \b -> \b(?!\@), didn't you?  Word boundary _not_ followed
> by '@', and not word boundary _OR_ '@' as you wrote...

Ah right, shame on me.