[Marble-devel] Review Request 123322: Improved placemark sorting to make layouting stable with placemarks on same zoomLevel

[Adam Dabrowski]

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

Review request for Marble.


Repository: marble


Description
-------

Sorting takes place after joining placemarks from all relevant tiles. The order is:
1) zoomLevel (placemarks were already sorted by zoomLevel before this patch)
2) popularity
3) pointer value
This prevent cases like label flickering for same zoomLevel placemarks that are close to each other and moving.


Diffs
-----

  src/lib/marble/PlacemarkLayout.cpp 6c0af30 

Diff: https://git.reviewboard.kde.org/r/123322/diff/


Testing
-------

Did testing and debugging to confirm that the order is exactly as intended. No unit tests are provided.


Thanks,

Adam Dabrowski

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