[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 13/34] maint: dfa: convert #if-MBS_SUPPORT (parse_bracket_exp
|
From: |
Jim Meyering |
|
Subject: |
Re: [PATCH 13/34] maint: dfa: convert #if-MBS_SUPPORT (parse_bracket_exp) |
|
Date: |
Thu, 15 Sep 2011 19:52:45 +0200 |
Paolo Bonzini wrote:
> On 09/15/2011 12:27 PM, Jim Meyering wrote:
>> --- a/src/dfa.c
>> +++ b/src/dfa.c
>> @@ -1025,34 +1025,33 @@ parse_bracket_exp (void)
>>
>> colon_warning_state |= (c == ':') ? 2 : 4;
>>
>> -#if MBS_SUPPORT
>> - if (MB_CUR_MAX> 1)
>> + if (!MBS_SUPPORT || MB_CUR_MAX<= 1)
>
> Can test == 1.
The answer is obvious, so perhaps I'm misunderstanding.
That expression will be 1 e.g., whenever MBS_SUPPORT is 0
[PATCH 27/34] build: enable compilation without MBS_SUPPORT, Jim Meyering, 2011/09/15
[PATCH 19/34] maint: dfa: convert #if-MBS_SUPPORT (dfastate), Jim Meyering, 2011/09/15
[PATCH 29/34] maint: stop using skip_test_; use skip_ instead, Jim Meyering, 2011/09/15
[PATCH 11/34] maint: dfa: convert #if-MBS_SUPPORT (dfaexec), Jim Meyering, 2011/09/15
[PATCH 34/34] maint: dfa: remove case-guarding #if-MBS_SUPPORT, Jim Meyering, 2011/09/15
[PATCH 22/34] maint: ensure that MB_CUR_MAX is defined even when !MBS_SUPPORT, Jim Meyering, 2011/09/15