On Mon, 20 Feb 2012 20:03:17 +0100 Colomban Wendling <lists....@herbesfolles.org> wrote:
> Le 19/02/2012 20:01, Eugene Arshinov a écrit : > > On Sun, 19 Feb 2012 17:24:00 +0100 > > Colomban Wendling <lists....@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 :) > Hello. There is some good news. Recently they applied a patch with an updated implementation of MOVESELECTEDLINES [1]. I guess, it will be in the next release. [1]:<http://sf.net/tracker/?func=detail&atid=102439&aid=3511023&group_id=2439> -- Best regards, Eugene. _______________________________________________ Geany-devel mailing list Geany-devel@uvena.de https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
