On 14 Jan 2013, at 22:33, Aleksey Shipilev <aleksey.shipi...@oracle.com> wrote:
> On 01/15/2013 02:17 AM, Alan Bateman wrote: >> The other thing I notice is that the serialized form of the original >> ThreadLocalRandom included the padding fields, I guess they should have >> been transient in the original implementation. > > Damn. These *are* the part of TLR serialized form :( > http://docs.oracle.com/javase/7/docs/api/serialized-form.html#java.util.concurrent.ThreadLocalRandom > > Also, the class changes probably screwed up the SUID. I think we need to > get the padding back (which should not be the problem since TLR is now a > singleton, and add the SUID with the previous value. Or serialPersistentFields? I was thinking of this anyway, so we could get rid of rnd. -Chris > > -Aleksey.