[Rubyeclipse-commit] SF.net SVN: rubyeclipse: [2623] trunk/com.aptana.rdt/src/com/aptana/rdt/ internal/parser/warnings/TooManyLocalsVisitor.java

[<caw...@us...>]

Revision: 2623
          http://svn.sourceforge.net/rubyeclipse/?rev=2623&view=rev
Author:   cawilliams
Date:     2007-06-18 13:07:09 -0700 (Mon, 18 Jun 2007)

Log Message:
-----------
handle locals defined in top-level

Modified Paths:
--------------
    trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/TooManyLocalsVisitor.java

Modified: trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/TooManyLocalsVisitor.java
===================================================================
--- trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/TooManyLocalsVisitor.java	2007-06-15 15:24:00 UTC (rev 2622)
+++ trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/TooManyLocalsVisitor.java	2007-06-18 20:07:09 UTC (rev 2623)
@@ -7,6 +7,7 @@
 import org.jruby.ast.DefnNode;
 import org.jruby.ast.DefsNode;
 import org.jruby.ast.LocalAsgnNode;
+import org.jruby.ast.RootNode;
 import org.jruby.evaluator.Instruction;
 import org.rubypeople.rdt.internal.core.parser.warnings.RubyLintVisitor;
 
@@ -40,6 +41,15 @@
 	protected String getOptionKey() {
 		return AptanaRDTPlugin.COMPILER_PB_MAX_LOCALS;
 	}
+	
+	@Override
+	public Instruction visitRootNode(RootNode iVisited) {
+		locals = new HashSet(); // FIXME Keep a scoped stack of locals?!
+		Instruction ins = super.visitRootNode(iVisited);
+		locals.clear();
+		return ins;
+	}
+	
 
 	@Override
 	public Instruction visitDefsNode(DefsNode iVisited) {


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