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

List:       koffice-devel
Subject:    Re: DataCenterMap
From:       Thomas Zander <zander () kde ! org>
Date:       2010-01-20 13:28:37
Message-ID: 201001201428.37950.zander () kde ! org
[Download RAW message or body]

On Wednesday 20. January 2010 13.58.41 Jan Hambrecht wrote:
> >> +            , hasExternalDataCenterMap(false),
> >> +            resourceManager(0)
> >>
> >> Please put the comma in front. (KarbonDocument.cpp)
> >
> > If I have to work on all of KOffice sourcecode I'm following the coding
> > style. I thought we agreed on using that everywhere. Can we avoid
> > exceptions for certain apps? That would be very useful :)
> 
> I am not sure that is handled by the coding style. So please use the one 
> that is already present in that file.

Many things are not detailed in the policy that are taken from the English 
language. Comma placement is one of them.
-- 
Thomas Zander
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic