Hi, Peter, Not a reviewer, and have barely any time to have the careful review, so just a few nitpicks below:
On 07/09/2013 12:54 AM, Peter Levart wrote: > http://cr.openjdk.java.net/~plevart/jdk8-tl/AnnotationType/webrev.05/ AnnotationParser.java: - Do you want make contains() generic? static <T> boolean contains(T[] array, T element); - Also, you might probably put the null-check in contains(), and simplify the usage - parseAnnotation2 seems a bad name; it seems just overloading parseAnnotation is good. -Aleksey.