On Tue, 18 Feb 2025 17:11:33 GMT, Hannes Wallnöfer <hann...@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? Indeed, would be much better if we can just check `log.noErrors()`. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23441#discussion_r1978498602