Use 'y#' instead of 's#' in write_rgba of the agg backend. Fixes matplotlib#1891.

bytbox · bytbox · commit 7dc06fbc6c66 · 2014-03-14T11:43:34.000-04:00
diff --git a/src/_backend_agg.cpp b/src/_backend_agg.cpp
@@ -2177,7 +2177,11 @@ RendererAgg::write_rgba(const Py::Tuple& args)
                 "Object does not appear to be a 8-bit string path or a Python file-like object");
         }
 
+        #if PY3K
+        PyObject_CallFunction(write_method, (char *)"y#", pixBuffer, NUMBYTES);
+        #else
         PyObject_CallFunction(write_method, (char *)"s#", pixBuffer, NUMBYTES);
+        #endif
 
         Py_XDECREF(write_method);
     }

Original file line number	Diff line number	Diff line change
`@@ -2177,7 +2177,11 @@ RendererAgg::write_rgba(const Py::Tuple& args)`
`2177`	`2177`	`"Object does not appear to be a 8-bit string path or a Python file-like object");`
`2178`	`2178`	`}`
`2179`	`2179`
	`2180`	`+ #if PY3K`
	`2181`	`+ PyObject_CallFunction(write_method, (char *)"y#", pixBuffer, NUMBYTES);`
	`2182`	`+ #else`
`2180`	`2183`	`PyObject_CallFunction(write_method, (char *)"s#", pixBuffer, NUMBYTES);`
	`2184`	`+ #endif`
`2181`	`2185`
`2182`	`2186`	`Py_XDECREF(write_method);`
`2183`	`2187`	`}`