update to latest 2geom (rev1497)

[inkscape.git] / src / 2geom / rect.h

diff --git a/src/2geom/rect.h b/src/2geom/rect.h
index c89946606fe86886924c9f54ae778882430dec1e..a136e91a8f01e8883a7fadd24c08cc65a320dee9 100644 (file)
--- a/src/2geom/rect.h
+++ b/src/2geom/rect.h
@@ -41,7 +41,7 @@
 #ifndef _2GEOM_RECT
 #define _2GEOM_RECT
 
-#include "matrix.h"
+#include <2geom/matrix.h>
 #include <boost/optional/optional.hpp>
 
 namespace Geom {
@@ -55,6 +55,10 @@ class D2<Interval> {
   private:
     Interval f[2];
   public:
+    /* The default constructor creates a rect that contains the point (0,0). Isn't it better to have it be empty?
+     * i.e.  f[X] = f[Y] = Interval(); .  Don't use Interval(+COORD_HUGE, -COORD_HUGE) !! because the arguments
+     * will be interchanged to Interval(-COORD_HUGE, +COORD_HUGE)!!
+     */
     D2<Interval>() { f[X] = f[Y] = Interval(0, 0); }
     
     D2<Interval>(Interval const &a, Interval const &b) {