On Fri, 24 May 2024 18:11:18 GMT, Nizar Benalla <nbena...@openjdk.org> wrote:
> 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
This pull request has now been integrated.
Changeset: c7d2841f
Author: Nizar Benalla <nbena...@openjdk.org>
Committer: Jaikiran Pai <j...@openjdk.org>
URL:
https://git.openjdk.org/jdk/commit/c7d2841fb4ac97c0edec175cf37abd90167ea56e
Stats: 3089 lines in 18 files changed: 1524 ins; 1565 del; 0 mod
8332070: Convert package.html files in `java.management` to package-info.java
Reviewed-by: alanb
-------------
PR: https://git.openjdk.org/jdk/pull/19399
