Fix bug where a line with NULL data limits prevents subsequent data limits from calculating correctly

mdboom · mdboom · commit 2c28ad2d5fc5 · 2008-12-18T13:41:35.000Z
svn path=/branches/v0_98_5_maint/; revision=6661
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,8 +1,14 @@
+2008-12-18 Fix bug where a line with NULL data limits prevents
+           subsequent data limits from calculating correctly - MGD
+
+2008-12-17 Major documentation generator changes - MGD
+
 2008-12-17 Applied macosx backend patch with support for path
            collections, quadmesh, etc... - JDH
 
 2008-12-17 fix dpi-dependent behavior of text bbox and arrow in annotate
             -JJL
+
 2008-12-16 Another attempt to fix dpi-dependent behavior of Legend. -JJL
 
 ======================================================================
diff --git a/src/_path.cpp b/src/_path.cpp
@@ -403,10 +403,25 @@ Py::Object _path_module::update_path_extents(const Py::Tuple& args)
         }
         else
         {
-            extents_data[0] = std::min(x0, x1);
-            extents_data[1] = std::min(y0, y1);
-            extents_data[2] = std::max(x0, x1);
-            extents_data[3] = std::max(y0, y1);
+            if (x0 > x1)
+            {
+                extents_data[0] = std::numeric_limits<double>::infinity();
+                extents_data[2] = -std::numeric_limits<double>::infinity();
+            }
+            else
+            {
+                extents_data[0] = x0;
+                extents_data[2] = x1;
+            }
+            if (y0 > y1) {
+                extents_data[1] = std::numeric_limits<double>::infinity();
+                extents_data[3] = -std::numeric_limits<double>::infinity();
+            }
+            else
+            {
+                extents_data[1] = y0;
+                extents_data[3] = y1;
+            }
             minpos_data[0] = xm;
             minpos_data[1] = ym;
         }
