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.veerabhadra...@oracle.com>; awt-dev@openjdk.java.net; swing-...@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.