----------------------------------------------------------- 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 > >
_______________________________________________ Marble-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/marble-devel
