Some minor changes

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

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ListExpression.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ListExpression.java
index ad5c35c..a277078 100644 (file)
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ListExpression.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ListExpression.java
@@ -1,17 +1,16 @@
 package net.sourceforge.phpdt.internal.compiler.ast;
 
 import java.util.List;
-import java.util.ArrayList;
 
 /**
  * A list expression.
  * it could be list($v1,$v2), list(,$v2) ...
  * @author Matthieu Casanova
  */
-public class ListExpression extends Expression {
+public final class ListExpression extends Expression {
 
-  public Expression[] vars;
-  public Expression expression;
+  private final Expression[] vars;
+  private Expression expression;
 
   public ListExpression(final Expression[] vars,
                         final Expression expression,
@@ -52,12 +51,15 @@ public class ListExpression extends Expression {
 
   /**
    * Get the variables from outside (parameters, globals ...)
+   *
+   * @param list the list where we will put variables
    */
-  public void getOutsideVariable(final List list) {
-  }
+  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) {
     for (int i = 0; i < vars.length; i++) {
@@ -72,6 +74,8 @@ public class ListExpression extends Expression {
 
   /**
    * Get the variables used.
+   *
+   * @param list the list where we will put variables
    */
   public void getUsedVariable(final List list) {
     if (expression != null) {