On 12/05/2017 7:55 AM, Mandy Chung wrote:
+1 (dropping the null check as Claes suggests).
Ok - dropping null check. :)
Thanks,
David
Mandy
On May 11, 2017, at 2:01 PM, Claes Redestad <claes.redes...@oracle.com> wrote:
The null check is redundant, so the code could be simplified a bit.
Reviewed regardless!
/Claes
On 2017-05-11 22:46, David Holmes wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8179515
webrev: http://cr.openjdk.java.net/~dholmes/8179515/webrev/
This is a fix contributed by Doug Lea, from a suggestion made by Alan Bateman.
While I'm handling the fix process for Doug, it isn't my fix so I am also a
Reviewer of this fix. I expect Martin to also review.
The fix is quite simple - break the dependency between ThreadLocalRandom and
the security framework by replacing a doPrivileged block that reads a system
property with a call to the JDK internal VM.getSavedProperty(). This allows TLR
to be used transitively from the security framework without creating a cyclic
initialization dependency which leads to failure to initialize.
This fixed is approved for 9.
Thanks,
David
