[Marble-devel] Review Request 122854: Marble: Fix comparison of GeoDataPolygon's

Boris Egorov egorov at linux.com
Tue Mar 10 04:32:27 UTC 2015


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

(Updated March 10, 2015, 4:32 a.m.)


Review request for Marble.


Changes
-------

Add test to make sure comparison of unequal polygons works.


Repository: marble


Description (updated)
-------

We should compare GeoDataLinearRing's of different GeoDataPolygon's, not
one with itself. Check that in TestEquality::polygonTest().

Issue detected by Cppcheck: "Same expression on both sides of '!='."


Diffs (updated)
-----

  src/lib/marble/geodata/data/GeoDataPolygon.cpp 9ae7091 
  tests/TestEquality.cpp a3f5498 

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


Testing
-------


Thanks,

Boris Egorov

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


More information about the Marble-devel mailing list