<i18n dev> RFR: 8167252: Some of Charset.availableCharsets() does not contain itself

Alan Bateman alanb at openjdk.org
Wed Jun 14 17:22:58 UTC 2023


On Wed, 14 Jun 2023 17:14:08 GMT, Lance Andersen <lancea at openjdk.org> wrote:

>> Adding themselves into their `contains()` method will fix it.
>
> test/jdk/java/nio/charset/Charset/Contains.java line 108:
> 
>> 106:     }
>> 107: 
>> 108:     static void selfContainmentTest() {
> 
> Be nice to add a comment regarding the purpose of the test

The spec is that Charset::contains return true for self so maybe it could be renamed to containsSelfTest, might help with the understanding the purpose (along with a comment)

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14473#discussion_r1229943556


More information about the i18n-dev mailing list