Alexandre Julliard wrote:
"Dimitrie O. Paun" dpaun@rogers.com writes:
On September 11, 2002 12:27 pm, Alexandre Julliard wrote:
Yes, now that whitespace has been cleaned up I've configured my Emacs to try to keep it that way.
That's not really obvious, and can screw people up... Not that people read docs, but maybe we should mention it in the "How to do a patch" bit? If you do a large patch, submit, and continue working, you're hosed!
Well, I'm happy to turn it off if people consider it to be a problem. I just don't want to have people constantly submit patches to remove whitespace that others added with their patches; so we either decide that we all try to avoid trailing spaces, or we decide that nobody should care about it.
Two solutions that come to mind: A. Commit the original patch immediatly followed by a commit to remove the white spaces (will not cause conflicts unless the same lines are changed after the commit). B. Add the "strip whitespaces at end" to the diff tool, and instruct people to use it.
Shachar