Simplify and move cleaning strip op. from 'format_eng' to '__call__'

afvincent · afvincent · commit 122cfa202a52 · 2017-08-25T09:39:45.000-07:00
diff --git a/lib/matplotlib/ticker.py b/lib/matplotlib/ticker.py
@@ -1242,6 +1242,7 @@ def __init__(self, unit="", places=None, space_sep=True):
 
     def __call__(self, x, pos=None):
         s = "%s%s" % (self.format_eng(x), self.unit)
+        s = s.strip()  # Remove separator when there is neither prefix nor unit
         return self.fix_minus(s)
 
     def format_eng(self, num):
@@ -1297,10 +1298,6 @@ def format_eng(self, num):
 
         formatted = format_str % (mant, prefix)
 
-        formatted = formatted.strip()
-        if (self.unit != "") and (prefix == self.ENG_PREFIXES[0]):
-            formatted = formatted + self.sep
-
         return formatted
 
 
