<Swing Dev> [13] Review Request: 8216008 -Djavax.accessibility.assistive_technologies empty list leads to exception

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Sat Apr 20 23:56:29 UTC 2019


For the record:

During CSR review the text was slightly modified:
"If the list ....... is the empty string or contains only white space characters then the method returns immediately"

Replaced by this:
"If the list ....... is the empty string or contains only white space characters it is ignored."

http://cr.openjdk.java.net/~serb/8216008/webrev.01

On 15/04/2019 11:01, Phil Race wrote:
> +1
> 
> -phil
> 
> On 4/14/19 8:34 PM, Sergey Bylokhov wrote:
>> Hello.
>> Please review the fix for JDK 13.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8216008
>> Fix: http://cr.openjdk.java.net/~serb/8216008/webrev.00
>>
>> The previous discussion of this fix can be found here[1].
>>
>> Changes in this version:
>>  - The check "atNames.strip().isEmpty()" was replaced by the "atNames.isBlank()".
>>  - As suggested in the CSR the link to the "Character#isWhitespace(int)" was added to the text about white spaces.
>>
>> [1] http://mail.openjdk.java.net/pipermail/swing-dev/2019-January/009260.html
>>
> 


-- 
Best regards, Sergey.


More information about the swing-dev mailing list