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 nio-dev
mailing list