GIF native IIOMetadata assumes characterCellWidth/Height as 2bytes

XMLWordPrintable

    • Type: CSR
    • Resolution: Approved
    • Priority: P4
    • 11
    • Component/s: client-libs
    • None
    • behavioral
    • minimal
    • The changes proposed in this CSR do not modify the API specification but pertain to two specific Attributes in the XML schema for GIF Image metadata.
    • Java API
    • SE

      Summary

      Correct the maximum permissible value for two Attributes of PlainTextExtension Element in the XML schema for GIF Image Metadata as recommended by the image specification.

      Problem

      . The maximum permissible value for two Attributes namely- Character Cell Width and Character Cell Height, of PlainTextExtension Element in the XML schema for GIF Image Metadata is incorrect. <br> . GIF Image specification version 89a mentions that the concerned attributes occupy 1 Byte each in the Plain Text Extension Block and hence, the maximum permissible value for these two attributes would be 255.

      Solution

      . Correct the maximum permissible value for the concerned Attributes in the XML schema for GIF Image Metadata. <br> . The XML schema for GIF Image metadata is documented in a HTML file that can be located at- src/java.desktop/share/classes/javax/imageio/metadata/doc-files/gif_metadata.html <br> . The proposed solution and relevant code changes have been reviewed for correctness. <br> . Review discussions can be found at- http://mail.openjdk.java.net/pipermail/2d-dev/2018-March/009030.html

      Specification

      File: src/java.desktop/share/classes/javax/imageio/metadata/doc-files/gif_metadata.html

           289       &lt;!ATTLIST "PlainTextExtension" "characterCellWidth" #CDATA
     290          #REQUIRED&gt;
     291         &lt;!-- The width of a character cell --&gt;
     292         &lt;!-- Data type: Integer --&gt;
     293         &lt;!-- Min value: 1 (inclusive) --&gt;
--   294         &lt;!-- Max value: 65535 (inclusive) --&gt;
++   294         &lt;!-- Max value: 255 (inclusive) --&gt;

     295       &lt;!ATTLIST "PlainTextExtension" "characterCellHeight" #CDATA
     296          #REQUIRED&gt;
     297         &lt;!-- The height of a character cell --&gt;
     298         &lt;!-- Data type: Integer --&gt;
     299         &lt;!-- Min value: 1 (inclusive) --&gt;
--   300         &lt;!-- Max value: 65535 (inclusive) --&gt;
++   300         &lt;!-- Max value: 255 (inclusive) --&gt;

            Assignee:
            Prahalad Kumar Narayanan (Inactive)
            Reporter:
            Igor Nekrestyanov (Inactive)
            Philip Race
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: