RFR : 7144488 StackOverflowError occurres on list via Collections.synchronizedList(List)

David Holmes david.holmes at oracle.com
Thu Feb 23 21:59:07 UTC 2012


On 24/02/2012 7:55 AM, Jason Mehrens wrote:
> David,
>
> For completeness, you might want to link this bug to bug id 6360946
> "(coll) SetFromMap.equals should perform identity check". Most of the
> wrapper classes were fixed to include an identity check for that bug.
> Digging up some old messages from December 2005, the synchXXX wrappers
> were not updated because we were only trying to fix the reflexive test
> for equals method in the face of concurrent modification. For the
> synchXXX wrappers it doesn't apply but, we missed the need for it when
> used in methods like remove.

Thanks for the info Jason, I've added the cross-reference. Not bad if 
this issue only crops up every 6 years or so :)

Cheers,
David

> Jason
>
>  > That said, an equals method that checks for o==this is pretty normal
>  > (and recommended by Effective Java Item 7) and we at least now have
>  > consistency amongst the wrapper classes. I'll add a note to the CR to
>  > that effect.
>  >
>  > Michael: good to go (if it hasn't already)
>  >
>  > Thanks,
>  > David
>  >
>



More information about the core-libs-dev mailing list