Code review request for 6990094 "ObjectInputStream cloneArray doesn't handle short[]"

Rémi Forax forax at univ-mlv.fr
Mon Dec 6 08:10:23 UTC 2010


  Le 06/12/2010 08:35, Joe Darcy a écrit :
> Hi Peter.

[...]

> Off-list, Alan found the a related closed test and Stuart and I have 
> developed an explicit test that tickles this bug:
>
>    http://cr.openjdk.java.net/~darcy/6990094.1/
>
> I plan to shortly push the fix with the test, although the test might 
> be renamed or tweaked first,
>
> Thanks,
>
> -Joe

Hi Joe,
In the test, I don't see why the replacement field has to be static in 
Resolver.
In my opinion, a private final field is sufficient.

Rémi



More information about the core-libs-dev mailing list