Comment #9 on issue 287 by [email protected]: Newline substitution behaviour
https://code.google.com/p/vim/issues/detail?id=287

Okay, that works as well.

However your patch does not compile because of the lowercase true.

Secondly, I personally do not think, changing the message about the changed lines is correct. Vim 7.4 reported "<addresscount> substitutions on 1 line", vim 7.4.232 reports "<addresscount> substitutions on <addresscount> lines" Which I personally think is correct, since this is the number of substitions performed.

But if we think about changing that regression, it should be changed as well back to the Vim 7.4.

And since we are skipping the join command for the :$s/\n// anyhow, Vim should also skip the message reporting the number of substititutions.

So here is a final patch addressing all those remarks.

Attachments:
        issue_287.diff  1013 bytes

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- You received this message because you are subscribed to the Google Groups "vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Raspunde prin e-mail lui