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

[Andrew Manson]

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

Ship it!


As we have been discussing on the mailing list here: http://mail.kde.org/pipermail/marble-devel/2009-July/000500.html I believe this patch needs to be applied to simplify the casting and improve the introspective capabilities of the GeoData classes. 

- Andrew


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
> 
>