[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)); } Paul.
