Re: [OpenJDK 2D-Dev] RFR: JDK-8155960: TIFF javadoc contains HTML entities inside {@code} tags

[Philip Race]

+1 (and one reviewer is enough for this minor doc fix).

-phil.

On 8/3/16, 3:58 PM, Brian Burkhalter wrote:

Please review at your convenience.

Issue:https://bugs.openjdk.java.net/browse/JDK-8155960
Patch:[1]

Note that the portion of this issue in javax.imageio.plugins.tiff was resolved as part of the fix for https://bugs.openjdk.java.net/browse/JDK-8149016.

Thanks,

Brian

[1] diff

--- a/src/java.desktop/share/classes/com/sun/imageio/plugins/tiff/TIFFColorConverter.java +++ b/src/java.desktop/share/classes/com/sun/imageio/plugins/tiff/TIFFColorConverter.java

@@ -47,7 +47,7 @@
      * @throws NullPointerException if {@code result} is
      * {@code null}.
      * @throws ArrayIndexOutOfBoundsException if
-     * {@code result.length < 3}.
+     * {@code result.length < 3}.
      */

public abstract void fromRGB(float r, float g, float b, float[] result);

@@ -63,7 +63,7 @@
      * @throws NullPointerException if {@code rgb} is
      * {@code null}.
      * @throws ArrayIndexOutOfBoundsException if
-     * {@code rgb.length < 3}.
+     * {@code rgb.length < 3}.
      */

public abstract void toRGB(float x0, float x1, float x2, float[] rgb);

 }

--- a/src/java.desktop/share/classes/com/sun/imageio/plugins/tiff/TIFFDecompressor.java +++ b/src/java.desktop/share/classes/com/sun/imageio/plugins/tiff/TIFFDecompressor.java

@@ -353,7 +353,7 @@
      * <p> The pixels in the source region to be copied are
      * those with X coordinates of the form {@code activeSrcMinX +
      * k*subsampleX}, where {@code k} is an integer such
-     * that {@code 0 &le; k &lt; dstWidth}.
+     * that {@code 0 <= k < dstWidth}.
      */
     protected int activeSrcMinX;


@@ -365,7 +365,7 @@
      * <p> The pixels in the source region to be copied are
      * those with Y coordinates of the form {@code activeSrcMinY +
      * k*subsampleY}, where {@code k} is an integer such
-     * that {@code 0 &le; k &lt; dstHeight}.
+     * that {@code 0 <= k < dstHeight}.
      */
     protected int activeSrcMinY;