<Swing Dev> [13] JDK-8216008: -Djavax.accessibility.assistive_technologies empty list leads to crash

[Sergey Bylokhov]

Hi, Shashi.

Should we trim all whitespaces from the "atNames"?
Otherwise the "atNames", which contains only one whitespace, will trigger the same error.

On 09/01/2019 00:20, Shashidhara Veerabhadraiah wrote:
> Hi Sergey, Thanks for pointing that out. Here is the new Webrev:
> 
> http://cr.openjdk.java.net/~sveerabhadra/8216008/webrev.01/
> 
> Thanks and regards,
> Shashi
> 
> -----Original Message-----
> From: Sergey Bylokhov
> Sent: Friday, January 4, 2019 3:48 AM
> To: Shashidhara Veerabhadraiah <shashidhara.veerabhadraiah at oracle.com>; awt-dev at openjdk.java.net; swing-dev at openjdk.java.net
> Subject: Re: <Swing Dev> [13] JDK-8216008: -Djavax.accessibility.assistive_technologies empty list leads to crash
> 
> Hi, Shashi.
> I think you can update an existing test:
> open/test/jdk/javax/accessibility/AccessibilityProvider/basic.sh
> 
> On 03/01/2019 01:36, Shashidhara Veerabhadraiah wrote:
>> Hi All, Please review a fix for the below bug.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8216008
>>
>> Webrev: http://cr.openjdk.java.net/~sveerabhadra/8216008/webrev.00/
>>
>> Since the mentioned list of assistive technologies for the system property(javax.accessibility.assistive_technologies) is loaded by jvm am unable to write a test case for this bug. The test file attached with the bug can be used for testing this fix.
>>
>> This fix removes the error(class not found error) by passing an empty list to system variable javax.accessibility.assistive_technologies and does not load any assistive technologies(as the list is empty). Currently empty list produce a crash which is not required as there was no class name mentioned in the list. Please read the bug comments for more information.
>>
>> Thanks and regards,
>>
>> Shashi
>>
> 
> 


-- 
Best regards, Sergey.