Re: [testsuite] gdb.c++/templates.exp

[Daniel Berlin <dberlin@dberlin.org>]

On Wednesday, July 9, 2003, at 11:15 AM, Daniel Berlin wrote:

>
> On Wednesday, July 9, 2003, at 11:05 AM, Michael Elizabeth Chastain 
> wrote:
>
>> Hi Michal,
>>
>> This part is not quite correct:
>>
>>   \\(unsigned ?(long|)\\);
>>
>> It would accept 'unsignedlong' which would be wrong.
>
> It shouldn't, unless the regex isn't whitespace sensitive

Just to be clear, this regex should accept nothing, "long", "unsigned 
long", and "unsigned".
This may be *wrong*, but it certainly shouldn't accept "unsignedlong".
The ? should be applying to "unsigned ", not just the whitespace.

>>
>> Can you try:
>>
>>   \\(unsigned( long|)\\);
>
> Now *this* would accept unsignedlong, unless the regexps in expect are 
> unlike any other system i've seen.
> This is because the alternation would be between " long" and "", *not* 
> "long" and "" (IE your regex would not be equivalent to 
> <space>long|<space>
>
I was on crack here and read it wrong, time to take my medication.
:)