Python 3 bugs ...

rocky · rocky · commit a81ffe89636a · 2016-11-13T10:08:41.000-05:00
- Was using "while 1 .. else" improperly
- docstring indent bug: was indenting docstring improperly
diff --git a/uncompyle6/parsers/parse3.py b/uncompyle6/parsers/parse3.py
@@ -338,10 +338,8 @@ def p_loop_stmt3(self, args):
         whileelsestmt     ::= SETUP_LOOP testexpr l_stmts_opt JUMP_BACK POP_BLOCK
                               else_suite COME_FROM_LOOP
 
-        # FIXME: This gets confused with if/else in a loop. But while/else in Python
-        # is probably pretty rare.
         while1elsestmt    ::= SETUP_LOOP          l_stmts     JUMP_BACK
-                              else_suite COME_FROM_LOOP
+                              else_suite
 
         whileelselaststmt ::= SETUP_LOOP testexpr l_stmts_opt JUMP_BACK POP_BLOCK
                               else_suitec COME_FROM_LOOP
diff --git a/uncompyle6/parsers/parse30.py b/uncompyle6/parsers/parse30.py
@@ -26,8 +26,6 @@ def p_30(self, args):
         # Used to keep index order the same in semantic actions
         jb_pop_top     ::= JUMP_BACK POP_TOP
 
-        # FIXME: Add COME_FROM designators
-        # This gets confused with while1elsestmt. But this is probably more common
         while1stmt ::= SETUP_LOOP l_stmts COME_FROM_LOOP
 
         else_suitel ::= l_stmts COME_FROM_LOOP JUMP_BACK
diff --git a/uncompyle6/semantics/make_function.py b/uncompyle6/semantics/make_function.py
@@ -542,7 +542,7 @@ def build_param(ast, name, default):
 
     if len(code.co_consts) > 0 and code.co_consts[0] is not None and not isLambda: # ugly
         # docstring exists, dump it
-        self.print_docstring(indent, code.co_consts[0])
+        self.print_docstring(self.indent, code.co_consts[0])
 
     code._tokens = None # save memory
     assert ast == 'stmts'
