Agreed, it doesn't harm readability. However some editors (like Vim, which I use) automatically strip trailing whitespace, so you still get commits with diffs for lines that have nothing to do with the actual code changes in those commits. If you prefer not checking for it that's fine with me, just want to point out that there is a good reason for the check.