[Marble-devel] PATCH : fix for some Krazy issues : includes

[Inge Wallin]

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 :
> http://techbase.kde.org/Policies/Library_Code_Policy#Include_guards
>
> > 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 
directory.

	-Inge