[Geany-devel] Line operations

Colomban Wendling lists.ban at xxxxx
Mon Feb 20 19:03:17 UTC 2012


Le 19/02/2012 20:01, Eugene Arshinov a écrit :
> On Sun, 19 Feb 2012 17:24:00 +0100
> Colomban Wendling <lists.ban at 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



More information about the Devel mailing list