Dear Geany devs,
After a (short) period of getting used to it, I really like the new feature to automatically insert quotes and brackets! It saves me a lot of keypresses to construct a piece of code ;)
Except for this example: "$var[] = 'test';"
Instead of typing [] I/we now have to type [ and right arrow key. Same goes for "$empty = ''", etc...
My request:
If the next character is the same as the one Geany has inserted(/closed), overwrite the previously inserted one.
So when typing "$test = '';" it is exactly as typed and not "$test = '';''"
What do you guys think of such an 'additional' feature?
-H-
On Thu, 18 Dec 2008 14:40:54 +0100, Harold Aling geany@sait.nl wrote:
Dear Geany devs,
After a (short) period of getting used to it, I really like the new feature to automatically insert quotes and brackets! It saves me a lot of keypresses to construct a piece of code ;)
Except for this example: "$var[] = 'test';"
Instead of typing [] I/we now have to type [ and right arrow key. Same goes for "$empty = ''", etc...
My request:
If the next character is the same as the one Geany has inserted (/closed), overwrite the previously inserted one.
Sounds like a good idea.
Attached is a patch which might fix it but I didn't test it well and I want you (Harold, Guillaume, whoever else is using this feature) to test it before, it might need some tweaking.
Regards, Enrico
what will happen with stuff of the form funcA(funcB))
On Fri, Dec 19, 2008 at 7:23 AM, Enrico Tröger enrico.troeger@uvena.de wrote:
On Thu, 18 Dec 2008 14:40:54 +0100, Harold Aling geany@sait.nl wrote:
Dear Geany devs,
After a (short) period of getting used to it, I really like the new feature to automatically insert quotes and brackets! It saves me a lot of keypresses to construct a piece of code ;)
Except for this example: "$var[] = 'test';"
Instead of typing [] I/we now have to type [ and right arrow key. Same goes for "$empty = ''", etc...
My request:
If the next character is the same as the one Geany has inserted (/closed), overwrite the previously inserted one.
Sounds like a good idea.
Attached is a patch which might fix it but I didn't test it well and I want you (Harold, Guillaume, whoever else is using this feature) to test it before, it might need some tweaking.
Regards, Enrico
-- Get my GPG key from http://www.uvena.de/pub.asc
Geany mailing list Geany@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany
On Thu, 18 Dec 2008 21:23:10 +0100, Enrico Tröger enrico.troeger@uvena.de wrote:
On Thu, 18 Dec 2008 14:40:54 +0100, Harold Aling geany@sait.nl wrote:
My request:
If the next character is the same as the one Geany has inserted (/closed), overwrite the previously inserted one.
Attached is a patch which might fix it but I didn't test it well and I want you (Harold, Guillaume, whoever else is using this feature) to test it before, it might need some tweaking.
Enrico,
Just tested it a bit. First thoughts: * There's a bit of a lag when the 'patch' kicks in * I believe the patch looks at the next character on a line so if you're prefixing a single quote with another quote that won't work.
-H-
On Fri, 19 Dec 2008 11:48:11 +0100, Harold Aling geany@sait.nl wrote:
On Thu, 18 Dec 2008 21:23:10 +0100, Enrico Tröger enrico.troeger@uvena.de wrote:
On Thu, 18 Dec 2008 14:40:54 +0100, Harold Aling geany@sait.nl wrote:
My request:
If the next character is the same as the one Geany has inserted (/closed), overwrite the previously inserted one.
Attached is a patch which might fix it but I didn't test it well and I want you (Harold, Guillaume, whoever else is using this feature) to test it before, it might need some tweaking.
Enrico,
Just tested it a bit. First thoughts:
- There's a bit of a lag when the 'patch' kicks in
- I believe the patch looks at the next character on a line so if
you're prefixing a single quote with another quote that won't work.
Yes, and the patch fails also for the case Gordon pointed out. This is why I told it's necessary to test it before.
Well, I won't spend more time on this, sorry. This can get pretty nasty to implement in a sane way. Let's hope someone who is using this feature actually, will work out a better patch.
Regards, Enrico