On Tue, 28 Sep 2021 08:23:30 GMT, Masanori Yano <[email protected]> wrote:
>> Could you please review the 8248001 bug fixes?
>>
>> The problem is that javadoc generates invalid HTML pages whose ftp:// links
>> are broken. The fix changes not to use protocol names directly, but to use
>> regular expression.
>
> Masanori Yano has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8248001: javadoc generates invalid HTML pages whose ftp:// links are broken
I updated the copyright date to 2021.
I think `checkHtml` is not necessary because `JavadocTester` checks links by
`checkLinks()`. The following is log when the fix has not been applyed.
Checked 11 files.
Found 189 references to 54 anchors in 16 files and 6 other URIs.
* 1 missing files
0 duplicate ids
0 missing ids
Schemes
1 file
1 ftp
1 http
2 https
1 mailto
Hosts
1 www.example.com
2 docs.oracle.com
1 www.exsample.com
FAILED: 1 errors found when checking links
at javadoc.tester.JavadocTester.checkLinks(JavadocTester.java:565)
at javadoc.tester.JavadocTester.javadoc(JavadocTester.java:384)
at TestHrefInDocComment.test(TestHrefInDocComment.java:46)
-------------
PR: https://git.openjdk.java.net/jdk/pull/5198