RFR (S): CR 8005926: (thread) Merge ThreadLocalRandom state into java.lang.Thread

[Martin Buchholz]

On Wed, Feb 20, 2013 at 3:16 AM, Chris Hegarty <chris.hegarty at oracle.com>wrote:

> Martin, Alan,
>
> OK, I finally got back to this.
>
> I agree that Martins changes should be fine for jdk8, but removing all
> these fields may cause an issue with jdk7. Removing pads, fine. The rnd and
> initialized fields are still applicable in jdk7 ( or 7 will need changes to
> remove their dependency ).
>
> For example, a serialized TLR from jdk8 with Martins changes, deserialized
> with jdk7 would contain no field values, therefore the defaults would
> apply. Calling setSeed on this object would not throw the expected UOE.
>

My expectation was that with my changes, the default serialization would
apply, and the serialized form would contain initiatialized=1.  rnd would
not be in the serialized form, but would deserialize to 0, which is the
same as currently done.

Completely untested.

BTW, we still don't have a good story on cross-JDK serialization testing.


>
> I think minimally TLR in jdk8 needs to keep initialized, and possibly
> write some reasonable value for rnd.


Whether rnd needs a reasonable value is a reasonable question.