[Marble-devel] Review Request 124672: gsoc: Introducing OsmPresetLibrary class ( refactoring of osm tag presets )

[Dennis Nienhüser]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124672/#review84174
-----------------------------------------------------------

Ship it!



src/lib/marble/osm/OsmPresetLibrary.cpp (line 97)
<https://git.reviewboard.kde.org/r/124672/#comment58274>

    Can you check with https://git.reviewboard.kde.org/r/124511/diff/6#index_header (will be pushed soon) that there are no conflicts between both patches?


- Dennis Nienhüser


On Aug. 19, 2015, 3:12 p.m., Marius Stanciu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124672/
> -----------------------------------------------------------
> 
> (Updated Aug. 19, 2015, 3:12 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> Moved the osm presets from the GeoDataFeature to a separate class. This makes accessing the preset styles much easier and clearer.
> Added additional tags to the list.
> 
> 
> Diffs
> -----
> 
>   src/lib/marble/geodata/data/GeoDataFeature.h ea23cd8 
>   src/lib/marble/geodata/data/GeoDataFeature.cpp 4443162 
>   src/lib/marble/geodata/data/GeoDataFeature_p.h 962f0d5 
>   src/lib/marble/osm/OsmPresetLibrary.h PRE-CREATION 
>   src/lib/marble/osm/OsmPresetLibrary.cpp PRE-CREATION 
>   src/plugins/runner/json/JsonParser.cpp 2c1de39 
>   src/plugins/runner/nominatim-search/OsmNominatimSearchRunner.cpp f21220d 
>   src/plugins/runner/osm/handlers/OsmTagTagHandler.cpp 148ea7d 
> 
> Diff: https://git.reviewboard.kde.org/r/124672/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marius Stanciu
> 
>

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