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

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

Revision: 2596
          http://svn.sourceforge.net/rubyeclipse/?rev=2596&view=rev
Author:   cawilliams
Date:     2007-06-11 09:23:39 -0700 (Mon, 11 Jun 2007)

Log Message:
-----------
only mark name of method when we have other sorts (besidess line count) of complexity errors

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

Modified: trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/CodeComplexityVisitor.java
===================================================================
--- trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/CodeComplexityVisitor.java	2007-06-11 16:22:28 UTC (rev 2595)
+++ trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/CodeComplexityVisitor.java	2007-06-11 16:23:39 UTC (rev 2596)
@@ -107,13 +107,13 @@
 
 	public void exitDefnNode(DefnNode iVisited) {
 		if (returnCount > maxReturns) {
-			createProblem(iVisited.getPosition(), "Too many explicit returns: " + returnCount);
+			createProblem(iVisited.getNameNode().getPosition(), "Too many explicit returns: " + returnCount);
 		}
 		if (branchCount > maxBranches) {
-			createProblem(iVisited.getPosition(), "Too many branches: " + branchCount);
+			createProblem(iVisited.getNameNode().getPosition(), "Too many branches: " + branchCount);
 		}
 		if (locals.size() > maxLocals) {
-			createProblem(iVisited.getPosition(), "Too many local variables: " + locals.size());
+			createProblem(iVisited.getNameNode().getPosition(), "Too many local variables: " + locals.size());
 		}
 		returnCount = 0;
 		branchCount = 0;


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