Gentlemen please, no flame wars.<br><br>@Can, Thomas is correct, a commit can't be undone, even if its revented the message is still in the records.  Better to concentrate on explaining why the regex needs changing.  Everyone makes mistakes.<br>

<br>@Enrico, Nick and Can are right, the regex [abc|def] matches any one of characters 'a', 'b', 'c', '|', 'd', 'e', 'f' not the sequences "abc or "def" that (abc|def) matches.  There is no way of grouping without saving a match unless the regex engine supports the non-capturing (?:  ) operator.<br>
<br>Cheers<br>Lex<br><br><div class="gmail_quote">
On 9 March 2010 07:15, Can Koy <span dir="ltr"><<a href="mailto:cankoy@ymail.com" target="_blank">cankoy@ymail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

well, who the hell are you?<br>
<br>
<br>
--- On Mon, 3/8/10, Thomas Martitz <<a href="mailto:thomas.martitz@student.HTW-Berlin.de" target="_blank">thomas.martitz@student.HTW-Berlin.de</a>> wrote:<br>
<br>
> From: Thomas Martitz <<a href="mailto:thomas.martitz@student.HTW-Berlin.de" target="_blank">thomas.martitz@student.HTW-Berlin.de</a>><br>
<div>> Subject: Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)<br>
> To: <a href="mailto:geany-devel@uvena.de" target="_blank">geany-devel@uvena.de</a><br>
</div>> Date: Monday, March 8, 2010, 7:46 PM<br>
<div><div></div><div>> Am 08.03.2010 20:23, schrieb Can<br>
> Koy:<br>
> >   <br>
> >> From: Nick Treleaven<<a href="mailto:nick.treleaven@btinternet.com" target="_blank">nick.treleaven@btinternet.com</a>><br>
> >> Subject: Re: [Geany-devel] [Patch] - Improve PHP<br>
> parser (call tips)<br>
> >> To: <a href="mailto:geany-devel@uvena.de" target="_blank">geany-devel@uvena.de</a><br>
> >> Date: Monday, March 8, 2010, 12:18 PM<br>
> >> On Sun, 7 Mar 2010 19:20:37 +0100<br>
> >> Enrico Tröger<<a href="mailto:enrico.troeger@uvena.de" target="_blank">enrico.troeger@uvena.de</a>><br>
> >> wrote:<br>
> >><br>
> >>     <br>
> >>>> You put a weird pattern:<br>
> >>>> [public|protected|private|static]*<br>
> >>>> If you wanted to group, you should use ()<br>
> >>>>         <br>
> >>> I didn't want to group, we don't need to know<br>
> the actual modifier, we<br>
> >>> just want to allow only this subset. And yes,<br>
> it was a group before but<br>
> >>> there is just no need to.<br>
> >>>       <br>
> >> I think Can is right, [] brackets are only for<br>
> character ranges, not<br>
> >> string matching. So () brackets are necessary for<br>
> matching even when<br>
> >> ignoring the group.<br>
> >><br>
> >>     <br>
> > Actually, I'm very uncomfortable with my name being<br>
> recorded in that patch's log (r4729). That regex is funny,<br>
> it exhibits an ignorance in the subject and it's not what I<br>
> submitted. So, I demand my name removed from the logs for<br>
> that patch.<br>
> ><br>
> > --<br>
> > Can Koy<br>
> ><br>
> ><br>
> >   <br>
><br>
><br>
> Then please submit a patch to the subversion project to<br>
> enable<br>
> post-editing of commit messages, since this is not possible<br>
> currently.<br>
><br>
> But honestly, I don't see why this is a big deal. Maybe we<br>
> can<br>
> concentrate on improving Geany and not insisting on minor<br>
> things like this.<br>
><br>
> Best regards.<br>
> _______________________________________________<br>
> Geany-devel mailing list<br>
> <a href="mailto:Geany-devel@uvena.de" target="_blank">Geany-devel@uvena.de</a><br>
> <a href="http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
><br>
<br>
<br>
<br>
_______________________________________________<br>
Geany-devel mailing list<br>
<a href="mailto:Geany-devel@uvena.de" target="_blank">Geany-devel@uvena.de</a><br>
<a href="http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel" target="_blank">http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel</a><br>
</div></div></blockquote></div><br>