Looks good to me. There are some additional optimization opportunities but they can certainly wait.
Mike On Mar 14 2014, at 05:04 , Paul Sandoz <paul.san...@oracle.com> wrote: > Hi, > > This patch overrides some default methods with more optimal ones for the > Arrays.asList implementation: > > > http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8037106-arrays.asList.forEach/webrev/ > > It required some surgical work on tests to shove in the Arrays.asList test > case, since it is a factory method and does not support structural > modification. > > There are of course other optimizations that could apply to Arrays.asList, > namely that of sub-list, but i have left that as another future possible > exercise (plus further work could be done to Abstract/List, including > providing a Spliterator for List & RandomAccess). > > Paul.