*** empty log message ***
[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpdt / internal / compiler / ast / MethodDeclaration.java
index 93b8758..615fc04 100644 (file)
@@ -160,7 +160,7 @@ public class MethodDeclaration extends Statement implements OutlineableWithChild
     if (arguments != null) {
       final Enumeration vars = arguments.keys();
       while (vars.hasMoreElements()) {
-        list.add(new VariableUsage((String) vars.nextElement(), sourceStart));
+        list.add(new VariableUsage((String) vars.nextElement(), getSourceStart()));
       }
     }
 
@@ -241,11 +241,20 @@ public class MethodDeclaration extends Statement implements OutlineableWithChild
 
 
     //look for used variables that were not declared before
+    findUnknownUsedVars(usedVars, declaredVars);
+  }
+
+  /**
+   * This method will add a warning on all used variables in a method that aren't declared before.
+   * @param usedVars the used variable list
+   * @param declaredVars the declared variable list
+   */
+  private void findUnknownUsedVars(final List usedVars, final List declaredVars) {
     for (int i = 0; i < usedVars.size(); i++) {
       VariableUsage variableUsage = (VariableUsage) usedVars.get(i);
       if (!isVariableDeclaredBefore(declaredVars, variableUsage)) {
         try {
-          PHPParserSuperclass.setMarker("warning, usage of an unknown : " + variableUsage.getName(),
+          PHPParserSuperclass.setMarker("warning, usage of an unknown variable : " + variableUsage.getName(),
                                         variableUsage.getStartOffset(),
                                         variableUsage.getStartOffset() + variableUsage.getName().length(),
                                         PHPParserSuperclass.WARNING,
@@ -254,7 +263,6 @@ public class MethodDeclaration extends Statement implements OutlineableWithChild
           PHPeclipsePlugin.log(e);
         }
       }
-
     }
   }
 }