Address review comments.

anntzer · anntzer · commit a103612323fd · 2018-02-03T13:46:10.000Z
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
@@ -82,8 +82,10 @@ def _convert_path(ctx, path, transform, clip=None):
 
 
 def _convert_paths(ctx, paths, transforms, clip=None):
-    return (_convert_paths_fast if HAS_CAIRO_CFFI else _convert_paths_slow)(
-        ctx, paths, transforms, clip)
+    if HAS_CAIRO_CFFI:
+        return _convert_paths_fast(ctx, paths, transforms, clip)
+    else:
+        return _convert_paths_slow(ctx, paths, transforms, clip)
 
 
 def _convert_paths_slow(ctx, paths, transforms, clip=None):
@@ -106,7 +108,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
@@ -230,8 +232,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 +296,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 +310,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:
