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

[Matthew Toseland]

On Tuesday 15 April 2008 01:08, you wrote:
> 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.

Ok. PS why iterate over a vector? Surely it is more efficient to just use a 
loop and get()?
> 
> 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)
> >  >
> _______________________________________________
> Devl mailing list
> Devl at freenetproject.org
> http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl
> 
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://emu.freenetproject.org/pipermail/devl/attachments/20080415/53649f87/attachment.pgp