[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-policies
Subject:    Re: KDE CVS Commit Policy (2nd version)
From:       Harri Porten <porten () trolltech ! com>
Date:       2003-04-01 14:31:18
[Download RAW message or body]

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.


_______________________________________________
Kde-policies mailing list
Kde-policies@mail.kde.org
http://mail.kde.org/mailman/listinfo/kde-policies
[prev in list] [next in list] [prev in thread] [next in thread]