<Beans Dev> [PATCH] JDK-8030123: java/beans/Introspector/Test8027648.java fails

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Mon Apr 20 16:42:52 UTC 2015


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.



More information about the beans-dev mailing list