RFR: 8359760: Remove the jdk.jsobject module
Kevin Rushforth
kcr at openjdk.org
Tue Jul 22 19:43:55 UTC 2025
On Tue, 22 Jul 2025 18:48:59 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> I wondered about that, and originally was going to remove it, but then noticed that the test in question parses the JDK 17 API docs (which does have the `jdk.jsobject` module).
>>
>> https://github.com/openjdk/jdk/blob/e70c702d6f864b494dc7df3eea4f1a317a3bc981/test/langtools/jdk/javadoc/doclet/testRecordTypes/TestRecordTypes.java#L55-56
>>
>>
>> private static final String externalDocs =
>> "https://docs.oracle.com/en/java/javase/17/docs/api";
>>
>>
>> I suspect that the test will run just fine if I remove it. I don't mind either way. What would you recommend?
>
> I ran the test and then removed those lines and the test still passed.
> I'd remove them to avoiding leaving a (little) mess behind.
As expected, the above test passes fine if I remove the jsobject entries from `test/langtools/jdk/javadoc/doclet/testRecordTypes/jdk17/element-list`. I'll go ahead and remove them, since it seems like good cleanup.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26393#discussion_r2223678605
More information about the javadoc-dev
mailing list