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

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

Revision: 3008
          http://rubyeclipse.svn.sourceforge.net/rubyeclipse/?rev=3008&view=rev
Author:   cawilliams
Date:     2007-08-20 06:17:20 -0700 (Mon, 20 Aug 2007)

Log Message:
-----------
set warning for possible accidental boolean assignment on the assignment position (not full if body)

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

Modified: trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/AccidentalBooleanAssignmentVisitor.java
===================================================================
--- trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/AccidentalBooleanAssignmentVisitor.java	2007-08-17 18:04:15 UTC (rev 3007)
+++ trunk/com.aptana.rdt/src/com/aptana/rdt/internal/parser/warnings/AccidentalBooleanAssignmentVisitor.java	2007-08-20 13:17:20 UTC (rev 3008)
@@ -25,7 +25,7 @@
 	public Instruction visitIfNode(IfNode iVisited) {
 		Node condition = iVisited.getCondition();
 		if (containsAssignment(condition)) {
-			createProblem(iVisited.getPosition(), "Possible accidental boolean assignment");
+			createProblem(condition.getPosition(), "Possible accidental boolean assignment");
 		}
 		return super.visitIfNode(iVisited);
 	}


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