This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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



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.
:)



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]