Remove unused fields
Clément DAVID [Mon, 11 Jan 2010 13:17:54 +0000 (14:17 +0100)]
Use accessor and protect fields (private final when possible)

scilab/modules/xcos/src/java/org/scilab/modules/graph/ScilabGraph.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/graph/PaletteDiagram.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/graph/XcosDiagram.java

index 840d0e4..68d50c0 100644 (file)
@@ -41,11 +41,8 @@ import com.mxgraph.view.mxGraph;
  */
 public class ScilabGraph extends mxGraph {
 
-    protected mxUndoManager undoManager = new mxUndoManager();
-    protected mxGraphOutline graphOutline = null;
-    protected mxKeyboardHandler keyboardHandler = null;
-    protected XcosComponent component = null;
-    //    protected mxGraphComponent component = null;
+    private final mxUndoManager undoManager = new mxUndoManager();
+    private final XcosComponent component;
 
     private String title = ScilabGraphMessages.UNTITLED;
     private String savedFile = null;
@@ -365,4 +362,11 @@ public class ScilabGraph extends mxGraph {
     public mxRubberband getRubberBand() {
         return rubberBand;
     }
+
+    /**
+     * @return The undo manager associated with this graph
+     */
+       protected final mxUndoManager getUndoManager() {
+               return undoManager;
+       }
 }
index 2907908..2384a30 100644 (file)
@@ -41,7 +41,7 @@ public class PaletteDiagram extends XcosDiagram {
        setCellsDeletable(false);
        setCellsEditable(false);
        
-       undoManager.setEventsEnabled(false);
+       getUndoManager().setEventsEnabled(false);
     }
 
     public boolean openDiagramAsPal(String diagramFileName) {
index 0d84f34..6249dfd 100644 (file)
@@ -409,7 +409,8 @@ public class XcosDiagram extends ScilabGraph {
        super();
        getModel().addListener(mxEvent.UNDO, undoEnabler);
        getView().addListener(mxEvent.UNDO, undoEnabler);
-       keyboardHandler = new XcosShortCut(this);
+       // The association is inverted there (by the parameter)
+       new XcosShortCut(this);
        mxCodec codec = new mxCodec();
 
        try {
@@ -609,8 +610,8 @@ public class XcosDiagram extends ScilabGraph {
        addListener(XcosEvent.FORCE_CELL_VALUE_UPDATE, new ForceCellValueUpdate());
        
        // Update the blocks view on undo/redo
-       undoManager.addListener(mxEvent.UNDO, new UndoUpdateTracker());
-       undoManager.addListener(mxEvent.REDO, new UndoUpdateTracker());
+       getUndoManager().addListener(mxEvent.UNDO, new UndoUpdateTracker());
+       getUndoManager().addListener(mxEvent.REDO, new UndoUpdateTracker());
        
        getAsComponent().getGraphControl().addMouseListener(new XcosMouseListener(this));
 
@@ -1705,7 +1706,7 @@ public class XcosDiagram extends ScilabGraph {
        //getParentTab().setName((String) properties.get("title"));
 
        // Clear all undo events in Undo Manager
-       undoManager.reset();
+       getUndoManager().reset();
        setModified(false);
     }
 
@@ -1996,7 +1997,7 @@ public class XcosDiagram extends ScilabGraph {
        super.undo();
 
        if (getParentTab() != null) {
-           if (undoManager.canUndo()) {
+           if (getUndoManager().canUndo()) {
                ((XcosTab) getParentTab()).setEnabledUndo(true);
            } else {
                ((XcosTab) getParentTab()).setEnabledUndo(false);
@@ -2030,12 +2031,12 @@ public class XcosDiagram extends ScilabGraph {
        updateUndoModifiedState();
 
        if (getParentTab() != null) {
-           if (undoManager.canUndo()) {
+           if (getUndoManager().canUndo()) {
                ((XcosTab) getParentTab()).setEnabledUndo(true);
            } else {
                ((XcosTab) getParentTab()).setEnabledUndo(false);
            }
-           if (undoManager.canRedo()) {
+           if (getUndoManager().canRedo()) {
                ((XcosTab) getParentTab()).setEnabledRedo(true);
            } else {
                ((XcosTab) getParentTab()).setEnabledRedo(false);
@@ -2047,7 +2048,7 @@ public class XcosDiagram extends ScilabGraph {
      * This function will reset the UndoManager in a stable state.
      */
     public void resetUndoManager() {
-       undoManager.reset();
+       getUndoManager().reset();
 
        resetUndoCounter();