Looks good. On Aug 21 2013, at 16:19 , Henry Jen wrote:
> On 08/20/2013 02:21 AM, Paul Sandoz wrote: >> [resending unsigned, sorry if a dup arrives later on] >> >> On Aug 19, 2013, at 9:18 PM, Henry Jen <[email protected]> wrote: >> >>> Hi, >>> >>> Please review the webrev at >>> http://cr.openjdk.java.net/~henryjen/tl/8023275/0/webrev/ >>> >>> The patch adds override on default methods for a couple wrapping classed >>> and delegate those to underlying class. >>> >>> There is a minor revise on synchronizedCollection javadoc to cover Stream. >>> >> >> Looks good. >> >> >>> A sanity check on wrapper classes to ensure default methods are override. >>> >> >> You might want to additionally use an ArrayList instance for >> unmodifiableList/synchronizedList/checkedList, so as to check both unmod >> list impls: >> >> public static <T> List<T> unmodifiableList(List<? extends T> list) { >> return (list instanceof RandomAccess ? >> new UnmodifiableRandomAccessList<>(list) : >> new UnmodifiableList<>(list)); >> } >> > > Good point, I also added Navigable into the test. Thank you for adding these. Mike > http://cr.openjdk.java.net/~henryjen/tl/8023275/2/webrev/ > > Cheers, > Henry
