----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124231/ -----------------------------------------------------------
(Updated July 19, 2015, 12:14 p.m.) Status ------ This change has been marked as submitted. Review request for Marble. Changes ------- Submitted with commit 987b3b089541b327ec4bebc90b95b42805a12aa0 by Marius Stanciu to branch master. Repository: marble Description ------- See comment on the header on what the class is for: A new folder lib/marble/osm was created because more sources will come in the following patches. Those sources are needed by both the Osm and Annotate plugins, but also the lib itself. I have tried several approaches for managing osm data, and this seemed the most feasible. I have tried making it polymorphic, ( OsmWayData: OsmPlacemarkData and OsmRelationData: OsmPlacemarkData ) but that approach really made things more complicated and seemed a bit like overdesigning for a simple data container class this is. The empty QHashes barely use any memory, so it should be great. Diffs ----- src/lib/marble/CMakeLists.txt 94ff3da src/lib/marble/geodata/data/GeoDataCoordinates.h 7b291cc src/lib/marble/osm/CMakeLists.txt PRE-CREATION src/lib/marble/osm/OsmPlacemarkData.h PRE-CREATION src/lib/marble/osm/OsmPlacemarkData.cpp PRE-CREATION Diff: https://git.reviewboard.kde.org/r/124231/diff/ Testing ------- Thanks, Marius Stanciu
_______________________________________________ Marble-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/marble-devel
