[Marble-devel] PATCH : fix for some Krazy issues : includes
inge at lysator.liu.se
Thu Apr 2 12:40:20 CEST 2009
On Thursday 02 April 2009 01:38:40 Bruno Bigras wrote:
> Torsten Rahn a écrit :
> > And the attempt to standardize the include guards is also very much
> > appreciated :-)
> It wasn't intentional :), they were spotted by Krazy as "missing or
> improper include guard in header".
> > I just wonder whether having a double-underscore to "indicate"
> > the Camel Case split __ shouldn't rather be a single-underscore ...
> > (easier to spot).
> I committed the patch (r947997) with singles underscore instead, it's
> more like the sample at :
> > Maybe we should take the coding style also into the Wiki:
> > http://techbase.kde.org/Projects/Marble
> Is there a KDE's coding style policy? I only saw one for Kdelibs and
> CMake on TechBase.
At least there is one for Marble. Check the file CODING in Marble's top
More information about the Marble-devel