[Jython-checkins] SF.net SVN: jython:[5236] branches/nowalker/src/org/python/antlr/ GrammarActions.java

[<fwi...@us...>]

Revision: 5236
          http://jython.svn.sourceforge.net/jython/?rev=5236&view=rev
Author:   fwierzbicki
Date:     2008-08-22 23:03:45 +0000 (Fri, 22 Aug 2008)

Log Message:
-----------
Bugfix for expr_contextType on assign values.

Modified Paths:
--------------
    branches/nowalker/src/org/python/antlr/GrammarActions.java

Modified: branches/nowalker/src/org/python/antlr/GrammarActions.java
===================================================================
--- branches/nowalker/src/org/python/antlr/GrammarActions.java	2008-08-22 23:02:32 UTC (rev 5235)
+++ branches/nowalker/src/org/python/antlr/GrammarActions.java	2008-08-22 23:03:45 UTC (rev 5236)
@@ -284,9 +284,9 @@
             if (o instanceof PythonParser.testlist_return) {
                 //XXX: Check to see if this is really happening anymore
                 PythonParser.testlist_return r = (PythonParser.testlist_return)o;
-                e[i] = (exprType)r.getTree();
+                e[i + 1] = (exprType)r.getTree();
             } else {
-                e[i] = (exprType)o;
+                e[i + 1] = (exprType)o;
             }
         }
         return e;
@@ -300,7 +300,6 @@
 
     void recurseSetContext(Tree tree, expr_contextType context) {
         if (tree instanceof Context) {
-            //XXX: for Tuples, etc -- will need to recursively set to expr_contextType.Load.
             ((Context)tree).setContext(context);
         }
         for (int i=0; i<tree.getChildCount(); i++) {


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.