Hi Joost and Thomas,

On 20.09.2014 12:02, Thomas Koenig wrote:
A somewhat trivial patch to cleanup whitespace issues in comments: sed "s/\. \*\//\. 
 \*\//g"
Tested with a recompile only. Ok for trunk ?
OK.  (Obvious, really).

I want to point a downside of massive code changes: It makes it more difficult to find out with "git/svn blame" when or why a line of code was changed.

That's not so much a problem of comment changes like this one – and I am fine with that change. :-)

However, I wanted to point out that seemingly trivial and obviously correct patches can have a downside. (One can still do such changes, but at least one should have weighted them against the downside.)

Tobias

Reply via email to