[Marble-devel] Review Request: provide nodeType information for downcasting

[Torsten Rahn]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1119/#review1748
-----------------------------------------------------------


Two thoughts:

Right now we have two ways of namespacing: Qt uses CamelCasing namespaces (QTest etc.).

In Marble it's currently a bit inconsistent: "Marble" vs. "dgml" "osm".

I think we should gradually adjust this for Marble 0.9 to be consistent with Qt's way.
So my suggestion would be to change this accordingly.

- Torsten


On 2009-07-24 20:35:16, tgridel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1119/
> -----------------------------------------------------------
> 
> (Updated 2009-07-24 20:35:16)
> 
> 
> Review request for marble.
> 
> 
> Summary
> -------
> 
> any class which inherits from GeoNode can specify a nodeType to provide downcasting
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataColorStyle.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataContainer.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataDocument.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataFeature.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataFolder.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataGeometry.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataHotSpot.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataIconStyle.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataLabelStyle.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataLatLonAltBox.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataLatLonBox.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataLineString.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataLineStyle.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataLinearRing.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataMultiGeometry.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataObject.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataPlacemark.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataPoint.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataPolyStyle.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataPolygon.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataStyle.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataStyleMap.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataStyleSelector.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/parser/GeoDocument.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/parser/GeoDocument.cpp 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneDocument.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneFilter.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneGroup.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneHead.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneIcon.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneItem.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneLayer.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneLegend.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneMap.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoScenePalette.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneProperty.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneSection.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneSettings.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneTexture.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneVector.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneXmlDataSource.h 1001551 
>   /trunk/KDE/kdeedu/marble/src/lib/geodata/scene/GeoSceneZoom.h 1001551 
> 
> Diff: http://reviewboard.kde.org/r/1119/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> tgridel
> 
>