RFR: 8340404: CharsetProvider specification updates [v6]

[Justin Lu]

On Sun, 22 Sep 2024 07:48:03 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review: spec correction - system class loader used, not context
>
> src/java.base/share/classes/java/nio/charset/spi/CharsetProvider.java line 38:
> 
>> 36:  * zero-argument constructor and some number of associated {@code Charset}
>> 37:  * implementation classes.  Charset providers are deployed by adding them to either
>> 38:  * the application module path or the application class path. In order to be looked
> 
> The latest update to the class description looks quite good. I think this sentence can be simplified down to "Charset provider are deployed on the application module path ...".

Fixed in https://github.com/openjdk/jdk/pull/21076/commits/08e8161c3d9a6c2efca7b5f578a72060537495e9

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21076#discussion_r1771819878