Some minor changes

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

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ReturnStatement.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ReturnStatement.java
index ff4a838..61e5cec 100644 (file)
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ReturnStatement.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ReturnStatement.java
@@ -6,9 +6,9 @@ import java.util.List;
  * A return statement.
  * @author Matthieu Casanova
  */
-public class ReturnStatement extends Statement {
+public final class ReturnStatement extends Statement {
 
-  public Expression expression;
+  private final Expression expression;
 
   public ReturnStatement(final Expression expression, final int sourceStart, final int sourceEnd) {
     super(sourceStart, sourceEnd);
@@ -23,14 +23,17 @@ public class ReturnStatement extends Statement {
     return s + "return " + expression.toStringExpression();//$NON-NLS-1$
   }
 
-    /**
+  /**
    * 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) {
     if (expression != null) {
@@ -40,6 +43,8 @@ public class ReturnStatement extends Statement {
 
   /**
    * Get the variables used.
+   *
+   * @param list the list where we will put variables
    */
   public void getUsedVariable(final List list) {
     if (expression != null) {