Minor whitespace changes relating to lines of code that are being changed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux