[freenet-dev] [freenet-cvs] r19279 - trunk/freenet/src/freenet/node

[Daniel Cheng]

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)
>  >