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.
--
Best regards, Sergey.
