Larry,
Thanks for your help with this. I still don't understand why we are getting
this exception. It must be becuase iteration can cause a
ConcurrentModificationException, despite what the Javadoc says. I'll get
this fix commited to OpenJUMP's code base tomorrow, I hope...
I might also try to find someone that can help me identify if this is a
possible problem with Sun's implementation of the
ConcurrentModificationException, or if I'm just overlooking something. I
just don't see where the code in the fireCategoryChanged method is
"modifying" the collection of listeners, which is supposed to be the only
way a ConcurrentModificationException can be thrown.
I'm also passing along this message to Vivid Solutions so they can make the
fix in JUMP. If they want, I can provide Vivid with the patched source code
file so all they have to do is make the replacement.
The Sunburned Surveyor
On 2/1/07, Larry Becker <[EMAIL PROTECTED]> wrote:
Sunburned,
I caught the Concurrent mod error in Eclipse and got the console
traceback, the same as I posted before.
I modified line 335 in fireCategoryChanged using the same technique as you
did in fireFeaturesChanged and it seems to have fixed the problem.
for (Iterator i = new ArrayList(layerListeners).iterator();
i.hasNext();) {//LDB added
I tried it 14 times in a row and didn't get the error. Previously it had
never gone past 8 without hitting it. Hope it is fixed for good now!
regards,
Larry
On 2/1/07, Sunburned Surveyor <[EMAIL PROTECTED]> wrote:
>
> Thanks a lot Larry. I notice that the last portion of your stack trace
> looks almost exactly like Pedro's. The one glaring similarity. Both your
> stack trace and Pedro's contain multiple and subseqent calls to the
> LayerManager.addCategoryMethod. I'm almost posotive that this is the
> root of the problem. I wonder if the JVM could be implementing the behavior
> of this exception differently than is stated in the Javadoc?
>
> I know that is a remote chance, and that I'm probably just overlooking
> something. Still, I thought I had put this stinker to rest.
>
> The good news is there's an easy fix if the problem is coming from what
> I think. I'll try the fix out and then see if I can duplicate the error with
> your sample project file.
>
> SS
>
>
> On 2/1/07, Larry Becker <[EMAIL PROTECTED] > wrote:
> >
> > Sunburned,
> >
> > I have attached the project that triggered the problem. There
> > doesn't seem to be anything special about it, but I got it to reproduce
> > again after only 7 tries. Just extract the zip and open the project file.
> >
> > The stack trace is in the last email. It looks very much like
> > Pedro's. I'll try to work on it too. Who knows, as I implement more of OJ
> > into SkyJUMP, I might copy the bug (assuming I haven't already).
> >
> > regards,
> > Larry
> >
> > On 2/1/07, Sunburned Surveyor <[EMAIL PROTECTED] > wrote:
> > >
> > > A seemingly random exception. That's got to be a timing thing with
> > > threads. I wish I knew how to fix the stinking thing.
> > >
> > > If it happens again try and grab me that stack trace.
> > >
> > > SS
> > >
> > > P.S. - To which logging function do you refer?
> > >
> > > On 2/1/07, Larry Becker <[EMAIL PROTECTED] > wrote:
> > > >
> > > > I should also note, so as not to overly alarm anyone, that the
> > > > error was fully recoverable (like all of JUMP's errors), and did not
happen
> > > > again although I immediately tried loading the project again several
times.
> > > >
> > > > Larry
> > > >
> > > > On 2/1/07, Larry Becker <[EMAIL PROTECTED] > wrote:
> > > > >
> > > > > The concurrent modification error occurred for me just as it did
> > > > > for Pedro - when opening a project file. It got almost all of the way
> > > > > through loading the layers before it hit. I'll try to reproduce it
again
> > > > > and get a trace this time. I was just taken off guard by the Log
command
> > > > > not being there.
> > > > >
> > > > > regards,
> > > > > Larry
> > > > >
> > > > > On 2/1/07, Sunburned Surveyor < [EMAIL PROTECTED] >
> > > > > wrote:
> > > > > >
> > > > > > I am wanting to use curse words so bad right now that I could
> > > > > > almost spit!
> > > > > >
> > > > > > Thanks for letting me know about this Larry. I don't know how
> > > > > > much I can do to fix this without a stack trace. I'm also surprised
that the
> > > > > > error only appears in OpenJUMP. That will help me narrow my search
to code
> > > > > > in JUMP's core that has been modified in OpenJUMP. I'd have to look
at the
> > > > > > PrintLayoutCode to determine what caused the Exception. Do you
remember what
> > > > > > exactly you were doing in the PrintLayoutPlugin when you got the
Exception?
> > > > > >
> > > > > > Landon
> > > > > >
> > > > > >
> > > > > > On 2/1/07, Larry Becker < [EMAIL PROTECTED] > wrote:
> > > > > >
> > > > > > > Hey Sunburned,
> > > > > > >
> > > > > > > While I was testing out the PrintLayout, I got a
> > > > > > > Concurrent modification error on a project with only shapefiles.
This seems
> > > > > > > to point to OJ specifically, since I have never yet gotten this
error with
> > > > > > > SkyJUMP. Unfortunately, I didn't get a trace. I tried to look
in the log,
> > > > > > > but I guess OpenJUMP doesn't have a log anymore.
> > > > > > >
> > > > > > > regards,
> > > > > > > Larry
> > > > > > > --
> > > > > > > http://amusingprogrammer.blogspot.com/
> > > > > > >
> > > > > > >
-------------------------------------------------------------------------
> > > > > > > Using Tomcat but need to do more? Need to support web
> > > > > > > services, security?
> > > > > > > Get stuff done quickly with pre-integrated technology to
> > > > > > > make your job easier.
> > > > > > > Download IBM WebSphere Application Server v.1.0.1 based on
> > > > > > > Apache Geronimo
> > > > > > >
> > > > > > >
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > Jump-pilot-devel mailing list
> > > > > > > Jump-pilot-devel@lists.sourceforge.net
> > > > > > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
-------------------------------------------------------------------------
> > > > > > Using Tomcat but need to do more? Need to support web
> > > > > > services, security?
> > > > > > Get stuff done quickly with pre-integrated technology to make
> > > > > > your job easier.
> > > > > > Download IBM WebSphere Application Server v.1.0.1 based on
> > > > > > Apache Geronimo
> > > > > >
> > > > > >
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > > > > > _______________________________________________
> > > > > > Jump-pilot-devel mailing list
> > > > > > Jump-pilot-devel@lists.sourceforge.net
> > > > > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > --
> > > > > http://amusingprogrammer.blogspot.com/
> > > >
> > > >
> > > >
> > > >
> > > > --
> > > > http://amusingprogrammer.blogspot.com/
> > > >
> > > >
-------------------------------------------------------------------------
> > > > Using Tomcat but need to do more? Need to support web services,
> > > > security?
> > > > Get stuff done quickly with pre-integrated technology to make your
> > > > job easier.
> > > > Download IBM WebSphere Application Server v.1.0.1 based on Apache
> > > > Geronimo
> > > >
> > > > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > > >
> > > > _______________________________________________
> > > > Jump-pilot-devel mailing list
> > > > Jump-pilot-devel@lists.sourceforge.net
> > > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> > > >
> > > >
> > > >
> > >
> > >
> > > -------------------------------------------------------------------------
> > > Using Tomcat but need to do more? Need to support web services,
> > > security?
> > > Get stuff done quickly with pre-integrated technology to make your
> > > job easier.
> > > Download IBM WebSphere Application Server v.1.0.1 based on Apache
> > > Geronimo
> > >
> > > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
> > > _______________________________________________
> > > Jump-pilot-devel mailing list
> > > Jump-pilot-devel@lists.sourceforge.net
> > > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> > >
> > >
> >
> >
> > --
> > http://amusingprogrammer.blogspot.com/
> >
> >
>
--
http://amusingprogrammer.blogspot.com/
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
