Hi all, 

I see commits all the time where someone changes a line of code and changes the 
whitespacing in that same line of code. 

So for example, in https://gerrit.libreoffice.org/c/core/+/178504 I have 
removed the using namespaces and added css:: as the prefix, but at the same 
time *only* on the lines of code that I touched I fixed up the inconsistent 
whitespacing. 

I see this all the time in commits. 

Given the code compiles and it is very obviously what change is being made, 
what is the problem with doing this?

I know you all took away my commit bit because of my whitespace changes close 
to a decade ago, but given I see this sort of thing being done by others all 
the time, I’m trying to understand where I’m being inconsistent with current 
practice. 

Chris

Reply via email to