On Jan 31 2014, at 11:50 , Martin Buchholz <[email protected]> wrote:
> Jason, > Thanks for pointing that out. I'm sure I have seen those bugs before (when > I owned them!) but had suppressed the memory. I'm currently the assignee for this bug. > I probably didn't try fixing them because there is no clean way out, and I > was afraid of getting bogged down in compatibility hell for what is a > non-issue for real-world users. Indeed. That's exactly why they still haven't been addressed. Suggestions are, of course, always welcome. Mike > > On Fri, Jan 31, 2014 at 11:43 AM, Jason Mehrens > <[email protected]>wrote: > >> 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: [email protected] >>> To: [email protected] >>> CC: [email protected] >> >>> >>> 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 <[email protected] >>> 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. >>>> >>
