On Tue, 28 Sep 2021 08:23:30 GMT, Masanori Yano <my...@openjdk.org> 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

Reply via email to