I use Atom, which ships by default with the ‘whitespace’ package. That removes trailing whitespaces. Did a quick check in our project, and we have it in 10 files in main/ - mostly whitespace on blank lines (which my editor is set to NOT remove - but I think it should). In our own project, I think we should in general remove this. But, as Steven says, better to do it as a separate commit. Mea culpa. Regards, Mark.
On 9 Apr 2018, at 1:20 AM, Michael Balzer <dexter@expeedo.de> wrote:
Stephen,
I don't care about trailing whitespace, but I know what you mean. I usually set my diff options to "-b" and "core.whitespace" to "-trailing-space" so git also doesn't care.
My editor is set to "do not remove trailing whitespace", but you're right, somehow the trailing ws in Kconfig and xtensa_api.h have been removed. Strange, I can't explain that… maybe a git feature I don't know about?
Regards, Michael
Am 08.04.2018 um 18:35 schrieb Stephen Casner:
Michael,
I just took a quick look at the diff. There are many lines changed because trailing whitespace was removed, presumably as an automatic action by your editor. Is that something you can configure it not to do?
I noticed the same phenomenon on some of Mark's recent commits as well, when the files being committed were ones that he had created/edited before, so perhaps you and he are using a new editor or its policy has changed.
I don't like trailing whitespace, and I am careful to avoid it as I write code. However, I also take extra effort to avoid removing any existing trailing whitespace (or make other unrelated changes like comment fixes) as part of committing a change, especially for source that isn't mine. I sometimes make separate commits with just those nonfunctional changes. This is to keep the annotation log meaningful.
I don't mean this as a major criticism, just a question.
-- Steve _______________________________________________ OvmsDev mailing list OvmsDev@lists.openvehicles.com http://lists.openvehicles.com/mailman/listinfo/ovmsdev
-- Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal Fon 02333 / 833 5735 * Handy 0176 / 206 989 26
_______________________________________________ OvmsDev mailing list OvmsDev@lists.openvehicles.com http://lists.openvehicles.com/mailman/listinfo/ovmsdev