[Marble-devel] Review Request 109379: ElevationProfile: support for gpx-tracks

[Roman Karlstetter]

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

(Updated April 16, 2013, 5:19 p.m.)


Review request for Marble and Bernhard Beschow.


Changes
-------

add marble to reviewers groups


Description
-------

Changes to the elevation profile plugin, it's now possible to display the height profile for a GPX-track opened from file.

I made a new classes for
a) providing the data, one for routes and one for tracks. They are in the same file, should this be split into multiple files?
b) the contextmenu, because it has some members which would only clutter ElevationProfileFloatItem. It is a friend class, is this ok?

Comments please :) 


Diffs
-----

  src/plugins/render/elevationprofilefloatitem/ElevationProfileContextMenu.h PRE-CREATION 
  src/plugins/render/elevationprofilefloatitem/CMakeLists.txt 872e5e1 
  src/plugins/render/elevationprofilefloatitem/ElevationProfileContextMenu.cpp PRE-CREATION 
  src/plugins/render/elevationprofilefloatitem/ElevationProfileDataSource.h PRE-CREATION 
  src/plugins/render/elevationprofilefloatitem/ElevationProfileDataSource.cpp PRE-CREATION 
  src/plugins/render/elevationprofilefloatitem/ElevationProfileFloatItem.h cb3bff1 
  src/plugins/render/elevationprofilefloatitem/ElevationProfileFloatItem.cpp 7cb3428 

Diff: http://git.reviewboard.kde.org/r/109379/diff/


Testing
-------


Thanks,

Roman Karlstetter

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