KWin GLES, cleanup and coding style change

Sebastian Kügler sebas at kde.org
Mon Jan 31 13:35:02 CET 2011


On Sunday, January 30, 2011 20:15:02 Martin Gräßlin wrote:
> > I am just going through the coding style changes and it looks fine so
> > far. Except that the astyle script generated
> > if(foo) {
> > }
> > instead of
> > if (foo) {
> > }
> >
> > I used the astyle-kdelibs script in kdesdk. If anyone knows how to fix
> > it, I will do a rerun, if not I think we have to live with it and just
> > define that future commits need to have the whitespace.
> 
> There is of course the option to change it manually. I would not like to
> do  it, but if there is no automatic way I will waste tomorrow evening ;-)

Globally replacing "if(" (and "for(" and "foreach(") with "if (" (and the 
others) should be easy enough, no? Not sure what doing it manually would have 
as benefit (that said, I'm an astyle n00b).

hth.
-- 
sebas

http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9


More information about the Plasma-devel mailing list