X-Git-Url: http://secure.phpeclipse.com

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/BinaryExpression.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/BinaryExpression.java
index 46643d1..7488cb8 100644
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/BinaryExpression.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/BinaryExpression.java
@@ -1,16 +1,26 @@
 package net.sourceforge.phpdt.internal.compiler.ast;
 
 import java.util.List;
-import java.util.ArrayList;
 
 /**
+ * a binary expression is a combination of two expressions with an operator.
+ * 
  * @author Matthieu Casanova
  */
-public class BinaryExpression extends OperatorExpression {
+public final class BinaryExpression extends OperatorExpression {
 
-  /** The two expressions. */
-  public Expression left,right;
+  /** The left expression. */
+  private final Expression left;
+  /** The right expression. */
+  private final Expression right;
 
+  /**
+   * Create a binary expression.
+   * 
+   * @param left     the left expression
+   * @param right    the right expression
+   * @param operator an operator taken in the {@link OperatorExpression} interface
+   */
   public BinaryExpression(final Expression left,
                           final Expression right,
                           final int operator) {
@@ -20,20 +30,27 @@ public class BinaryExpression extends OperatorExpression {
   }
 
   public String toStringExpression() {
-    final StringBuffer buff = new StringBuffer(left.toStringExpression());
-    buff.append(operatorToString());
-    buff.append(right.toStringExpression());
+    final String leftString = left.toStringExpression();
+    final String operatorString = operatorToString();
+    final String rightString = right.toStringExpression();
+    final StringBuffer buff = new StringBuffer(leftString.length() + operatorString.length() + rightString.length());
+    buff.append(leftString);
+    buff.append(operatorString);
+    buff.append(rightString);
     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) {
-  }
+  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) {
     left.getModifiedVariable(list);
@@ -42,6 +59,8 @@ public class BinaryExpression extends OperatorExpression {
 
   /**
    * Get the variables used.
+   * 
+   * @param list the list where we will put variables
    */
   public void getUsedVariable(final List list) {
     left.getUsedVariable(list);