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

[Dennis Nienhüser]

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

Ship it!


Works fine here from a quick test.

- Dennis Nienhüser


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/20130915/15600d7c/attachment.html>