Address review comments.

anntzer · anntzer · commit 245f5de92c1d · 2018-02-06T09:24:51.000+01:00
diff --git a/doc/api/backend_cairo_api.rst b/doc/api/backend_cairo_api.rst
@@ -2,7 +2,12 @@
 :mod:`matplotlib.backends.backend_cairo`
 ========================================
 
-.. automodule:: matplotlib.backends.backend_cairo
-   :members:
-   :undoc-members:
-   :show-inheritance:
+.. Building the docs requires either adding pycairo/cairocffi as docs build
+   dependency, or bumping the minimal numpy version to one that supports
+   MagicMocks (which does define `__index__`) as indices (recent numpys do, but
+   1.7.1 doesn't).
+
+.. .. automodule:: matplotlib.backends.backend_cairo
+..    :members:
+..    :undoc-members:
+..    :show-inheritance:
diff --git a/lib/matplotlib/backends/backend_cairo.py b/lib/matplotlib/backends/backend_cairo.py
@@ -77,15 +77,6 @@ def buffer_info(self):
 _CAIRO_PATH_TYPE_SIZES[cairo.PATH_CLOSE_PATH] = 1
 
 
-def _convert_path(ctx, path, transform, clip=None):
-    return _convert_paths(ctx, [path], [transform], clip)
-
-
-def _convert_paths(ctx, paths, transforms, clip=None):
-    return (_convert_paths_fast if HAS_CAIRO_CFFI else _convert_paths_slow)(
-        ctx, paths, transforms, clip)
-
-
 def _convert_paths_slow(ctx, paths, transforms, clip=None):
     for path, transform in zip(paths, transforms):
         for points, code in path.iter_segments(transform, clip=clip):
@@ -106,7 +97,7 @@ def _convert_paths_slow(ctx, paths, transforms, clip=None):
 
 def _convert_paths_fast(ctx, paths, transforms, clip=None):
     # We directly convert to the internal representation used by cairo, for
-    # which ABI compatibility is guaranteed.  The layout is for each item is
+    # which ABI compatibility is guaranteed.  The layout for each item is
     # --CODE(4)--  -LENGTH(4)-  ---------PAD(8)---------
     # ----------X(8)----------  ----------Y(8)----------
     # with the size in bytes in parentheses, and (X, Y) repeated as many times
@@ -152,6 +143,13 @@ def _convert_paths_fast(ctx, paths, transforms, clip=None):
     cairo.cairo.cairo_append_path(ctx._pointer, ptr)
 
 
+_convert_paths = _convert_paths_fast if HAS_CAIRO_CFFI else _convert_paths_slow
+
+
+def _convert_path(ctx, path, transform, clip=None):
+    return _convert_paths(ctx, [path], [transform], clip)
+
+
 class RendererCairo(RendererBase):
     fontweights = {
         100          : cairo.FONT_WEIGHT_NORMAL,
@@ -230,8 +228,7 @@ def draw_markers(self, gc, marker_path, marker_trans, path, transform,
 
         ctx.new_path()
         # Create the path for the marker; it needs to be flipped here already!
-        _convert_path(
-            ctx, marker_path, marker_trans + Affine2D().scale(1, -1))
+        _convert_path(ctx, marker_path, marker_trans + Affine2D().scale(1, -1))
         marker_path = ctx.copy_path_flat()
 
         # Figure out whether the path has a fill
@@ -295,7 +292,7 @@ def _draw_paths():
             for k, v in gc_vars.items():
                 try:
                     getattr(gc, "set" + k)(v)
-                except (AttributeError, TypeError):
+                except (AttributeError, TypeError) as e:
                     pass
             gc.ctx.new_path()
             paths, transforms = zip(*grouped_draw)
@@ -309,8 +306,8 @@ def _draw_paths():
                 offsetTrans, facecolors, edgecolors, linewidths, linestyles,
                 antialiaseds, urls, offset_position):
             path, transform = path_id
-            transform = (Affine2D(transform.get_matrix()).translate(xo, yo)
-                         + Affine2D().scale(1, -1).translate(0, self.height))
+            transform = (Affine2D(transform.get_matrix())
+                         .translate(xo, yo - self.height).scale(1, -1))
             # rgb_fc could be a ndarray, for which equality is elementwise.
             new_key = vars(gc0), tuple(rgb_fc) if rgb_fc is not None else None
             if new_key == reuse_key:
