Le 05/02/2012 13:51, Eugene Arshinov a écrit :
Hello all.
Hey Eugene,
I have several suggestions and questions about certain line operations implemented in Geany.
- Recently I found that "move lines up/down" command does not work
properly for the last line not ending with a newline. You can easily check it yourself. Couple of minutes ago I made a pull request [1] with an implementation of `editor.c:move_lines()` which handles the problem. Dear unknown someone, please review and pull if it's okay.
I know this problem, but it needed a so big code refactoring it hurts (I want as a proof the fact your rewrite is... huge), so... I just postponed it. Ok, shame on me.
I haven't reviewed the new code yet, but I'll do. Just as a note, Scintilla has a command to do that that suffers (suffered?) of the exact same bug we had. Maybe it'd be good to fix their copy and use the SCI message in place of our code.
- 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?
Regards, Colomban
- Here there should have been a point about some bug reports we have
in Geany bug tracker, referring to "reflow paragraph" glitches. As this message is already too long, I decided to designate a separate message for that (later).
Thank you for reading :) Please write your thoughts about 1) and 2)
-- Best regards, Eugene. _______________________________________________ Geany-devel mailing list Geany-devel@uvena.de https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel