Integrated: 8331855: Convert jdk.jdeps jdeprscan and jdeps to use the Classfile API

Chen Liang liach at openjdk.org
Fri May 17 12:29:09 UTC 2024


On Sat, 11 May 2024 19:35:33 GMT, Chen Liang <liach at openjdk.org> wrote:

> Summary of the changes:
>  - Moved `com.sun.tools.classfile.Dependency` and `Dependencies` to jdeps; they are exclusively used by jdeps in sources, and they are not used in any tests too. This will ease the removal of `com.sun.tools.classfile` later.
>  - A few visitor patterns have been rewritten with pattern matching, notably in:
>    - `CPEntries`/`CPSelector` (removed)
>    - `Dependencies.BasicDependencyFinder.Visitor` has been rewritten to use pattern matching to capture dependencies.
>  - `MethodSig` and its tests have been removed in favor of `MethodTypeDesc`.
>  - Otherwise, the changes are mostly mechanical replacements.
> 
> All tests in `test/langtools/tools/jdeprscan` and `test/langtools/tools/jdeps` pass.

This pull request has now been integrated.

Changeset: d4c2edf2
Author:    Chen Liang <liach at openjdk.org>
Committer: Adam Sotona <asotona at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/d4c2edf2c91a790874c80f1a7bea5bfd4f438bde
Stats:     2156 lines in 20 files changed: 744 ins; 1197 del; 215 mod

8331855: Convert jdk.jdeps jdeprscan and jdeps to use the Classfile API

Reviewed-by: asotona

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

PR: https://git.openjdk.org/jdk/pull/19193


More information about the compiler-dev mailing list