Hey,

> "Bordet, Simone" wrote:
> > it should be:
> > 
> > public boolean isActive(Object id)
> > {
> >         // Check whether an object with the given id is 
> available in the
> > cache
> >         boolean isActive;
> >         synchronized (getCacheLock())
> >         {
> >                 isActive = getCache().peek(id) != null;
> >         }
> >         return isActive;
> > }
> 
> True, but this check is so fragile anyway 

WDYM ? If not sync'ed this may not tell you if id is really in the cache.

> (regardless of how one does
> it) that I think unsynched is ok. 
> 
> YMMV.

Yeah, MMMV, but unsync'ed isn't OK...

Simon

Reply via email to