On Sat, 8 Jan 2022 21:00:37 GMT, Yasser Bazzi <d...@openjdk.java.net> wrote:

> 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

This pull request has now been integrated.

Changeset: df8c2be5
Author:    Yasser Bazzi Bordonal <yasser...@gmail.com>
Committer: Stuart Marks <sma...@openjdk.org>
URL:       
https://git.openjdk.java.net/jdk/commit/df8c2be5fedd3a066b1d7371a3e3cbb7191977b0
Stats:     250 lines in 3 files changed: 225 ins; 3 del; 22 mod

8279598: Provide adapter from RandomGenerator to Random

Reviewed-by: smarks, darcy

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

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

Reply via email to