Some minor changes

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

diff --git a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ClassDeclaration.java b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ClassDeclaration.java
index af9ed52..adb4332 100644 (file)
--- a/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ClassDeclaration.java
+++ b/net.sourceforge.phpeclipse/src/net/sourceforge/phpdt/internal/compiler/ast/ClassDeclaration.java
@@ -1,14 +1,15 @@
 package net.sourceforge.phpdt.internal.compiler.ast;
 
+import java.util.ArrayList;
+import java.util.List;
+
 import net.sourceforge.phpdt.internal.compiler.parser.Outlineable;
 import net.sourceforge.phpdt.internal.compiler.parser.OutlineableWithChildren;
 import net.sourceforge.phpdt.internal.ui.PHPUiImages;
+
 import org.eclipse.jface.resource.ImageDescriptor;
 import org.eclipse.jface.text.Position;
 
-import java.util.ArrayList;
-import java.util.List;
-
 
 /**
  * This class is my ClassDeclaration declaration for php.
@@ -16,12 +17,12 @@ import java.util.List;
  * It directly extends AstNode because a class cannot appear anywhere in php
  * @author Matthieu Casanova
  */
-public class ClassDeclaration extends Statement implements OutlineableWithChildren {
+public final class ClassDeclaration extends Statement implements OutlineableWithChildren {
 
   /** The name of the class. */
-  public String name;
+  private final String name;
   /** The superclass. */
-  public String superclass;
+  private String superclass;
 
   public int declarationSourceStart;
   public int declarationSourceEnd;
@@ -30,18 +31,18 @@ public class ClassDeclaration extends Statement implements OutlineableWithChildr
   /** The methods of the class. */
   private final ArrayList methods = new ArrayList();
   /** The constructor of the class. */
-  public MethodDeclaration constructor;
+  private MethodDeclaration constructor;
   /** The fields of the class. */
-  private ArrayList fields = new ArrayList();
+  private final ArrayList fields = new ArrayList();
 
-  private Object parent;
+  private final Object parent;
   /** The outlineable children (those will be in the node array too. */
-  private ArrayList children = new ArrayList();
+  private final ArrayList children = new ArrayList();
 
-  private Position position;
+  private final Position position;
 
   /**
-   * Create a class giving starting and ending offset
+   * Create a class giving starting and ending offset.
    * @param sourceStart starting offset
    * @param sourceEnd ending offset
    */
@@ -58,7 +59,7 @@ public class ClassDeclaration extends Statement implements OutlineableWithChildr
   }
 
   /**
-   * Create a class giving starting and ending offset
+   * Create a class giving starting and ending offset.
    * @param sourceStart starting offset
    * @param sourceEnd ending offset
    */
@@ -118,7 +119,7 @@ public class ClassDeclaration extends Statement implements OutlineableWithChildr
    * @param tab how many tabs before the body of the class
    * @return the body as String
    */
-  public String toStringBody(final int tab) {
+  private String toStringBody(final int tab) {
     final StringBuffer buff = new StringBuffer(" {");//$NON-NLS-1$
     if (fields != null) {
       for (int i = 0; i < fields.size(); i++) {
@@ -141,7 +142,7 @@ public class ClassDeclaration extends Statement implements OutlineableWithChildr
    * Return the header of the class as String.
    * @return the header of the class
    */
-  public String toStringHeader() {
+  private String toStringHeader() {
     final StringBuffer buff = new StringBuffer("class ").append(name);//$NON-NLS-1$
     if (superclass != null) {
       buff.append(" extends "); //$NON-NLS-1$
@@ -189,25 +190,22 @@ public class ClassDeclaration extends Statement implements OutlineableWithChildr
 
   /**
    * Get the variables from outside (parameters, globals ...)
-   * @return the variables from outside
+   *
+   * @param list the list where we will put variables
    */
-  public List getOutsideVariable() {
-    return new ArrayList(1);
-  }
+  public void getOutsideVariable(final List list) {}
 
   /**
    * get the modified variables.
-   * @return the variables from we change value
+   *
+   * @param list the list where we will put variables
    */
-  public List getModifiedVariable() {
-    return new ArrayList(1);
-  }
+  public void getModifiedVariable(final List list) {}
 
   /**
    * Get the variables used.
-   * @return the variables used
+   *
+   * @param list the list where we will put variables
    */
-  public List getUsedVariable() {
-    return new ArrayList(1);
-  }
+  public void getUsedVariable(final List list) {}
 }