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