[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: FYI, dfaexec bug-fix coming up
From: |
Jim Meyering |
Subject: |
Re: FYI, dfaexec bug-fix coming up |
Date: |
Mon, 08 Mar 2010 15:17:48 +0100 |
Paolo Bonzini wrote:
> On 03/08/2010 02:49 PM, Jim Meyering wrote:
>> This adds a failing test.
>> I was flabbergasted to see that this test fails.
>>
>> The only reason it doesn't fail with Debian's patches is
>> because they've turned off the DFA matcher altogether.
>> Set the envvar to show it fail there, too:
>>
>> $ echo AA| GREP_USE_DFA=1 LC_ALL=en_US.UTF-8 /bin/grep '\([A]\|[B]\)\{2\}'
>> [Exit 1]
>> $
>>
>> Thanks to Paolo for the test.
>
> Can you check if it fixes https://bugzilla.redhat.com/544406 too?
Yes, it does:
$ echo 1 2 3 | LC_ALL=de_DE.UTF-8 src/grep -E '^([[:digit:]]+[[:space:]]+){2}'
1 2 3
Even if it's the same bug, that regexp is different enough
that I'll add it to the test, along with the other.