RFR 8014076: Arrays parallel and serial sorting improvements

Mike Duigou mike.duigou at oracle.com
Fri May 10 23:46:09 UTC 2013


On May 10 2013, at 07:14 , Chris Hegarty wrote:

> Updated webrev and specdiff.
> 
> http://cr.openjdk.java.net/~chegar/8014076/ver.01/specdiff/java/util/Arrays.html

Docs changes look fine to me.

>  http://cr.openjdk.java.net/~chegar/8014076/ver.01/webrev/

ArraysParallelSortHelpers::

- It's strange to see serialversionid defined for classes that will never be serialized.

ParallelSorting::

- Arrays.MIN_ARRAY_SORT_GRAN does this reference still work?

Otherwise changes look good to me. (Depending very much on regression testing to ensure correctness of the algorithm).

> 
> I incorporated the feedback so far, and reverted the change to make MIN_ARRAY_SORT_GRAN public ( there doesn't appear to be enough justification for this ).
> 
> -Chris.




More information about the core-libs-dev mailing list