Post-merge tweaks to #85, update release notes

Author: cowtowncoder

release-notes/CREDITS

@@ -131,3 +131,7 @@ Paul Galbraith (pgalbraith@github)
   * Contributed #73: Add `Generators.defaultTimeBasedGenerator()` to use "default"
     interface  address for time/based UUIDs
   [4.2.0]
+
+Maia Everett (Maia-Everett@github)
+  * Contributed #85: Fix `LazyRandom` for native code generation tools
+  [5.0.0]

release-notes/VERSION

@@ -7,6 +7,8 @@ Releases
 5.0.0 (not yet released)
 
 #53: Increase JDK baseline to JDK 8
+#85: Fix `LazyRandom` for native code generation tools
+ (contributed by @Maia-Everett)
 
 4.3.0 (12-Sep-2023)
 

src/main/java/com/fasterxml/uuid/impl/LazyRandom.java

@@ -6,26 +6,20 @@
  * Trivial helper class that uses class loading as synchronization
  * mechanism for lazy instantiation of the shared secure random
  * instance.
+ *<p>
+ * Since 5.0 has been lazily created to avoid issues with native-generation
+ * tools like Graal.
  */
 public final class LazyRandom
 {
     private static final Object lock = new Object();
     private static volatile SecureRandom shared;
 
     public static SecureRandom sharedSecureRandom() {
-        // Double check lazy initialization idiom (Effective Java 3rd edition item 11.6)
-        // Use so that native code generation tools do not detect a SecureRandom instance in a static final field.
-        SecureRandom result = shared;
-
-        if (result != null) {
-            return result;
-        }
-
         synchronized (lock) {
-            result = shared;
-
+            SecureRandom result = shared;
             if (result == null) {
-                result = shared = new SecureRandom();
+                shared = result = new SecureRandom();
             }
 
             return result;