<Swing Dev> Rejected: [8] Review request for CR 8009880 TEST_BUG: Test java/beans/Introspector/TestTypeResolver.java should be modified again

[Sergey Malenkov]

Hi Konstantin,

There are a lot of unnecessary changes in your fix.  They make it 
difficult to analyze changes in the future.  Could you please prepare a 
new webrev with the needed changes only?

For example:
http://hg.openjdk.java.net/jdk8/jdk8/jdk/diff/a51777c9228a/test/java/beans/Introspector/TestTypeResolver.java
http://hg.openjdk.java.net/jdk8/jdk8/jdk/diff/32721a1a8da8/test/java/beans/Introspector/TestTypeResolver.java

Thanks,
SAM

On 19.03.2013 18:23, Konstantin Shefov wrote:
> Hello,
>
> Please review a fix for the issue:
>
> 8009880 TEST_BUG: Test java/beans/Introspector/TestTypeResolver.java
> should be modified again
>
> The webrev is: http://cr.openjdk.java.net/~kshefov/8009880/webrev.00/
>
> Added necessary method declarations for successful test compilation:
>
>   218         @Override
>   219         public AnnotatedType[] getAnnotatedBounds() {
>   220             return null; // not used
>   221         }
>   222
>   223         @Override
>   224         public <T extends Annotation> T[]
> getAnnotationsByType(Class<T> annotationClass) {
>   225             return null; // not used
>   226         }
>   227
>   228         @Override
>   229         public <T extends Annotation> T[]
> getDeclaredAnnotationsByType(Class<T> annotationClass) {
>   230             return null; // not used
>   231         }
>
> Thanks,
> Konstantin