Alexander,

Sorry, could you please look at the updated version?
http://cr.openjdk.java.net/~avstepan/8132566/webrev.01/

A new test was added to cover JDK-8132669 (and now is failing because of it): http://cr.openjdk.java.net/~avstepan/8132566/webrev.01/test/java/beans/Introspector/8132566/OverrideUserDefPropertyInfoTest.java.html

It was sent to the mailing list a couple of hours ago...

Thanks,
Alexander


On 8/5/2015 6:27 PM, Alexander Scherbatiy wrote:

  The fix looks good to me.

  Thanks,
  Alexandr.

On 8/4/2015 3:49 PM, Alexander Stepanov wrote:
Hello,

Could you please review the following fix:
http://cr.openjdk.java.net/~avstepan/8132566/webrev.00/
for
https://bugs.openjdk.java.net/browse/JDK-8132566

At the moment 'OverridePropertyInfoTest.java' will fail because of
https://bugs.openjdk.java.net/browse/JDK-8132565
(and, additionally, both of them, - because of https://bugs.openjdk.java.net/browse/JDK-8131939).

If something like, e.g., (ClassProperty.isExpert() || BaseClassProperty.isExpert()) is indeed should be returned when requesting ClassProperty.isExpert(), then please let me know.

Thanks,
Alexander


Reply via email to