RFR: 8288528: broken links in java.desktop [v2]

Phil Race prr at openjdk.org
Fri Jun 17 20:48:57 UTC 2022


On Fri, 17 Jun 2022 17:14:56 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> 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



More information about the client-libs-dev mailing list