<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, Feb 14, 2025 at 12:18 AM Stephan Bergmann <<a href="mailto:stephan.bergmann@allotropia.de">stephan.bergmann@allotropia.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 13.02.25 11:36, Chris Sherlock wrote:<br>
> I see commits all the time where someone changes a line of code and changes the whitespacing in that same line of code.<br>
<br>
I occasionally see such too, and always wonder why people do that.  My <br>
objections are twofold:<br>
<br>
For one, unnecessary and unrelated changes are distractions when looking <br>
at a commit.  <br></blockquote><div>... </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
And for another, what is the benefit of such random whitespace changes? <br>
The code is readable, before and after. </blockquote><div><br></div><div>I'm not following the argument. If whitespacing changes don't make the code less readable or more readable, then why would a small whitespacing change that makes the code more consistent with the surrounding code style on the lines of code you are modifying make a difference? </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">  (To paraphrase <br>
an example from the original PragProg book:  Mechanical workers also <br>
don't spend time making their power drills look beautiful.  It's tools <br>
for them that need to get their job done in a professional way, but it's <br>
irrelevant that they may look patched and ugly.)<br></blockquote><div><br></div><div>There are plenty of craftsmen who want to look after their tools. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
That's my take on it.  Of course, YMMV<br></blockquote><div><br></div><div>Indeed. </div></div></div>