Further simplify _preprocess_data.

anntzer · anntzer · commit daceb69b09d2 · 2020-03-01T11:57:34.000+01:00
We don't need to evaluate needs_label at the top, we can just do it when
needed.  Also rename all_kwargs to args_and_kwargs, as it's really a
mapping of all arg names to their values, whether they have been passed
positionally or by keyword.
diff --git a/lib/matplotlib/__init__.py b/lib/matplotlib/__init__.py
@@ -1382,9 +1382,6 @@ def inner(ax, *args, data=None, **kwargs):
             return func(ax, *map(sanitize_sequence, args), **kwargs)
 
         bound = new_sig.bind(ax, *args, **kwargs)
-        needs_label = (label_namer
-                       and "label" not in bound.arguments
-                       and "label" not in bound.kwargs)
         auto_label = (bound.arguments.get(label_namer)
                       or bound.kwargs.get(label_namer))
 
@@ -1403,10 +1400,10 @@ def inner(ax, *args, data=None, **kwargs):
         new_args = bound.args
         new_kwargs = bound.kwargs
 
-        if needs_label:
-            all_kwargs = {**bound.arguments, **bound.kwargs}
+        args_and_kwargs = {**bound.arguments, **bound.kwargs}
+        if label_namer and "label" not in args_and_kwargs:
             new_kwargs["label"] = _label_from_arg(
-                all_kwargs.get(label_namer), auto_label)
+                args_and_kwargs.get(label_namer), auto_label)
 
         return func(*new_args, **new_kwargs)
 
