<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <p>Looks good to me. <br>
    </p>
    <p>Only thing is that in the test, if one test fails it will not
      proceed to the next test. I think we should test all combinations
      and fail at last with the informations of all failed combinations.<br>
    </p>
    Regards<br>
    Prasanta<br>
    <div class="moz-cite-prefix">On 1/11/2017 11:19 PM, Jayathirth D V
      wrote:<br>
    </div>
    <blockquote cite="mid:bbe0397e-1c6f-4733-951b-e01f3a54cf1e@default"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      <meta name="Generator" content="Microsoft Word 15 (filtered
        medium)">
      <style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:#954F72;
        text-decoration:underline;}
p.MsoPlainText, li.MsoPlainText, div.MsoPlainText
        {mso-style-priority:99;
        mso-style-link:"Plain Text Char";
        margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle17
        {mso-style-type:personal-compose;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.PlainTextChar
        {mso-style-name:"Plain Text Char";
        mso-style-priority:99;
        mso-style-link:"Plain Text";
        font-family:"Calibri",sans-serif;}
.MsoChpDefault
        {mso-style-type:export-only;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
/* List Definitions */
@list l0
        {mso-list-id:1870987899;
        mso-list-type:hybrid;
        mso-list-template-ids:775459510 859180862 67698713 67698715 67698703 67698713 67698715 67698703 67698713 67698715;}
@list l0:level1
        {mso-level-text:"%1\)";
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:47.25pt;
        text-indent:-.25in;}
@list l0:level2
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:83.25pt;
        text-indent:-.25in;}
@list l0:level3
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        margin-left:119.25pt;
        text-indent:-9.0pt;}
@list l0:level4
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:155.25pt;
        text-indent:-.25in;}
@list l0:level5
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:191.25pt;
        text-indent:-.25in;}
@list l0:level6
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        margin-left:227.25pt;
        text-indent:-9.0pt;}
@list l0:level7
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:263.25pt;
        text-indent:-.25in;}
@list l0:level8
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:299.25pt;
        text-indent:-.25in;}
@list l0:level9
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        margin-left:335.25pt;
        text-indent:-9.0pt;}
ol
        {margin-bottom:0in;}
ul
        {margin-bottom:0in;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
      <div class="WordSection1">
        <p class="MsoNormal">Hello All,<o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText">Please review the following  fix in
          JDK9:<o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText">Bug :              <a
            moz-do-not-send="true"
            href="https://bugs.openjdk.java.net/browse/JDK-8152561">https://bugs.openjdk.java.net/browse/JDK-8152561</a>
          <o:p></o:p></p>
        <p class="MsoPlainText">Webrev :     <a moz-do-not-send="true"
            href="http://cr.openjdk.java.net/%7Ejdv/8152561/webrev.00/">http://cr.openjdk.java.net/~jdv/8152561/webrev.00/</a>
          <o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText">Issue : There is difference in how we
          interpret “count” variable passed to
          TIFFField.createArrayForType(int dataType, int count) and
          constructor TIFFField(TIFFTag tag, int type, int count, Object
          data).<o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText">Root cause : There are certain
          limitation on what the “count” value should be based on
          dataType of TIFFTag. We check these conditions in
          TIFFField(TIFFTag tag, int type, int count, Object data) but
          we don’t verify all the conditions in
          TIFFField.createArrayForType(int dataType, int count).<o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText">Solution : Verify all the required
          conditions that has to be applied on “count” variable based on
          datatype of TIFFTag in TIFFField.createArrayForType(int
          dataType, int count). Also I have made specification changes
          for TIFFField(TIFFTag tag, int type, int count) which will
          elaborate on what exceptions will be thrown if we don’t follow
          same “count” and “dataType” relationship.<o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText">Note : Because of tighter conditions in
          TIFFField.createArrayForType(int dataType, int count) under
          the proposed fix 2 JCK tests are failing.<o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText"
          style="margin-left:47.25pt;text-indent:-.25in;mso-list:l0
          level1 lfo1"><!--[if !supportLists]--><span
            style="mso-list:Ignore">1)<span style="font:7.0pt
              "Times New Roman"">      </span></span><!--[endif]--><u>api/javax_imageio/plugins/tiff/TIFFField/index.html#TIFFFieldTest
            : <i>CreateArrayForType_ValidScenario</i><o:p></o:p></u></p>
        <p class="MsoPlainText" style="margin-left:47.25pt">This is
          failing because they are passing count values of 0 for
          TIFFTag.TIFF_RATIONAL & TIFFTag.TIFF_SRATIONAL which will
          throw IAE. Also it will throw IAE if we pass count value 0 or
          2 for TIFFTag.TIFF_IFD_POINTER.<o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText"
          style="margin-left:47.25pt;text-indent:-.25in;mso-list:l0
          level1 lfo1"><!--[if !supportLists]--><span
            style="mso-list:Ignore">2)<span style="font:7.0pt
              "Times New Roman"">      </span></span><!--[endif]--><u>api/javax_imageio/plugins/tiff/TIFFField/index.html#ConstructorTests
            : <i>Constructor02_TagIsNull_ThrowNPE</i><o:p></o:p></u></p>
        <p class="MsoPlainText" style="margin-left:47.25pt">I think this
          scenario expects constructor to throw NPE when “tag” is null.
          But before it checks for “tag” value in TIFFField(TIFFTag tag,
          int type, int count, Object data) we call
          TIFFField.createArrayForType(int dataType, int count) to
          create needed data. So like mentioned in previous JCK test
          which is failing, we will be throwing IAE in all cases where
          as test expects us to throw NPE.<o:p></o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText"><o:p> </o:p></p>
        <p class="MsoPlainText">Thanks,<o:p></o:p></p>
        <p class="MsoPlainText">Jay<o:p></o:p></p>
        <p class="MsoNormal"><o:p> </o:p></p>
      </div>
    </blockquote>
    <br>
  </body>
</html>