> Hi, could i get a review on this implementation proposed by Stuart Marks, i 
> decided to use the 
> https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/util/random/RandomGenerator.html
>  interface to create the default method `asRandom()` that wraps around the 
> newer algorithms to be used on classes that do not accept the new interface.
> 
> Some things to note as proposed by the bug report, the protected method 
> next(int bits) is not overrided and setSeed() method if left blank up to 
> discussion on what to do with it.
> 
> Small test done on 
> https://gist.github.com/YShow/da678561419cda8e32fccf3a27a649d4

Yasser Bazzi has updated the pull request incrementally with three additional 
commits since the last revision:

 - Change variable name
 - Do not allow serialization
 - Changes proposed by Stuart Marks, ExE Boss and liach

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7001/files
  - new: https://git.openjdk.java.net/jdk/pull/7001/files/0e48edf6..5c5bd873

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7001&range=12
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7001&range=11-12

  Stats: 43 lines in 1 file changed: 23 ins; 2 del; 18 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7001.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7001/head:pull/7001

PR: https://git.openjdk.java.net/jdk/pull/7001

Reply via email to