On Fri, 17 Jun 2022 17:14:56 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> doccheck reports some broken links which are rectified
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Review comment fix

Changes requested by prr (Reviewer).

src/java.desktop/share/classes/java/awt/doc-files/AWTThreadIssues.html line 157:

> 155:   <li> Other packages can create displayable components for internal
> 156:        needs and never make them undisplayable. See
> 157: <a href="https://bugs.openjdk.org/browse/JDK-4515058";>

This is about Solaris IMF .. I think we don't need to worry about it any more.

src/java.desktop/share/classes/java/awt/doc-files/AWTThreadIssues.html line 159:

> 157: <a href="https://bugs.openjdk.org/browse/JDK-4515058";>
> 158: 4515058</a>,
> 159: <a href="https://bugs.openjdk.org/browse/JDK-4671025";>

And even better it was resolved in 1.5 under this bug id which I guess was 
still open when this doc was written

src/java.desktop/share/classes/java/awt/doc-files/AWTThreadIssues.html line 161:

> 159: <a href="https://bugs.openjdk.org/browse/JDK-4671025";>
> 160: 4671025</a>, and
> 161: <a href="https://bugs.openjdk.org/browse/JDK-4465537";>

This bug was about some off-screen image problem with JScrollBar
It was closed as a dup of 4417287 which was fixed in 1.4

So in summary it is wrong to keep these linked bugs in here.
Remove the lot of them

src/java.desktop/share/classes/javax/imageio/plugins/tiff/ExifTIFFTagSet.java 
line 36:

> 34:  * manufacturers.  The Exif specification may be found at
> 35:  * <a 
> href="https://urldefense.com/v3/__http://web.archive.org/web/20131018091152/http:/*exif.org/Exif2-2.PDF__;Lw!!ACWV5N9M2RV99hQ!I-Fd-s8_Zomn1UUt78DbuGfA1KfJRFl6Xzr8upsTCb7FiZOGBhxJxmrm3a6ie_WywxyifkaYQQKu36oPy5wO$";>
> 36:  * {@code 
> http://web.archive.org/web/20131018091152/http://exif.org/Exif2-2.PDF}

Let's not point to web archives
https://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf seems like a better 
option. 
Perhaps @bplb has another idea.

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

PR: https://git.openjdk.org/jdk/pull/9195

Reply via email to