Re: [TSVN] Re: cpp regex stuff

Douglas Stonham wrote:

> Given I know little to nothing of cpp...
>
> (MULTILINE) .h, .hpp = ^\s*class\s*([\w_]+)|([\w_]+)\([^\)]+\);
> (MULTILINE) .cpp = ([\w_]+)::([\w_]+)
>
> Changed to allow multiple whitespace characters before the word 'class'.
> Changed to capture _ in any of the word groups
> Changed to capture at least one character immediately before :: (or it
> won't match at all)
>
> Does that sound correct?

Yes! Thanks a lot.
Committed in revision 2706.

> Also, is it intentional that function names are only captured if they have
> at least one character in the following brackets? (e.g.
> thisiscaptured(a), thisisnot())
>
> If not, that can be simplified:
> (MULTILINE) .h, .hpp = ^\s*class\s*([\w_]+)|([\w_]+)\(;
>
> Let me know if they test out?

If you leave the ';' at the end, it works ;)

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org