[Marble-commits] KDE/kdeedu/marble

Jens-Michael Hoffmann jensmh at gmx.de
Mon Aug 10 22:03:33 CEST 2009


Am Monday 10 August 2009 19:32:48 schrieb Bastian Holst:
> SVN commit 1009713 by bholst:
> 
> Marble's GeoPainterPrivate got its own file. Test for GeoPainter things. These are commented out
> due to linker problems.
> 
> 
>  M  +119 -134  src/lib/GeoPainter.cpp  
>  AM            src/lib/GeoPainter_p.h   [License: LGPL]
>  M  +1 -0      tests/CMakeLists.txt  
>  AM            tests/TestGeoPainter.cpp   [License: LGPL]
> 
> 
> --- trunk/KDE/kdeedu/marble/src/lib/GeoPainter.cpp #1009712:1009713
> @@ -9,6 +9,7 @@
>  
>  
>  #include "GeoPainter.h"
> +#include "GeoPainter_p.h"
>  
>  #include <QtCore/QDebug>
>  #include <QtCore/QList>
> @@ -28,170 +29,156 @@
>  
>  // #define MARBLE_DEBUG
>  
> +using namespace Marble;
>  
> -namespace Marble

what was the problem with it?

Imho it is preferred to use

namespace Marble
{
}


instead of just "using namespace".


Best Regards

Jens-Michael





More information about the Marble-commits mailing list