optimize: don't autocurry Values(...) calls

Technologicat · Technologicat · commit dd5dd188cfa0 · 2021-05-27T00:51:11.000+03:00
diff --git a/unpythonic/syntax/autocurry.py b/unpythonic/syntax/autocurry.py
@@ -85,7 +85,10 @@ def transform(self, tree):
                 return tree
 
             hascurry = self.state.hascurry
-            if type(tree) is Call:
+            # Curry all calls; except as a small optimization, skip `Values(...)`,
+            # which accepts any args and kwargs, so currying it does not make sense.
+            # (It represents multiple-return-values in `unpythonic`.)
+            if type(tree) is Call and not isx(tree.func, "Values"):
                 if has_curry(tree):  # detect decorated lambda with manual curry
                     # the lambda inside the curry(...) is the next Lambda node we will descend into.
                     hascurry = True
