matplotlib
diff --git a/‎lib/matplotlib/tests/baseline_images/test_simplification/clipper_edge.pdf
1.23 KB b/‎lib/matplotlib/tests/baseline_images/test_simplification/clipper_edge.pdf
1.23 KB
diff --git a/‎lib/matplotlib/tests/baseline_images/test_simplification/clipper_edge.png
2.39 KB b/‎lib/matplotlib/tests/baseline_images/test_simplification/clipper_edge.png
2.39 KB
diff --git a/‎lib/matplotlib/tests/baseline_images/test_simplification/clipper_edge.svg
Lines changed: 104 additions & 0 deletions b/‎lib/matplotlib/tests/baseline_images/test_simplification/clipper_edge.svg
Lines changed: 104 additions & 0 deletions
diff --git a/‎lib/matplotlib/tests/test_simplification.py
Lines changed: 17 additions & 0 deletions b/‎lib/matplotlib/tests/test_simplification.py
Lines changed: 17 additions & 0 deletions
diff --git a/‎src/path_converters.h
Lines changed: 7 additions & 4 deletions b/‎src/path_converters.h
Lines changed: 7 additions & 4 deletions
@@ -186,6 +186,23 @@ def test_throw_rendering_complexity_exceeded():
     else:
         rcParams['path.simplify'] = True
 
+@image_comparison(baseline_images=['clipper_edge'])
+def test_clipper():
+    dat = (0, 1, 0, 2, 0, 3, 0, 4, 0, 5)
+    fig = plt.figure(figsize=(2, 1))
+    fig.subplots_adjust(left = 0, bottom = 0, wspace = 0, hspace = 0)
+
+    ax = fig.add_axes((0, 0, 1.0, 1.0), ylim = (0, 5), autoscale_on = False)
+    ax.plot(dat)
+    ax.xaxis.set_major_locator(plt.MultipleLocator(1))
+    ax.xaxis.set_major_formatter(plt.NullFormatter())
+    ax.yaxis.set_major_locator(plt.MultipleLocator(1))
+    ax.yaxis.set_major_formatter(plt.NullFormatter())
+    ax.xaxis.set_ticks_position('bottom')
+    ax.yaxis.set_ticks_position('left')
+
+    ax.set_xlim(5, 9)
+    fig.savefig('clipper_edge')
 
 if __name__=='__main__':
     import nose
 
@@ -288,18 +288,21 @@ class PathClipper
     PathClipper(VertexSource& source, bool do_clipping,
                 double width, double height) :
         m_source(&source), m_do_clipping(do_clipping),
-        m_cliprect(0.0, 0.0, width, height), m_moveto(true),
+        m_cliprect(-1.0, -1.0, width + 1.0, height + 1.0), m_moveto(true),
         m_has_next(false)
     {
         // empty
     }
 
     PathClipper(VertexSource& source, bool do_clipping,
                 const agg::rect_base<double>& rect) :
-            m_source(&source), m_do_clipping(do_clipping),
-            m_cliprect(rect), m_moveto(true), m_has_next(false)
+        m_source(&source), m_do_clipping(do_clipping),
+        m_cliprect(rect), m_moveto(true), m_has_next(false)
     {
-        // empty
+        m_cliprect.x1 -= 1.0;
+        m_cliprect.y1 -= 1.0;
+        m_cliprect.x2 += 1.0;
+        m_cliprect.y2 += 1.0;
     }
 
     inline void
Original file line number	Diff line number	Diff line change
`@@ -288,18 +288,21 @@ class PathClipper`
`288`	`288`	`PathClipper(VertexSource& source, bool do_clipping,`
`289`	`289`	`double width, double height) :`
`290`	`290`	`m_source(&source), m_do_clipping(do_clipping),`
`291`		`- m_cliprect(0.0, 0.0, width, height), m_moveto(true),`
	`291`	`+ m_cliprect(-1.0, -1.0, width + 1.0, height + 1.0), m_moveto(true),`
`292`	`292`	`m_has_next(false)`
`293`	`293`	`{`
`294`	`294`	`// empty`
`295`	`295`	`}`
`296`	`296`
`297`	`297`	`PathClipper(VertexSource& source, bool do_clipping,`
`298`	`298`	`const agg::rect_base<double>& rect) :`
`299`		`- m_source(&source), m_do_clipping(do_clipping),`
`300`		`- m_cliprect(rect), m_moveto(true), m_has_next(false)`
	`299`	`+ m_source(&source), m_do_clipping(do_clipping),`
	`300`	`+ m_cliprect(rect), m_moveto(true), m_has_next(false)`
`301`	`301`	`{`
`302`		`- // empty`
	`302`	`+ m_cliprect.x1 -= 1.0;`
	`303`	`+ m_cliprect.y1 -= 1.0;`
	`304`	`+ m_cliprect.x2 += 1.0;`
	`305`	`+ m_cliprect.y2 += 1.0;`
`303`	`306`	`}`
`304`	`307`
`305`	`308`	`inline void`