On 2/27/2013 9:57 PM, Sergey Malenkov wrote:
Hello,

> Could you please review the following fix:
> bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8000183
> fix: http://cr.openjdk.java.net/~malenkov/8005239.0/

Please use the following fix instead:
fix: http://cr.openjdk.java.net/~malenkov/8000183.1/

     The fix looks good for me.

I tried the test test/java/beans/Introspector/TestTypeResolver.java and it fails with a compilation error: TestTypeResolver.java:204: error: ClassTypeVariable is not abstract and does not override abstract method getAnnotatedBounds() in TypeVariable
   I think it can be fixed as a separated issue.

   Thanks,
   Alexandr.


This fix reverts the changes from the 7192955 fix.

> This is a regression after the following fix
> bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7192955
> fix: http://cr.openjdk.java.net/~malenkov/7192955.1/

The test still works as expected because
the 7193977 fix resolves the 7192955 issue as well.
I've added some things to test the 8000183 too.

bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7193977
fix: http://cr.openjdk.java.net/~malenkov/7193977.0/

> Note, it must be backported to JDK 7.

Thanks,
SAM

Reply via email to