Re: [JPP-Devel] Modified LayerManager class that fixes Bug #1487099...

2007-01-18 Thread Sunburned Surveyor
Larry, You could be right. I didn't even think about the iteration occuring in the GUI Thread. Does this mean that the Runnable argument passed to the LayerManager.fireCategoryChanged() method is in fact the GUI Thread? It looked like OpenJUMP/JUMP was creating a brand new thread using an inner

Re: [JPP-Devel] Modified LayerManager class that fixes Bug #1487099...

2007-01-18 Thread Larry Becker
Sunburned, Well, it calls fireLayerEvent with a new inline Runnable class method run defined to do layerListener.categoryChanged. Then fireLayerEvent invokes GUIUtil.invokeOnEventThread which does: if (SwingUtilities.isEventDispatchThread()) { r.run(); } else {

Re: [JPP-Devel] Modified LayerManager class that fixes Bug #1487099...

2007-01-18 Thread Sunburned Surveyor
I know exactly what you mean! I'm sorry I ever selected this bug out of the tracker. :] I should have followed the method calls that result from the call to LayerManager.fireCategoryChanged() farther down the chain than I did. I would still like to understand how things are working. I will

Re: [JPP-Devel] Modified LayerManager class that fixes Bug #1487099...

2007-01-18 Thread Stefan Steiniger
This is one of those pieces of code that the more I look at it, the more I see I don't see. Know what I mean? i dont understand that really but i know that i do not understand.. thats why i am a GIS guy and not a programmer :o) good night, stefan