On Tuesday 15 April 2008 01:08, you wrote:
> On Tue, Apr 15, 2008 at 5:18 AM, Matthew Toseland
> <[EMAIL PROTECTED]> 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, [EMAIL PROTECTED] 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
> [email protected]
> http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl
> 
> 

Attachment: pgpxY0eKL4Woe.pgp
Description: PGP signature

_______________________________________________
Devl mailing list
[email protected]
http://emu.freenetproject.org/cgi-bin/mailman/listinfo/devl

Reply via email to