On 16/09/2008, Jörg Schaible <[EMAIL PROTECTED]> wrote: > sebb wrote: > > On 16/09/2008, Simon Kitching <[EMAIL PROTECTED]> wrote: > > [snip] > > >> It's not quite clear to me why a threadlocal is being used here. I'm > >> guessing it is for performance to avoid recreating the HashSet > >> object. > > > > I don't know for sure, but that is probably the case. > > > >> Or is > >> it to "tunnel" the registry set between cooperating classes? > > > > Not at present, although the registy access methods are > > package protected. > > > > That may be a mistake... > > > My guess: It's used in case of a circular object graph ... >
Yes, they are used in the enclosing class for just that purpose. But as far as I can see there is no need for the methods to be anything other than private - and the contents of the HashMap only contain anything during the calculation of the hashcode, so it's not obvious how the methods could be called anyway. I meant it was probably a mistake for the methods to be package protected. > - Jörg > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
