The fix looks good to me.
Thanks,
Alexandr.
On 8/5/2015 6:40 PM, Alexander Stepanov wrote:
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
