> The argument of the `CertAttrSet::encode` and `DerEncoder::derEncode` > interface methods are modified from `OutputStream` to `DerOutputStream`. All > implementations are modified the same way. > > `OutputStream` is still used by > `sun.security.x509.Extension::encode(OutputStream os)` because it's inherited > from `java.security.cert.Extension`. The method is now marked final to avoid > accidental override. > > In `CertificateExtensions` and `CRLExtensions`, only > `Extension::encode(DerOutputStream out)` is called. It used to call > `CertAttrSet::encode` for a known extension and > `Extension::encode(DerOutputStream out)` for an unknown one. This makes sure > the overridden `encode` methods in known extensions are always called. Now > that they have the same argument, there is no need for this check.
Weijun Wang has updated the pull request incrementally with one additional commit since the last revision: same for DerEncoder::derEncode only in patch2: unchanged: ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10906/files - new: https://git.openjdk.org/jdk/pull/10906/files/05f129b2..da7221f2 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10906&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10906&range=01-02 Stats: 50 lines in 13 files changed: 1 ins; 23 del; 26 mod Patch: https://git.openjdk.org/jdk/pull/10906.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10906/head:pull/10906 PR: https://git.openjdk.org/jdk/pull/10906