Hi,

You're probably right, but that's not so simple because the LayerManager does't know which layer each layerListener relates to.

Maybe we need to keep a reference to the Layer in LayerListeners or to replace the layerListener list of the LayerManager by a Map mapping layers to Listeners.

Anyway, I tried to fix it without implementation change. Now, any improvement is welcome.

Michaël

Le 04/06/2018 à 11:29, edgar.sol...@web.de a écrit :
hey Mike,

On 04.06.2018 00:37, jump-pilot-svn--- via Jump-pilot-devel wrote:
also remove layerListeners added with associated to AttributeTablePanel,
     AttributeTab, OneLayerAttributeTab, ViewAttributesFrame, InfoFrame,
     WorkbenchFrame, EnsureAllLayersHaveSrid when the Layer is removed
wouldn't it be cleaner and much more /robust/ to remove the listeners where they are 
registered/the event is handled? that way no listener can "forget" to implement 
the cleanup.

..ede

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel



------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to