On Thu, 10 Nov 2022 01:02:00 GMT, pandaapo wrote:
> The cache named `signerToCodeSource` in `JarVerifier` is never used now.
This pull request has now been integrated.
Changeset: f0e99c63
Author: pandaapo <1052156...@qq.com>
Committer: Weijun Wang
URL:
https://git.openjd
On Tue, 22 Nov 2022 01:44:29 GMT, Weijun Wang wrote:
> Everything looks fine now. Thanks so much for the cleanup. It's always nice
> to see unused lines removed.
Thank you for your review.
-
PR: https://git.openjdk.org/jdk/pull/11072
> The cache named `signerToCodeSource` in `JarVerifier` is never used now.
pandaapo has updated the pull request incrementally with one additional commit
since the last revision:
Modify as reviews.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/11072/files
- new: ht
On Sun, 20 Nov 2022 15:37:24 GMT, Weijun Wang wrote:
> `codomain`, `VerifierCodeSource`, `isSigningRelated`, and `getUnsignedCS`
> also used nowhere.
Thanks. I think your `codomain` must mean `csdomain`.
-
PR: https://git.openjdk.org/jdk/pull/11072
> The cache named `signerToCodeSource` in `JarVerifier` is never used now.
pandaapo has updated the pull request incrementally with one additional commit
since the last revision:
Modify as review and update copyright.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/11
The cache named `signerToCodeSource` is never used now.
-
Commit messages:
- Make signerToCodeSource cache work.
Changes: https://git.openjdk.org/jdk/pull/11072/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=11072&range=00
Issue: https://bugs.openjdk.org/browse/JDK-829673
