[Kde-pim] [patch] small code style modification

[Tom Albers]

Op Sunday 12 July 2009 22:10 schreef u:
> On Saturday 11 July 2009, Frank Osterfeld wrote:
> > Radu,
> >
> > On Monday 06 July 2009 20:04:19 Radu Voicilas wrote:
> > > Hi,
> > >
> > > Starting from this post http://www.kdedevelopers.org/node/3955 I
> > > took a quick look over akregator's source and made small code style
> > > modifications to article.cpp as my first, hopefully accepted,
> > > patch. I am not sure if this could be considered for inclusion, but
> > > I am sending it either ways.
> >
> > although I'd prefer your version in the code, I try to avoid
> > whitespace changes just for the sake of identing style, as they break
> > the SVN history, making it hard to follow who committed/did actual
> > changes to the code. Thus I won't apply the patch.
> 
> FWIW, "svn blame -x -b foo" can be used to ignore whitespace changes. 
> This should avoid the breakage of SVN history. The only downside is 
> that this does not seem to be supported by websvn.

I've put out a request upstream for that option.

Toma
-- 
KDE Developer
-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/