Seems like a harmless change to me.
Larry
On 7/11/07, Sunburned Surveyor <[EMAIL PROTECTED]> wrote:
> This seems like a logical patch to apply Paul. If no one pipes up
> about a problem I think you should go for it.
>
> SS
>
> On 7/11/07, Paul Austin <[EMAIL PROTECTED]> wrote:
> > I have created a patch for the InfoModel class so that it does not
> > actually remove the features for a Layer until after it has notified the
> > listeners. This enables the listeners to get the features that are going
> > to be removed.
> >
> > Can people have quick look and see if their plugins would be affected
> > by this change before I check-in.
> >
> > Paul
> >
> > Index: C:/Documents and Settings/paustin/My
> > Documents/workspace/openjump/src/com/vividsolutions/jump/workbench/ui/InfoModel.java
> > ===================================================================
> > --- C:/Documents and Settings/paustin/My
> > Documents/workspace/openjump/src/com/vividsolutions/jump/workbench/ui/InfoModel.java
> > (revision 887)
> > +++ C:/Documents and Settings/paustin/My
> > Documents/workspace/openjump/src/com/vividsolutions/jump/workbench/ui/InfoModel.java
> > (working copy)
> > @@ -92,8 +92,6 @@
> >
> > public void remove(Layer layer) {
> > LayerTableModel layerTableModel = getTableModel(layer);
> > - ((LayerTableModel) layerToTableModelMap.get(layer)).dispose();
> > - layerToTableModelMap.remove(layer);
> >
> > for (Iterator i = listeners.iterator(); i.hasNext();) {
> > InfoModelListener listener = (InfoModelListener) i.next();
> > @@ -99,6 +97,8 @@
> > InfoModelListener listener = (InfoModelListener) i.next();
> > listener.layerRemoved(layerTableModel);
> > }
> > + ((LayerTableModel) layerToTableModelMap.get(layer)).dispose();
> > + layerToTableModelMap.remove(layer);
> > }
> >
> > public void clear() {
> >
> >
> > -------------------------------------------------------------------------
> > This SF.net email is sponsored by DB2 Express
> > Download DB2 Express C - the FREE version of DB2 express and take
> > control of your XML. No limits. Just data. Click to get it now.
> > http://sourceforge.net/powerbar/db2/
> > _______________________________________________
> > Jump-pilot-devel mailing list
> > Jump-pilot-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> >
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
--
http://amusingprogrammer.blogspot.com/
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
