On Wed, 14 Jun 2023 18:08:34 GMT, Naoto Sato <na...@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

Thanks for the review, Jai.
As to `PropertyResourceBundleCharset`, yes it is an internal implementation as 
Alan mentioned, to auto detect UTF-8/ISO-8859-1, so even though `contains()` 
does not include itself, it is fine.

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

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

Reply via email to