On Tue, Apr 15, 2008 at 5:18 AM, Matthew Toseland <toad at amphibian.dyndns.org> wrote: > Umm, what do you think the synchronized() is for? Why lock an object, then > clone it, then do something with the clone, and finally unlock? >
r19281 fixed this without using a clone. The problem is, we can't remove any item while holding an iterator/enumerator. see this thread for details, http://archives.freenetproject.org/thread/20080414.092855.331da8b8.en.html Regards, Daniel Cheng > > On Monday 14 April 2008 00:56, j16sdiz at freenetproject.org wrote: > > Author: j16sdiz > > Date: 2008-04-13 23:56:23 +0000 (Sun, 13 Apr 2008) > > New Revision: 19279 > > > > Modified: > > trunk/freenet/src/freenet/node/LocationManager.java > > Log: > > fix concurrent modification (give NPE in gcj) > >
