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