RFR: 8349369: test/docs/jdk/javadoc/doccheck/checks/jdkCheckLinks.java did not report on missing man page files
Hannes Wallnöfer
hannesw at openjdk.org
Tue Feb 18 19:22:35 UTC 2025
On Tue, 4 Feb 2025 14:39:49 GMT, Nizar Benalla <nbenalla at openjdk.org> wrote:
> Please review this small update to make the test more robust, the test did not always check links to other files if they did not contain a fragment or a `#`symbol.
>
> I'd like to add that the original [doccheck](https://github.com/openjdk/doccheck) also didn't pick up on these missing files when I ran it.
>
> TIA
test/docs/jdk/javadoc/doccheck/doccheckutils/checkers/LinkChecker.java line 227:
> 225: && missingFiles == 0
> 226: && badSchemes == 0
> 227: && log.noErrors();
So we have both these counters and the `log` instance to track errors. Shouldn't the `missingIds` counter have bee increased when the error was logged?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23441#discussion_r1960166755
More information about the javadoc-dev
mailing list