[Marble-devel] [Patch] Include own header first

Torsten Rahn torsten.rahn at credativ.de
Wed Apr 16 19:19:11 CEST 2008


On Wednesday 16 April 2008 18:21:10 Jens-Michael Hoffmann wrote:
> Yes, I've seen it in some of the files. Besides "own header first" is there
> a technical reason for the order? Or is there a convention in KDE?

Not sure. I just found it easier to see and find entries and "pushes" people 
to stick to the order.

Ingwa: Any input on this? :-)

> > // Own
> > #include "MapThemeManager.h"
> Instead of "local dir" I would prefer "Marble" because the header need not
> be in the local dir.

Yes, in the meantime I've seen that e.g. the plasmoid is using that. So let's 
use "Marble" indeed. Ingwa: What's your opinion on this?

BR,
Torsten

-- 
 Torsten Rahn

 Tel.: 0 21 61 - 46 43 - 192

credativ GmbH, HRB Mönchengladbach 12080
Hohenzollernstr. 133, 41061 Mönchengladbach
Geschäftsführung: Dr. Michael Meskes, Jörg Folz


More information about the Marble-devel mailing list