timothycrosley / deprecated.pies

The simplest (and tastiest) way to write one program that runs on both Python 2 and Python 3.
MIT License
148 stars 7 forks source link

Typo naming variables in pies/ast.py #12

Closed Shura1oplot closed 10 years ago

Shura1oplot commented 10 years ago

I think, it is just a typo in naming variables. It sometimes crashes while using frosted with undefined global name exception.

diff -ru pies.orig/pies/ast.py pies/pies/ast.py
--- pies.orig/pies/ast.py...2013-12-30 13:43:14.000000000 +0400
+++ pies/pies/ast.py....2013-12-30 14:05:02.961252572 +0400
@@ -21,8 +21,8 @@
     def argument_names(node):
         return [arg.arg for arg in node.args.args]

-    def kw_only_argument_names(self):
+    def kw_only_argument_names(node):
         return [arg.arg for arg in node.args.kwonlyargs]

-    def kw_only_default_count(self):
+    def kw_only_default_count(node):
         return sum(1 for n in node.args.kw_defaults if n is not None)
timothycrosley commented 10 years ago

Thanks for reporting! Will push a new release soon that fixes this issues

timothycrosley commented 10 years ago

Fixed in release 2.5.2, thanks again!