Looks good to me. Cheers, Henry
On Aug 13, 2013, at 6:46 AM, Paul Sandoz <paul.san...@oracle.com> wrote: > Hi, > > This patch fixes a bug in the data provider of the SpliteratorCollisions.java > test, which was attempting to add an element to an unmodifable map. > > Interestingly although this caused the data provider to barf, and therefore > tests using such a provider were not run, it did not result in jtreg > reporting the test-infrastructure failure. > > Paul. > > diff -r 35483a6aa27e test/java/util/Spliterator/SpliteratorCollisions.java > --- a/test/java/util/Spliterator/SpliteratorCollisions.java Tue Aug 13 > 15:24:53 2013 +0200 > +++ b/test/java/util/Spliterator/SpliteratorCollisions.java Tue Aug 13 > 15:41:39 2013 +0200 > @@ -148,7 +148,6 @@ > List<Object[]> data = new ArrayList<>(); > for (int size : SIZES) { > List<HashableInteger> exp = listIntRange(size, true); > - exp.add(0, null); > SpliteratorDataBuilder<HashableInteger> db = new > SpliteratorDataBuilder<>(data, exp); > > // Maps