[Marble-devel] Review Request 111810: GeoJSON parser adapted to match the standard.

[Commit Hook]

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


This review has been submitted with commit d778fa679221cbbbcb6df8f27e11a12d4da068ff by Ander Pijoan ander.pijoan at deusto.es to branch master.

- Commit Hook


On Aug. 1, 2013, 4:53 p.m., Ander Pijoan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111810/
> -----------------------------------------------------------
> 
> (Updated Aug. 1, 2013, 4:53 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Description
> -------
> 
> -Changed Marble's GeoJSON parser (which was a test for just matching Kothic's GeoJSON like format) to match the exact GeoJSON standard : 
> 
> http://www.geojson.org/geojson-spec.html#id3
> 
> -Supported geometry types :
> 
> Point
> MultiPoint
> LineString
> MultiLineString
> Polygon
> MultiPolygon
> 
> -Not supported geometry types:
> 
> GeometryCollection
> 
> -Geometry properties are compared with GeoDataFeature's OsmVisualCategory to see if any of the properties matches a style to give to the geometry. If not it will be shown as a black line.
> 
> 
> Diffs
> -----
> 
>   src/plugins/runner/json/JsonParser.h 4fb072c 
>   src/plugins/runner/json/JsonParser.cpp 86cf66b 
>   src/plugins/runner/json/JsonPlugin.cpp e7ff770 
>   src/plugins/runner/json/JsonRunner.cpp 2098e1f 
> 
> Diff: http://git.reviewboard.kde.org/r/111810/diff/
> 
> 
> Testing
> -------
> 
> Some test files :
> 
> http://tile.openstreetmap.us/vectiles-skeletron/12/1206/1539.json (this file will matches road styles)
> 
> http://tile.openstreetmap.us/vectiles-buildings/12/1206/1539.json (this file matches some other geometry types)
> 
> http://tile.openstreetmap.us/vectiles-water-areas/12/1206/1539.json (this file matches some other multi geometry types)
> 
> 
> Thanks,
> 
> Ander Pijoan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20130919/27ae2a70/attachment.html>