[Marble-devel] Review Request 109426: use GeoDataLatLonBox as boundary in GeoPolygon

[Bernhard Beschow]

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

Review request for Marble.


Description
-------

GeoPolygon is deprecated. In order to move away from this class, this patch attempts to solve its bounding box concerns the GeoData{Polygon, LineString, LinearRing} way (which make use of GeoDataLatLon(Alt)Box).

I wonder - in particular - if the spherical case works like before. I welcome any clues here. Thanks for your reviews!


Diffs
-----

  src/lib/GeoPolygon.h 37fb401d7b50a323ff9a3408d16065f7e56cac9c 
  src/lib/GeoPolygon.cpp efd8e9bd4bdf963535e562818d84e32def680009 
  src/lib/VectorMap.cpp cf3dcef082235374d0bdb9ea163a91a3055e4c1e 

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


Testing
-------

Tested with "Atlas" theme in spherical and Mercator projection: The IDL seems to look the same as before and the continents seem to get colored correctly when zoomed in and out. Please verify since I'm not that familiar with the code.


Thanks,

Bernhard Beschow

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