The fix is pushed to JDK 9:
http://hg.openjdk.java.net/jdk9/client/jdk/rev/be4d1f1a6d53
The original webrev:
http://cr.openjdk.java.net/~alexsch/nicholas-cull/8030123/webrev.00
Thanks,
Alexandr.
On 4/20/2015 7:42 PM, Sergey Bylokhov wrote:
Hello.
The looks good to me too.
On 25.03.15 18:21, Alexander Scherbatiy wrote:
The fix looks good.
Sorry for the long delay.
Thanks,
Alexandr.
On 7/30/2014 12:49 PM, Nicholas Cull wrote:
Hi there,
I've attached a patch to address JDK-8030123. This is the bug where
one of the existing beans tests succeeds on Windows, but fails on
Ubuntu/Debian Linux.
The approach here is to change the merging code in the
IndexedPropertyDescriptor(PD, PD) constructor to be similar to the
same code in the PropertyDescriptor(PD, PD) class. In particular,
use the isAssignable() method in PropertyDescriptor to determine
which method object should be used in the final merge.
I've regression tested against the existing beans tests, on Windows
and Debian, and confirm the patch fixes the regression on Debian,
while continuing to work correctly on Windows.
Hopefully this is useful, or at least a good starting point.
Regards,
Nicholas.
