RFR: 8299227: host `exif.org` not found in link in doc comment

Phil Race prr at openjdk.org
Thu Dec 22 18:35:51 UTC 2022


On Thu, 22 Dec 2022 17:07:44 GMT, Abhishek Kumar <abhiscxk at openjdk.org> wrote:

> Broken links are replaced by the similar links fixed in [JDK-8288527](https://bugs.openjdk.org/browse/JDK-8288527) for file `src/java.desktop/share/classes/javax/imageio/plugins/tiff/ExifTIFFTagSet.java`.

Changes requested by prr (Reviewer).

src/java.desktop/share/classes/com/sun/imageio/plugins/tiff/TIFFImageWriteParam.java line 81:

> 79:  * <td>Exif JPEG</td>
> 80:  * <td>Exif-specific JPEG compression (see note following this table)</td>
> 81:  * <td><a href="https://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf">Exif 2.2 Specification</a>

BTW the link at the other end is EXIF 2.3 .. so you probably want to fix the text too.

src/java.desktop/share/classes/com/sun/imageio/plugins/tiff/TIFFImageWriteParam.java line 82:

> 80:  * <td>Exif-specific JPEG compression (see note following this table)</td>
> 81:  * <td><a href="https://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf">Exif 2.2 Specification</a>
> 82:  * (PDF), section 4.5.5, "Basic Structure of Thumbnail Data"</td>

This is now 4.5.8

src/java.desktop/share/classes/javax/imageio/metadata/doc-files/tiff_metadata.html line 579:

> 577: <td><a href="https://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf">
> 578:     Exif 2.2 Specification</a>
> 579: (PDF), section 4.5.5, "Basic Structure of Thumbnail Data"</td>

This is now 4.5.8

src/java.desktop/share/native/libjavajpeg/imageioJPEG.c line 1619:

> 1617:  * (see https://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf, section 4.7, page 58)
> 1618:  */
> 1619: #define IS_EXIF(c) \

Page number is no longer correct

src/java.desktop/share/native/libjavajpeg/imageioJPEG.c line 1718:

> 1716:              *     Must be YCbCr (see http://www.w3.org/Graphics/JPEG/jfif3.pdf, page 2)
> 1717:              *  - we got EXIF image
> 1718:              *     Must be YCbCr (see https://www.cipa.jp/std/documents/e/DC-008-2012_E.pdf, section 4.7, page 63)

Page number is no longer correct

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

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



More information about the client-libs-dev mailing list