On Thursday 13 November 2008 18:24:30 Matthew Toseland wrote:
> On Thursday 13 November 2008 16:45, xor wrote:
> > > > +               try {
> > > > +                       Identity id = Identity.getByURI(db,
> > >
> > > state.getURI());
> > >
> > > > +                       identities.remove(id);
> > >
> > > You *really* should check the return value here. I've had
> > > wierd bugs with this sort of thing.
> > >
> > > > +               } catch(UnknownIdentityException ex) {}
> > > > +
> > > >         }
> >
> > You mean db4o could return a different ientity than the requested
> > one or what?!?
>
> No... well hopefully not. I mean state.getURI() might return something
> wrong, identities.remove() might fail because of some wierd bug ... just
> check the retval and log an ERROR if it's not successfully removed.

This is fixed in the recent code.

Attachment: signature.asc
Description: This is a digitally signed message part.

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

Reply via email to