Minor fixups from qlogic review

efiring · efiring · commit 748c759f6f89 · 2017-06-15T16:49:33.000-04:00
diff --git a/lib/matplotlib/axes/_base.py b/lib/matplotlib/axes/_base.py
@@ -966,7 +966,7 @@ def cla(self, clear_axis=True):
 
         if clear_axis:
             self.xaxis.cla(shared_x)
-            self.yaxis.cla(shared_y)            #
+            self.yaxis.cla(shared_y)
 
         for name, spine in six.iteritems(self.spines):
             spine.cla(clear_axis=False)  # Clears only the position.
diff --git a/lib/matplotlib/axis.py b/lib/matplotlib/axis.py
@@ -160,7 +160,7 @@ def __init__(self, axes, loc, label,
         self._grid_linewidth = (rcParams['grid.linewidth']
                                 if grid_linewidth is None else grid_linewidth)
         self._grid_alpha = (rcParams['grid.alpha']
-                                if grid_alpha is None else grid_alpha)
+                            if grid_alpha is None else grid_alpha)
 
         self.apply_tickdir(tickdir)
 
@@ -737,7 +737,6 @@ def cla(self, shared=None):
         'clear the current axis'
 
         self.label.set_text('')  # self.set_label_text would change isDefault_
-        self._set_artist_props(self.label)  # sets figure; needed here?
 
         if shared is None:
             self._set_scale('linear')
@@ -798,7 +797,6 @@ def set_tick_params(self, which='major', reset=False, **kw):
 
         if reset:
             self.reset_ticks()
-        # Is this "else" correct? Shouldn't kwargs be applied after a reset?
         else:
             if which == 'major' or which == 'both':
                 for tick in self.majorTicks:
diff --git a/lib/matplotlib/scale.py b/lib/matplotlib/scale.py
@@ -76,7 +76,7 @@ def set_default_locators_and_formatters(self, axis):
         axis.set_major_formatter(ScalarFormatter())
         axis.set_minor_formatter(NullFormatter())
         # update the minor locator for x and y axis based on rcParams
-        if (rcParams['xtick.minor.visible']):
+        if rcParams['xtick.minor.visible']:
             axis.set_minor_locator(AutoMinorLocator())
         else:
             axis.set_minor_locator(NullLocator())
