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

[Alexander Scherbatiy]

   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.
>>
>
>