[Ovmsdev] Crash debugging

Michael Balzer dexter at expeedo.de
Mon Apr 9 01:20:07 HKT 2018


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 at 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




More information about the OvmsDev mailing list