reverted a bug I introduced when toying around with copy region in agg

jdh2358 · jdh2358 · commit d08eb5157e75 · 2008-11-20T20:22:08.000Z
svn path=/trunk/matplotlib/; revision=6424
diff --git a/examples/event_handling/path_editor.py b/examples/event_handling/path_editor.py
@@ -1,3 +1,5 @@
+import matplotlib
+matplotlib.use('TkAgg')
 import numpy as np
 import matplotlib.path as mpath
 import matplotlib.patches as mpatches
diff --git a/lib/matplotlib/collections.py b/lib/matplotlib/collections.py
@@ -701,6 +701,9 @@ def fill_between_masked(x, y, mask, yboundary=0, **kwargs):
             theseverts = []
             xslice = x[ind0:ind1]
             yslice = y[ind0:ind1]
+            if not len(xslice):
+                continue
+
             N = len(xslice)
             X = np.zeros((2*N+2, 2), np.float)
             X[0] = xslice[0], yboundary
@@ -750,6 +753,8 @@ def span_masked(x, mask, ymin, ymax, **kwargs):
         xranges = []
         for ind0, ind1 in mlab.contiguous_regions(mask):
             xslice = x[ind0:ind1]
+            if not len(xslice):
+                continue
             xranges.append((xslice[0], xslice[-1]-xslice[0]))
 
         collection = BrokenBarHCollection(xranges, [ymin, ymax-ymin], **kwargs)
diff --git a/src/_backend_agg.cpp b/src/_backend_agg.cpp
@@ -386,7 +386,7 @@ RendererAgg::copy_from_bbox(const Py::Tuple& args) {
     throw Py::TypeError("Invalid bbox provided to copy_from_bbox");
 
   //  std::cout << l << " " << b << " " << r << " " << t << " " << (height - (int)b) << " " << height - (int)t << std::endl;
-  agg::rect_i rect((int)l, height - (int)b, (int)r, height - (int)t);
+  agg::rect_i rect((int)l, height - (int)t, (int)r, height - (int)b);
 
   BufferRegion* reg = NULL;
   try {

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,5 @@`
	`1`	`+import matplotlib`
	`2`	`+matplotlib.use('TkAgg')`
`1`	`3`	`import numpy as np`
`2`	`4`	`import matplotlib.path as mpath`
`3`	`5`	`import matplotlib.patches as mpatches`