Re: RFR: 8280915: Better parallelization for AbstractSpliterator and IteratorSpliterator when size is unknown [v2]

2022-01-29 Thread Tagir F . Valeev
> See the bug description for details. > > I propose a simple solution. Let's allow ArraySpliterator to be non-SIZED and > report artificial estimatedSize(), much bigger than the real one. This will > allow AbstractSpliterator and IteratorSpliterator to produce prefix whose > size is comparable

Re: RFR: 8279598: Provide adapter from RandomGenerator to Random [v3]

2022-01-29 Thread Jim Laskey
On Sat, 29 Jan 2022 16:13:46 GMT, Yasser Bazzi 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 me

Re: RFR: 8279598: Provide adapter from RandomGenerator to Random [v2]

2022-01-29 Thread Stuart Marks
On Sat, 29 Jan 2022 16:10:28 GMT, Yasser Bazzi wrote: > Oh sorry about that, my IDE probably did the changes and it went unnoticed by > me, should i revert the offending commit and recommit only the appropriate > changes? Just add a new commit that undoes the unwanted changes. Please don't reb

RFR: 8280915: Better parallelization for AbstractSpliterator and IteratorSpliterator when size is unknown

2022-01-29 Thread Tagir F . Valeev
See the bug description for details. I propose a simple solution. Let's allow ArraySpliterator to be non-SIZED and report artificial estimatedSize(), much bigger than the real one. This will allow AbstractSpliterator and IteratorSpliterator to produce prefix whose size is comparable to Long.MAX

Re: RFR: 8279598: Provide adapter from RandomGenerator to Random [v2]

2022-01-29 Thread Yasser Bazzi
On Fri, 28 Jan 2022 16:01:56 GMT, Roger Riggs wrote: > (For future reference: As tempting as it is to make other small improvements > to the code such as missing but optional @ Overrides, it obscures the changes > that are the subject of the PR.). Oh sorry about that, my IDE probably did the c

Re: RFR: 8279598: Provide adapter from RandomGenerator to Random [v3]

2022-01-29 Thread Yasser Bazzi
> 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