RFR: 8332070: Convert package.html files in `java.management` to package-info.java

[Nizar Benalla]

On Sat, 25 May 2024 06:21:51 GMT, Alan Bateman <alanb at 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: tier 1 passes on my machine, will check why a certain test fails later. As this change shouldn't cause breakage
>> 
>> 
>> ==============================
>> Test summary
>> ==============================
>>    TEST                                              TOTAL  PASS  FAIL ERROR   
>>    jtreg:test/hotspot/jtreg:tier1_serviceability       353   353     0     0   
>> ==============================
>> TEST SUCCESS
>
> src/java.management/share/classes/java/lang/management/package-info.java line 27:
> 
>> 25: 
>> 26: /**
>> 27:  *   Provides the management interfaces for monitoring and management of the
> 
> I assume you used a script to convert these, I'm just curious why puts two spaces after the *, the usual convention is one. It's not a big deal but if this is something that can be fixed in the scripts then it would avoid a future cleanup.

I think it can be fixed, I just thought one space looked cramped

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19399#discussion_r1614530454