Docstring/comments fixes & PEP8.

afvincent · jenshnielsen · commit d3af67dfbe54 · 2016-05-30T13:35:46.000+01:00
diff --git a/lib/matplotlib/patches.py b/lib/matplotlib/patches.py
@@ -4072,7 +4072,7 @@ def set_dpi_cor(self, dpi_cor):
     def get_dpi_cor(self):
         """
         dpi_cor is currently used for linewidth-related things and
-        shrink factor. Mutation scale *is now* affected by this.
+        shrink factor. Mutation scale is affected by this.
         """
 
         return self._dpi_cor
@@ -4224,11 +4224,12 @@ def get_path_in_displaycoord(self):
                                                patchB=self.patchB,
                                                shrinkA=self.shrinkA * dpi_cor,
                                                shrinkB=self.shrinkB * dpi_cor
-                                              )
+                                               )
         else:
             _path = self.get_transform().transform_path(self._path_original)
 
-        _path, fillable = self.get_arrowstyle()(_path,
+        _path, fillable = self.get_arrowstyle()(
+                                        _path,
                                         self.get_mutation_scale() * dpi_cor,
                                         self.get_linewidth() * dpi_cor,
                                         self.get_mutation_aspect()
@@ -4542,13 +4543,14 @@ def get_path_in_displaycoord(self):
                                            patchB=self.patchB,
                                            shrinkA=self.shrinkA * dpi_cor,
                                            shrinkB=self.shrinkB * dpi_cor
-                                          )
+                                           )
 
-        _path, fillable = self.get_arrowstyle()(_path,
+        _path, fillable = self.get_arrowstyle()(
+                                        _path,
                                         self.get_mutation_scale() * dpi_cor,
                                         self.get_linewidth() * dpi_cor,
                                         self.get_mutation_aspect()
-                                       )
+                                        )
 
         return _path, fillable
 
diff --git a/lib/matplotlib/tests/test_arrow_patches.py b/lib/matplotlib/tests/test_arrow_patches.py
@@ -5,7 +5,7 @@
 
 import matplotlib.pyplot as plt
 from matplotlib.testing.decorators import image_comparison
-import matplotlib.patches as mpatches  # adopting scipy import conventions
+import matplotlib.patches as mpatches
 
 
 def draw_arrow(ax, t, r):
@@ -54,7 +54,7 @@ def test_boxarrow():
 
 def prepare_fancyarrow_dpi_cor_test():
     """
-    Convenience function that prepares and return a FancyArrowPatch. It aims
+    Convenience function that prepares and returns a FancyArrowPatch. It aims
     at being used to test that the size of the arrow head does not depend on
     the DPI value of the exported picture.
 
@@ -76,7 +76,7 @@ def prepare_fancyarrow_dpi_cor_test():
 def test_fancyarrow_dpi_cor_100dpi():
     """
     Check the export of a FancyArrowPatch @ 100 DPI. FancyArrowPatch is
-    instanciated through a dedicated function because another similar test
+    instantiated through a dedicated function because another similar test
     checks a similar export but with a different DPI value.
 
     Remark: test only a rasterized format.
diff --git a/lib/matplotlib/text.py b/lib/matplotlib/text.py
@@ -2163,14 +2163,7 @@ def _update_position_xytext(self, renderer, xy_pixel):
                         " use 'headlength' to set the head length in points.")
                 headlength = d.pop('headlength', 12)
 
-                # Old way:
-                # ms_pix = renderer.points_to_pixels(ms)
-                # to_style = self.figure.dpi / (72 * ms_pix)
-                # NB: is there a reason to use self.figure.dpi / (72 * ms_pix)
-                # instead of renderer.points_to_pixels(1.) / ms_pix? Both
-                # should be equal to '1 / ms', shouldn't they (by the way)?
-                #
-                # New way: '* to_style' <- '/ ms' (ms is now still in pts here)
+                # NB: ms is in pts
                 stylekw = dict(head_length=headlength / ms,
                                head_width=headwidth / ms,
                                tail_width=width / ms)
