Martin, 
 Unifying the List testing code might be kind of tricky with 
https://bugs.openjdk.java.net/browse/JDK-4506427 as unresolved. 
http://docs.oracle.com/javase/7/docs/api/java/util/List.html
http://docs.oracle.com/javase/7/docs/api/java/util/AbstractList.html The patch 
looks good though. Cheers, Jason
 
> Date: Fri, 31 Jan 2014 10:07:31 -0800
> Subject: Re: RFR [9] 8011645: CopyOnWriteArrayList.COWSubList.subList does 
> not        validate range properly
> From: marti...@google.com
> To: chris.hega...@oracle.com
> CC: core-libs-dev@openjdk.java.net
> 
> The jtreg test is fine, but:
> 
> s/IOBE/IOOBE/
> 
> When I created MOAT.java many years ago, I intended tests such as this to
> get added to that, so that all of the List implementations could share the
> same test code.  jsr166 does not have the same concern, since it only has
> one List implementation at the moment.  Today, there are other choices,
> like sharing test infrastructure with Guava e.g. ListTestSuiteBuilder.
>  More generally, openjdk core libraries can benefit from all the great
> testing work that guava folk have done.
> 
> 
> On Fri, Jan 31, 2014 at 8:23 AM, Chris Hegarty 
> <chris.hega...@oracle.com>wrote:
> 
> > Trivial change to CopyOnWriteArrayList.COWSubList.subList to catch the
> > case where the fromIndex is greater that the toIndex.
> >
> > http://cr.openjdk.java.net/~chegar/8011645/webrev.00/webrev/
> >
> > -Chris.
> >

                                          

Reply via email to