[Bprocessor-commit] model/src/net/sourceforge/bprocessor/model Line.java, 1.33, 1.34 Space.java, 1.158, 1.159 Point.java, 1.17, 1.18 CoordinateSystem.java, 1.53, 1.54 Constructor.java, 1.20, 1.21

[Michael L. <he...@us...>]

Update of /cvsroot/bprocessor/model/src/net/sourceforge/bprocessor/model
In directory sc8-pr-cvs3.sourceforge.net:/tmp/cvs-serv13397/src/net/sourceforge/bprocessor/model

Modified Files:
	Line.java Space.java Point.java CoordinateSystem.java 
	Constructor.java 
Log Message:
Refactoring of delete

Index: Constructor.java
===================================================================
RCS file: /cvsroot/bprocessor/model/src/net/sourceforge/bprocessor/model/Constructor.java,v
retrieving revision 1.20
retrieving revision 1.21
diff -C2 -d -r1.20 -r1.21
*** Constructor.java	18 Jul 2007 12:24:05 -0000	1.20
--- Constructor.java	18 Jul 2007 13:35:39 -0000	1.21
***************
*** 119,126 ****
      return origin;
    }
-   
-   /** {@inheritDoc} */
-   @Override
-   public abstract void delete();
  
    /**
--- 119,122 ----

Index: CoordinateSystem.java
===================================================================
RCS file: /cvsroot/bprocessor/model/src/net/sourceforge/bprocessor/model/CoordinateSystem.java,v
retrieving revision 1.53
retrieving revision 1.54
diff -C2 -d -r1.53 -r1.54
*** CoordinateSystem.java	18 Jul 2007 12:24:04 -0000	1.53
--- CoordinateSystem.java	18 Jul 2007 13:35:39 -0000	1.54
***************
*** 515,518 ****
--- 515,521 ----
    @Override
    public void delete() {
+     // FIXME other coordinatesystems may refer to this
+     // CoordinateSystem. These references should be
+     // cleared.
      getOwner().remove(this);
    }

Index: Point.java
===================================================================
RCS file: /cvsroot/bprocessor/model/src/net/sourceforge/bprocessor/model/Point.java,v
retrieving revision 1.17
retrieving revision 1.18
diff -C2 -d -r1.17 -r1.18
*** Point.java	18 Jul 2007 12:24:05 -0000	1.17
--- Point.java	18 Jul 2007 13:35:38 -0000	1.18
***************
*** 41,44 ****
--- 41,45 ----
    public void delete() {
      getOwner().remove(this);
+     // FIXME could other objects refer to this Point?
    }
  

Index: Space.java
===================================================================
RCS file: /cvsroot/bprocessor/model/src/net/sourceforge/bprocessor/model/Space.java,v
retrieving revision 1.158
retrieving revision 1.159
diff -C2 -d -r1.158 -r1.159
*** Space.java	18 Jul 2007 12:24:04 -0000	1.158
--- Space.java	18 Jul 2007 13:35:38 -0000	1.159
***************
*** 1688,1691 ****
--- 1688,1692 ----
    @Override
    public void delete() {
+     // FIXME should we clear this Space?
      clear();
      Space owner = this.getOwner();

Index: Line.java
===================================================================
RCS file: /cvsroot/bprocessor/model/src/net/sourceforge/bprocessor/model/Line.java,v
retrieving revision 1.33
retrieving revision 1.34
diff -C2 -d -r1.33 -r1.34
*** Line.java	18 Jul 2007 12:24:04 -0000	1.33
--- Line.java	18 Jul 2007 13:35:38 -0000	1.34
***************
*** 106,109 ****
--- 106,110 ----
    @Override
    public void delete() {
+     // FIXME can other objects in the model refer to this Line?
      getOwner().remove(this);
    }