On Mon, 11 Mar 2024 09:58:42 GMT, Eirik Bjørsnøs <eir...@openjdk.org> wrote:
> Please review this cleanup PR which removes the protected, deprecated field > `X509Key.key` in the internal package `sun.security.x509`. > > This field and the associated field `unusedBits` have been marked > `@Deprecated` since the initial load. The recommended replacement is to use > the `BitArray` representation, which does not require keys to be byte-aligned. > > Two use sites of the `key` field were found in subclasses: > > * The `ECPublicKeyImpl` constructor was updated to use a `BitArray` instead. > * `ECPublicKeyImpl.parseKeyBits()` was updated to call > `getKey().toByteArray()` to produce the byte array. > > Somewhat unrelated, the method `ECPublicKeyImpl.getEncodedPublicValue()` was > deemed unused and has been removed. > > Testing and verification: This PR does not update any tests. GHA runs green. This pull request has now been integrated. Changeset: 4eefda91 Author: Eirik Bjørsnøs <eir...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/4eefda91021e07768ace03725c88a56af33ca344 Stats: 55 lines in 2 files changed: 2 ins; 48 del; 5 mod 8327779: Remove deprecated internal field sun.security.x509.X509Key.key Reviewed-by: mullan ------------- PR: https://git.openjdk.org/jdk/pull/18185
