On Tue, 18 Aug 2009 10:52:02 +0200, Harold wrote:

>Dear Geany devs,
>
>I've been tinkering with the regex in tagmanager/php.c to better
>detect functions. My regex is still far from perfect, but I'll only
>detect functions prefixed with either whitespace or on or more of
>these: public, protected, private or static.
>
>I'd guess a better approach would be to detect the opening parenthesis
>'(' after the function name...
>
>The first 2 commented 'functions' are indexed by the current regex as
>the functions 'should' and 'would'.

Giving that I'm not very good in reading and understanding regular
expressions, the patch looks good to me.
The approach is pretty much ok, I think. At least it should make the
current situation much better even if not perfect.

Any objections in committing it? Any other PHP users around?

Regards,
Enrico

-- 
Get my GPG key from http://www.uvena.de/pub.asc

Attachment: pgp7GByFpBZAu.pgp
Description: PGP signature

_______________________________________________
Geany mailing list
[email protected]
http://lists.uvena.de/cgi-bin/mailman/listinfo/geany

Reply via email to