[Geany-devel] Line operations

Colomban Wendling lists.ban at xxxxx
Thu Feb 23 13:52:10 UTC 2012


Le 23/02/2012 11:08, Eugene Arshinov a écrit :
> On Mon, 6 Feb 2012 11:55:25 +0400
> Eugene Arshinov <earshinov at gmail.com> wrote:
> 
>> On Sun, 05 Feb 2012 20:50:38 +0100
>> Colomban Wendling <lists.ban at herbesfolles.org> wrote:
>>
>>> Le 05/02/2012 13:51, Eugene Arshinov a écrit :
>>>> Hello all.
>>>
>> [snip]
>>>
>>>> 2) I want to raise a question, do we need a "join lines" command?
>>>> This command would be a companion of the existing "reflow
>>>> paragraph" command, but in contrast with the latter, it would
>>>> only join lines, but not split them.
>>>>
>>>> This command may be useful when, let's say, you have a document
>>>> created by someone else who sticks with line breaking and inserts
>>>> \n at column 80.  Suppose that you prefer using line wrapping
>>>> instead and want to remove those \n in a peace of a document which
>>>> you're editing.  The new command would help you a bit.
>>>>
>>>> Implementation of the new "join lines" command could use the bits
>>>> of code already written for "reflow paragraph" (though, those bits
>>>> need to be extracted/refactored first).  Moreover, I already
>>>> implemented it and, if the new command seems useful to anyone, I
>>>> can put my implementation in a pull request.
>>>
>>> Not sure I see a use case for this (read: I probably wouldn't use it
>>> if it simply removes the EOLs), but why not if some wants it.  But
>>> maybe Scintilla already have a message for it and thus we'd only
>>> need to use it?
>>>
>>
>> As Lex already wrote, there is SCI_LINESJOIN.  And that's the command
>> which is already used in Geany to implement "reflow paragraph" (to be
>> more precise, two commands are used: LINESJOIN and LINESSPLIT).
>>
>> By the way, here is a related discussion: "[Geany-devel] Support
>> SCI_LINESJOIN and SCI_LINESSPLIT (patch included)" [1].  The first
>> message is authored by me :)
>>
>> [1]: http://lists.uvena.de/geany-devel/2009-July/000834.html
>>
> 
> I created pull request #26 [2] containing the implementation of the new
> "Join lines" command so that it won't get lost.

Good thing, since I finally took a look at it :)
A few remarks on the PR

Cheers,
Colomban

> 
> [2]: https://github.com/geany/geany/pull/26
> 
>> --
>> Best regards,
>> Eugene.
> _______________________________________________
> Geany-devel mailing list
> Geany-devel at uvena.de
> https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel




More information about the Devel mailing list