On Tue, 28 Oct 2025 09:42:17 GMT, Volkan Yazici <[email protected]> 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
