<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <meta content="text/html;charset=windows-1251"

 http-equiv="Content-Type">

  <title></title>

</head>

<body bgcolor="#ffffff" text="#000000">

Hi Enrico.<br>

<br>

Enrico Tröger wrote:

<blockquote cite="mid:20090826205028.6a070790.enrico.troeger@uvena.de"

 type="cite">

  <pre wrap="">On Wed, 26 Aug 2009 19:48:30 +0400, Eugene wrote:

Hey,

  </pre>

  <blockquote type="cite">

    <pre wrap="">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.

    </pre>

  </blockquote>

  <pre wrap=""><!---->

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?</pre>

</blockquote>

Putting it to the addons plugin sounds reasonable. I'll do it this or

next week.<br>

<br>

Regards,<br>

Eugene.<br>

</body>

</html>