Re: Re: Alignment Nazi?
"Andrew Semprebon" <[email protected]>
| Newsgroups | gmane.comp.programming.language-of-the-year |
|---|---|
| Message-ID | <[email protected]> |
On Mon, Apr 28, 2008 at 3:21 AM, Jeroen Wenting <[email protected]> wrote: > > nope. Generally only functional changes should make it into version > control. I guess adding/correcting comment is out then... > OTOH it makes it a lot more work changing that code as every time > someone comes up with variable name that including its type name is > longer than anything that's already there an ever expanding block of > code needs to be rewritten to please your fetish. > Makes tracking diffs a lot harder when half the sourcefile changes > every time someone touches it just to make it match his personal > layout preferences. On the third hand, In most places I have worked, once the code if formatted into something decent, developers won't change it just to fit their preference (although their IDE may do that for them), so I think its worth while to get poorly (or worse, misleadingly indented) code into shape. Why should everyone else have to read poorly indented code forever more just because the original developer did a poor job in the first place? Of course, the real solution is to fire that guy who cares so little about his work products that he checks in the poorly laid-out code. -- -- Andrew Semprebon * http://semprebon.blogspot.com/