RFR: Fix javadoc generation and add check to testing suite [v2]

James Yuzawa duke at openjdk.org
Thu Mar 28 12:07:56 UTC 2024


On Thu, 28 Mar 2024 02:59:52 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> James Yuzawa has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix javadoc generation and add check to testing suite
>>   
>>   Motivation: There is bad javadoc being generated. This fixes that and also adds a check to the testing suite to make sure it does not happen again.
>
> test/lib/testlib/TestUtils.java line 89:
> 
>> 87:             System.err.println("javadoc sources @ " + sourcePath.toAbsolutePath());
>> 88:             List<String> commands = new ArrayList<>();
>> 89:             commands.addAll(files);
> 
> I suggest disabling `missing` doclint warnings here, since we have a lot of those, and they drown out the actual errors when javadoc fails.
> Suggestion:
> 
>             commands.add("-Xdoclint:all,-missing");
>             commands.addAll(files);

@JornVernee thank for the review. I have updated the arguments as you suggested and set the base for this PR to master.

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

PR Review Comment: https://git.openjdk.org/jextract/pull/227#discussion_r1542810737


More information about the jextract-dev mailing list