[Geany-devel] Strip trailing blank lines (patch included)

[Eugene Arshinov]

Hi Enrico.

Enrico Tro"ger wrote:
> On Wed, 26 Aug 2009 19:48:30 +0400, Eugene wrote:
>
> Hey,
>
>   
>> I wrote new `editor_strip_trailing_newlines' function (see the
>> attached patch). Currently I call it directly from
>> `document_save_file' if `strip_trailing_spaces' setting is set. Of
>> course, it's not how it should be. Maybe separate menu item and option
>> should be added, like for trailing spaces. Alternative way is to
>> combine those, i.e., rename "Strip trailing spaces" to "Remove
>> excessive whitespace" :-) These all are usability issues, with which I
>> don't like to deal. If any of the developers has an opinion upon
>> those, I'll consider it and update the patch.
>>     
>
> Hmm, I'm not sure that would be an often used feature but of course it
> can be useful and you already wrote the code (and used it on the
> modified files in your patch...I like to have two or three trailing
> blank links at EOF :D).
> Back to the topic, what about putting this into a plugin?
>
> Either as a separate plugin which might be a bit overhead, or as part
> of the addons plugin which aims to collect such small pieces of
> additions or as a new (core) plugin into which we could move the
> existing other both related features, too.
>
> Just some thoughts. Opinions?
Putting it to the addons plugin sounds reasonable. I'll do it this or 
next week.

Regards,
Eugene.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.geany.org/pipermail/devel/attachments/20090827/3aa68a0a/attachment.html>