Re: KDE CVS Commit Policy (2nd version)
Harri Porten <[email protected]> Tue, 1 Apr 2003 16:31:18 +0200 (CEST)
| Newsgroups | gmane.comp.kde.policies |
|---|---|
| Message-ID | <[email protected]> |
On Tue, 1 Apr 2003, Pedro Morais wrote: > > > - avoid 1000 lines change consisting of 10 lines changing code and 990 > > > lines changing indentation/whitespace (as this screws up cvs annotate, > > > kwin/workspace.cpp is a "nice" example of this). > > > > If people would obey the style already in place, cleanups wouldn't be > > needed. > > But maybe the restyling could be done in a separate commit, that > is, you first commit the 10 lines you really changed and after > that you reindent the code. I agree with what the others say regarding the non-intrusive changes and leave the idea of "restyling" out completely. Any point coming close to the topic of coding style will open a can of worms and is worth an independant policy - if any at all. Harri.