I don’t need to rationalize being anal about my commits, of most I never visit again ;)
I don’t need to rationalize being anal about my commits, of most I never visit again ;)
It's just a matter of basic politeness to others.
And in fact, I would reject a patch to any of my code out there that doesn't clearly separate different things into different patches.
Too many times in my SVN days I witnessed strange bugs appearing in some commits that otherwise only contained whitespace changes labelled with the all-famous commit message "fixes".
Never again.
I wasn’t being sarcastic, I do spend lots of time making my commits (and everything else for that matter) ready for inspection by her royal queen — I know others with this same behavior, and my comment was merely meant to say “don’t rationalize it” — I do it to feel good, well, to not feel bad (about “leaving a mess”) is probably closer to the truth.
But it’s like properly formatting and indenting your source. Any one particular instance of sloppiness is irrelevant, but in the aggregate, the care you put into it really makes a difference in the feel of your works.