<i18n dev> RFR: 8363925: Remove unused sun.nio.cs.ArrayEncoder::encode [v4]

Chen Liang liach at openjdk.org
Tue Oct 28 14:03:24 UTC 2025


On Tue, 28 Oct 2025 09:42:17 GMT, Volkan Yazici <vyazici at openjdk.org> wrote:

>> [JDK-6636319] introduced `ArrayEncoder`, composed of _only_ `encode` method. Later on [JDK-8054307] (JEP 254: Compact Strings)  removed all its usages. This method can safely be removed, hence this PR. Confirmed that `tier1-2` tests are clear on several platforms after changes.
>> 
>> [JDK-6636319]: https://bugs.openjdk.org/browse/JDK-6636319
>> [JDK-8054307]: https://bugs.openjdk.org/browse/JDK-8054307
>
> Volkan Yazici has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   Remove `ArrayEncoder` defaults and add `dp` parameter

Thank you for this great cleanup. Looks much more maintainable and straightforward. We can re-implement for CESU_8 if there is request later.

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

Marked as reviewed by liach (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27999#pullrequestreview-3388946460


More information about the i18n-dev mailing list