Le 19/02/2012 20:01, Eugene Arshinov a écrit :
On Sun, 19 Feb 2012 17:24:00 +0100 Colomban Wendling lists.ban@herbesfolles.org wrote:
[...]
The commands indeed work similarly to our own `move_lines` function. I posted pull request #24 [3] which removes `move_lines` function and leverages the commands. I hope it can be committed so that I can switch my effort of improving "Move lines" feature from Geany to Scintilla.
Looks fine to -- apart of course it doesn't fix the initial problem yet. At least so there would be only one copy of the functionality.
So just to be sure we understand each other: I drop your previous pull request (#21), apply this one (#24) and wait for your patch to be in Scintilla?
Yes. But I don't promise that I'll post a request to Scintilla *soon*…
No big deal IMO regarding how long this bug existed without any reports about it. See, while I use the feature quite often, I ran into the bug only once in real situation.
So, it's now applied; Waits for Scintilla update :)
Cheers, Colomba