RFR: 8332070: Convert package.html files in `java.management` to package-info.java [v3]
Nizar Benalla
nbenalla at openjdk.org
Wed Jun 5 01:21:36 UTC 2024
> This is a simple noreg cleanup. The motivation was that I noticed javac doesn't recognise package.html files well.
>
> Some of the contents of the `package.html` files (and code in the package) may be outdated, but I think it is out of scope for this PR.
>
> I have also changed three `{@link }` usages with the `href` that would have been in the generated HTML.
> Because referencing an element from an other module wouldn't work even when using the `module/package.class#member`, if the `module-info.java` file does not have "require `<name>`".
>
> I am referring to line 69 in
> `src/java.management/share/classes/javax/management/monitor/package-info.java`
> and lines 90 and 120 in
> `src/java.management/share/classes/javax/management/remote/package-info.java`
>
> Adding 2 dependencies just for 3 links didn't seem right.
>
> edit: passes all tests, the failing test is a github actions issue
Nizar Benalla has updated the pull request incrementally with one additional commit since the last revision:
remove double spacing
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19399/files
- new: https://git.openjdk.org/jdk/pull/19399/files/c5d3147a..963ff90f
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19399&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19399&range=01-02
Stats: 211 lines in 1 file changed: 0 ins; 0 del; 211 mod
Patch: https://git.openjdk.org/jdk/pull/19399.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19399/head:pull/19399
PR: https://git.openjdk.org/jdk/pull/19399
More information about the serviceability-dev
mailing list