Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)
On Sun, 7 Mar 2010 19:20:37 +0100 Enrico Tröger enrico.troe...@uvena.de wrote: You put a weird pattern: [public|protected|private|static]* If you wanted to group, you should use () I didn't want to group, we don't need to know the actual modifier, we just want to allow only this subset. And yes, it was a group before but there is just no need to. I think Can is right, [] brackets are only for character ranges, not string matching. So () brackets are necessary for matching even when ignoring the group. Regards, Nick ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)
From: Nick Treleaven nick.trelea...@btinternet.com Subject: Re: [Geany-devel] [Patch] - Improve PHP parser (call tips) To: geany-devel@uvena.de Date: Monday, March 8, 2010, 12:18 PM On Sun, 7 Mar 2010 19:20:37 +0100 Enrico Tröger enrico.troe...@uvena.de wrote: You put a weird pattern: [public|protected|private|static]* If you wanted to group, you should use () I didn't want to group, we don't need to know the actual modifier, we just want to allow only this subset. And yes, it was a group before but there is just no need to. I think Can is right, [] brackets are only for character ranges, not string matching. So () brackets are necessary for matching even when ignoring the group. Actually, I'm very uncomfortable with my name being recorded in that patch's log (r4729). That regex is funny, it exhibits an ignorance in the subject and it's not what I submitted. So, I demand my name removed from the logs for that patch. -- Can Koy ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)
well, who the hell are you? --- On Mon, 3/8/10, Thomas Martitz thomas.mart...@student.htw-berlin.de wrote: From: Thomas Martitz thomas.mart...@student.htw-berlin.de Subject: Re: [Geany-devel] [Patch] - Improve PHP parser (call tips) To: geany-devel@uvena.de Date: Monday, March 8, 2010, 7:46 PM Am 08.03.2010 20:23, schrieb Can Koy: From: Nick Treleavennick.trelea...@btinternet.com Subject: Re: [Geany-devel] [Patch] - Improve PHP parser (call tips) To: geany-devel@uvena.de Date: Monday, March 8, 2010, 12:18 PM On Sun, 7 Mar 2010 19:20:37 +0100 Enrico Trögerenrico.troe...@uvena.de wrote: You put a weird pattern: [public|protected|private|static]* If you wanted to group, you should use () I didn't want to group, we don't need to know the actual modifier, we just want to allow only this subset. And yes, it was a group before but there is just no need to. I think Can is right, [] brackets are only for character ranges, not string matching. So () brackets are necessary for matching even when ignoring the group. Actually, I'm very uncomfortable with my name being recorded in that patch's log (r4729). That regex is funny, it exhibits an ignorance in the subject and it's not what I submitted. So, I demand my name removed from the logs for that patch. -- Can Koy Then please submit a patch to the subversion project to enable post-editing of commit messages, since this is not possible currently. But honestly, I don't see why this is a big deal. Maybe we can concentrate on improving Geany and not insisting on minor things like this. Best regards. ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)
Am Montag, den 08.03.2010, 12:15 -0800 schrieb Can Koy: well, who the hell are you? Can, Thomas is contributing to the Geany project for a very long time. As developer as well as a tester. I do not see any reason to ask such questions here. Especially not in that caustically way. Actually, I'm very uncomfortable with my name being recorded in that patch's log (r4729). That regex is funny, it exhibits an ignorance in the subject and it's not what I submitted. So, I demand my name removed from the logs for that patch. Maybe it could be, Enrico modified some things on your patch before he commited it, I don't know. If this is the case, he maybe forgot to write the commit message as based on patch from ... as he usually does. Well, that could happen to anyone, we're all just humans. So, thanks very much for your understanding. Anyway, Enrico has every right to modify code before applying it onto Geany. As you may have noticed he is one of the main authors of Geany and initiated the project. He still is one of the last ones who decide if code makes it into Geany or not. Geany is Free Software under GPL and any contribution should be too. Thus I see no reason why patches may not be modified before applying them to the project. If the modification is an improvement or not is a technical question which I am not asking for at present. We're open to discuss any technical issue. :) [snip] Then please submit a patch to the subversion project to enable post-editing of commit messages, since this is not possible currently. But honestly, I don't see why this is a big deal. Maybe we can concentrate on improving Geany and not insisting on minor things like this. The fact that your name is mentioned in the commit message can not be undone due to the technical reasons Thomas wrote. I'm afraid you will have to live with that. Please keep in mind that Free Software projects are not mainly intended to make someones name public or not. Such projects are intended to provide good software for the user. - Thus a lot of your patches we're very appreciated. - In most Free Software projects it is the case, that the developers actually are people who have very long working days and do their efforts for the projects in their free time after work. This actually is also the case for the Geany project. I think, a bit of understanding from your side would be appreciated by anyone here. Mistakes can happen to anyone. Regards, Dominic -- Dominic Hopf dma...@googlemail.com http://dominichopf.de/ Key Fingerprint: A7DF C4FC 07AE 4DDC 5CA0 BD93 AAB0 6019 CA7D 868D signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)
Gentlemen please, no flame wars. @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. @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. Cheers Lex On 9 March 2010 07:15, Can Koy can...@ymail.com wrote: well, who the hell are you? --- On Mon, 3/8/10, Thomas Martitz thomas.mart...@student.htw-berlin.de wrote: From: Thomas Martitz thomas.mart...@student.htw-berlin.de Subject: Re: [Geany-devel] [Patch] - Improve PHP parser (call tips) To: geany-devel@uvena.de Date: Monday, March 8, 2010, 7:46 PM Am 08.03.2010 20:23, schrieb Can Koy: From: Nick Treleavennick.trelea...@btinternet.com Subject: Re: [Geany-devel] [Patch] - Improve PHP parser (call tips) To: geany-devel@uvena.de Date: Monday, March 8, 2010, 12:18 PM On Sun, 7 Mar 2010 19:20:37 +0100 Enrico Trögerenrico.troe...@uvena.de wrote: You put a weird pattern: [public|protected|private|static]* If you wanted to group, you should use () I didn't want to group, we don't need to know the actual modifier, we just want to allow only this subset. And yes, it was a group before but there is just no need to. I think Can is right, [] brackets are only for character ranges, not string matching. So () brackets are necessary for matching even when ignoring the group. Actually, I'm very uncomfortable with my name being recorded in that patch's log (r4729). That regex is funny, it exhibits an ignorance in the subject and it's not what I submitted. So, I demand my name removed from the logs for that patch. -- Can Koy Then please submit a patch to the subversion project to enable post-editing of commit messages, since this is not possible currently. But honestly, I don't see why this is a big deal. Maybe we can concentrate on improving Geany and not insisting on minor things like this. Best regards. ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] Scintilla 2.03 Update
Enrico Tröger a écrit : Hey guys, I just updated our Scintilla copy to its latest version 2.03. This brings some bugfixes but most importantly it adds a new feature: virtual spaces. There is a new preference in the Preferences dialog, Editor-Display tab to enable and disable it. […] There may be some regressions or other small problems with the update. If you find anything, just report it. Regards, Enrico Hi, Interesting stuff indeed! But there's a major (small, but I use it often) change that can (is, for me) be very annoying: middle-click somewhere /always/ places the cursor on the click position, even if virtual spaces are disabled. Why is this a big deal? because when pasting with the mouse (X-style), the pasted chunk is placed exactly where the mouse where at the click rather than at the start of the line - well, virtual space. Regards, Colomban PS: if I should have reported this through the bugtracker, please tell me so I could do so last time :) ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)
On Mon, 8 Mar 2010 12:18:39 +, Nick wrote: On Sun, 7 Mar 2010 19:20:37 +0100 Enrico Tröger enrico.troe...@uvena.de wrote: You put a weird pattern: [public|protected|private|static]* If you wanted to group, you should use () I didn't want to group, we don't need to know the actual modifier, we just want to allow only this subset. And yes, it was a group before but there is just no need to. I think Can is right, [] brackets are only for character ranges, not string matching. So () brackets are necessary for matching even when ignoring the group. Just for the records: Yup, we talked about that yesterday afterwards on IRC and once he told me about that mistake, it was clear. Regards, Enrico -- Get my GPG key from http://www.uvena.de/pub.asc pgpNtcmfd6qL1.pgp Description: PGP signature ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)
On Mon, 8 Mar 2010 12:15:39 -0800 (PST), Can wrote: well, who the hell are you? As Dominic said, Thomas is a regular Geany user and contributor, a nice guy and not less important, just a subscriber to this list as you and I are as well. This is a public mailing list, where anyone can state his opinion. Please keep this in mind. Regards, Enrico -- Get my GPG key from http://www.uvena.de/pub.asc pgpyynhdulEEv.pgp Description: PGP signature ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] [Patch] - Improve PHP parser (call tips)
On Mon, 8 Mar 2010 11:23:30 -0800 (PST), Can wrote: From: Nick Treleaven nick.trelea...@btinternet.com Subject: Re: [Geany-devel] [Patch] - Improve PHP parser (call tips) To: geany-devel@uvena.de Date: Monday, March 8, 2010, 12:18 PM On Sun, 7 Mar 2010 19:20:37 +0100 Enrico Tröger enrico.troe...@uvena.de wrote: You put a weird pattern: [public|protected|private|static]* If you wanted to group, you should use () I didn't want to group, we don't need to know the actual modifier, we just want to allow only this subset. And yes, it was a group before but there is just no need to. I think Can is right, [] brackets are only for character ranges, not string matching. So () brackets are necessary for matching even when ignoring the group. Actually, I'm very uncomfortable with my name being recorded in that patch's log (r4729). That regex is funny, it exhibits an ignorance in the subject and it's not what I submitted. So, I demand my name Well, yes, obviously I made a mistake in the commited regexp. One sentence about my motivation for the changes between the current version in trunk and those of your patch: before your patch, 'public function foo' was matched, after your patch those functions didn't match anymore. I tried to improve this by the changes we are talking about. Without a doubt, I did it wrong. Sorry for crediting my changes to your changes with your name. Usually, people appreciate this more than not being credited at all. But in any case, it's too late. The only thing I don't understand is your indignation now, after the yesterday's commit, you showed up in IRC and you probably remember we talked about exactly this then. For me, it sounded like you want to fix my mistake, though probably I misunderstood you. Just for the records and those who were not on IRC, here is the relevant log (the full log can be found on http://irc.geany.org/logs/log_20100307.html): 18:25 cankoy has joined #geany 18:43 cankoy eht16: a bracket expression in regex matches single char. ,not strings. 18:43 eht16 dammit, you are right 18:44 cankoy actually your pattern works, but pnly by luck :-) 18:44 eht16 gimme a second to answer to your other patches, then we'll fix it 18:44 eht16 hates regexps 18:44 eht16 not because they are bad but because eht16 always use them wrong 18:44 cankoy well, that's why I asked you not to do everything :-) 18:45 eht16 haha 18:58 eht16 cankoy: is there a difference between [[:space:]]* and [ \t]* ? 18:59 cankoy yes by definition, but not in current mode of regex in ctags 18:59 eht16 I think we should use one of both then 19:00 cankoy I was thinking about doing \n matches in the future, that's why I kept [:space:] 19:00 cankoy feel free to change 19:01 eht16 I don't get it working for function foo() 19:01 cankoy eht16: my initial patch was matching that. 19:01 eht16 no 19:02 eht16 I thought I tested this 19:02 eht16 well, I try it again to get sure 19:04 eht16 ah, yeah, of course 19:04 eht16 but it didn't work with modifiers 19:05 cankoy eht16: let me take this over. You don't have to commit until it matures. 19:05 eht16 ok, just fix it as you think it should be and sorry for destroying your work :( 19:05 cankoy np, I know you don't have much time. 19:31 cankoy eht16: Harold's regex is also not correct, e.g. it does not match public static function... I'll try to fix that too. 19:32 cankoy eht16: no, sorry. It matches that actually. 19:32 eht16 I didn't even know that's possible in PHP :) I thought we agreed that I made a mistake and this can be fixed. I don't understand why you now raise such a hard voice. removed from the logs for that patch. Done, at least for the ChangeLog. As the others said, removing it from the SVN commit message isn't possible. Regards, Enrico -- Get my GPG key from http://www.uvena.de/pub.asc pgpyWWfethX9n.pgp Description: PGP signature ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] Scintilla 2.03 Update
On Mon, 08 Mar 2010 23:22:08 +0100, Colomban wrote: Hey, Interesting stuff indeed! But there's a major (small, but I use it often) change that can (is, for me) be very annoying: middle-click somewhere /always/ places the cursor on the click position, even if virtual spaces are disabled. Why is this a big deal? because when pasting with the mouse (X-style), the pasted chunk is placed exactly where the mouse where at the click rather than at the start of the line - well, virtual space. Besides the fact that you don't like the behaviour, I'm pretty sure this has been the behaviour for a quite some time. To verify, I just tried an old 0.17 build and it behaves the same. So, I think this is at least no new bug/regression of the Scintilla update. I think to remember we changed this some time in the past from the behaviour you want and the current one. Changing it back probably would cause new bug reports of those users who like the current behaviour :). So, as often, the solution seems to be a new preference. Still, I might be wrong. I'll have a closer look tomorrow and will try to find the corresponding bug report (I think to remember there was one). PS: if I should have reported this through the bugtracker, please tell me so I could do so last time :) No worries, posting bugs here on the list is fully ok. Regards, Enrico -- Get my GPG key from http://www.uvena.de/pub.asc pgpZwAms6lRiW.pgp Description: PGP signature ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
Re: [Geany-devel] Scintilla 2.03 Update
Hi, Enrico Tröger wrote: On Mon, 08 Mar 2010 23:22:08 +0100, Colomban wrote: Hey, Interesting stuff indeed! But there's a major (small, but I use it often) change that can (is, for me) be very annoying: middle-click somewhere /always/ places the cursor on the click position, even if virtual spaces are disabled. Why is this a big deal? because when pasting with the mouse (X-style), the pasted chunk is placed exactly where the mouse where at the click rather than at the start of the line - well, virtual space. Besides the fact that you don't like the behaviour, I'm pretty sure this has been the behaviour for a quite some time. To verify, I just tried an old 0.17 build and it behaves the same. So, I think this is at least no new bug/regression of the Scintilla update. […] No no, you misunderstood me, it's not about click position vs. cursor position, for me it's all OK (even more, better than pasting at cursor position). No, what I'm talking about is clearly virtual-spaces-related: if you paste something with middle-click by clicking after the end of a line (at the right of the EOL), it was used to paste at the end of line. Now, it pastes at the exact click position, then it adds as much spaces as it needs from the former EOL to the click position. Just try to middle-click somewhere outside line bounds (even without data to paste), you'll see ;) Regards, Colomban ___ Geany-devel mailing list Geany-devel@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
