Some minor changes

[phpeclipse.git] / net.sourceforge.phpeclipse / src / net / sourceforge / phpdt / internal / compiler / ast / FunctionCall.java

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/FunctionCall.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/FunctionCall.java
index 4aee466..d897240 100644 (file)
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/FunctionCall.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/FunctionCall.java
@@ -1,18 +1,31 @@
 package net.sourceforge.phpdt.internal.compiler.ast;
 
+import java.util.List;
+
 /**
+ * A Function call.
  * @author Matthieu Casanova
  */
-public class FunctionCall extends AbstractSuffixExpression {
+public final class FunctionCall extends AbstractSuffixExpression {
+
+  /** the function name. */
+  private final Expression functionName;
 
-  public Expression prefix;
-  public ArgumentDeclaration[] args;
+  /** the arguments. */
+  private final Expression[] args;
 
-  public FunctionCall(Expression prefix,
-                      ArgumentDeclaration[] args,
-                      int sourceEnd) {
-    super(prefix.sourceStart, sourceEnd);
-    this.prefix = prefix;
+  /**
+   * a function call.
+   * it's <code>functionName(args ...)
+   * @param functionName the function name
+   * @param args the arguments
+   * @param sourceEnd the source end
+   */
+  public FunctionCall(final Expression functionName,
+                      final Expression[] args,
+                      final int sourceEnd) {
+    super(functionName.sourceStart, sourceEnd);
+    this.functionName = functionName;
     this.args = args;
   }
 
@@ -21,16 +34,52 @@ public class FunctionCall extends AbstractSuffixExpression {
    * @return the expression
    */
   public String toStringExpression() {
-    final StringBuffer buff = new StringBuffer(prefix.toStringExpression());
+    final StringBuffer buff = new StringBuffer(functionName.toStringExpression());
     buff.append('(');
-    for (int i = 0; i < args.length; i++) {
-      ArgumentDeclaration arg = args[i];
-      if (i != 0) {
-        buff.append(',');
+    if (args != null) {
+      for (int i = 0; i < args.length; i++) {
+        final Expression arg = args[i];
+        if (i != 0) {
+          buff.append(',');
+        }
+        buff.append(arg.toStringExpression());
       }
-      buff.append(arg);
     }
     buff.append(')');
     return buff.toString();
   }
+
+  /**
+   * Get the variables from outside (parameters, globals ...)
+   *
+   * @param list the list where we will put variables
+   */
+  public void getOutsideVariable(final List list) {}
+
+  /**
+   * get the modified variables.
+   *
+   * @param list the list where we will put variables
+   */
+  public void getModifiedVariable(final List list) {
+    if (args != null) {
+      for (int i = 0; i < args.length; i++) {
+        args[i].getModifiedVariable(list);
+      }
+    }
+  }
+
+  /**
+   * Get the variables used.
+   *
+   * @param list the list where we will put variables
+   */
+  public void getUsedVariable(final List list) {
+    functionName.getUsedVariable(list);
+    if (args != null) {
+      for (int i = 0; i < args.length; i++) {
+        args[i].getUsedVariable(list);
+      }
+    }
+  }
 }