[Liblas-commits] hg: 2 new changesets

[liblas-commits at liblas.org]

changeset eb40e08ffeda in /Volumes/Data/www/liblas.org/hg
details: http://hg.liblas.orghg?cmd=changeset;node=eb40e08ffeda
summary: Removed redundant member operators from Bounds class.

changeset 9dcdde9c7334 in /Volumes/Data/www/liblas.org/hg
details: http://hg.liblas.orghg?cmd=changeset;node=9dcdde9c7334
summary: Removed redundant member operators from Bounds class.

diffstat:

 include/liblas/lasbounds.hpp |  10 ----------
 src/lasbounds.cpp            |   1 +
 2 files changed, 1 insertions(+), 10 deletions(-)

diffs (31 lines):

diff -r 37d20d5726aa -r 9dcdde9c7334 include/liblas/lasbounds.hpp
--- a/include/liblas/lasbounds.hpp	Thu Jul 29 15:17:46 2010 +0100
+++ b/include/liblas/lasbounds.hpp	Thu Jul 29 16:45:57 2010 +0100
@@ -85,16 +85,6 @@
     bool equal(Bounds const& other) const;
     bool intersects2d(Bounds const& other) const;
     bool intersects3d(Bounds const& other) const;
-
-    bool operator==(Bounds const& rhs)
-    {
-        return equal(rhs);
-    }
-
-    bool operator!=(Bounds const& rhs)
-    {
-        return (!equal(rhs));
-    }
         
 private:
     Array mins;
diff -r 37d20d5726aa -r 9dcdde9c7334 src/lasbounds.cpp
--- a/src/lasbounds.cpp	Thu Jul 29 15:17:46 2010 +0100
+++ b/src/lasbounds.cpp	Thu Jul 29 16:45:57 2010 +0100
@@ -149,6 +149,7 @@
 
 bool Bounds::equal(Bounds const& other) const
 {
+    // FIXME: direct comparison of float-point values may give wrong result --mloskot
     for (uint32_t i = 0; i < 3; i++) {
         if (!(min(i) == other.min(i)) && !(max(i) == other.max(i))) 
         {