RFR: 8167252: Some of Charset.availableCharsets() does not contain itself [v2]

Jaikiran Pai jpai at openjdk.org
Thu Jun 15 06:21:58 UTC 2023


On Wed, 14 Jun 2023 18:08:34 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> Adding themselves into their `contains()` method will fix it.
>
> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Refined the test

Hello Naoto, should `sun.util.PropertyResourceBundleCharset` be fixed too? Its `contains()` currently just returns `false`. Then there is `sun.font.X11Johab` in `java.desktop` module whose `contains()` appears to have a typo and is checking against an unrelated `X11GB18030_1` instance.

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

PR Comment: https://git.openjdk.org/jdk/pull/14473#issuecomment-1592434256


More information about the nio-dev mailing list