<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <meta http-equiv="content-type" content="text/html;
      charset=windows-1252">
    <pre><span class="new">} else if((dataType == TIFFTag.TIFF_RATIONAL

I know that someone (ahem Brian ;-)) has a lot of "if(" code in here
but please make any new ones "if ("

Other than that, +1

-phil.
</span></pre>
    <br>
    <br>
    On 1/24/17, 1:08 PM, Brian Burkhalter wrote:
    <blockquote
      cite="mid:665EA001-9810-4961-BD57-E4C188641B58@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      Hi Jay,
      <div><br>
      </div>
      <div>I think line 910 of TIFFField needs to state “and count !=
        1”. Also the copyright year should now be 2017.</div>
      <div><br>
      </div>
      <div>OK to make these changes without another webrev: +1.</div>
      <div><br>
      </div>
      <div>Thanks,</div>
      <div><br>
      </div>
      <div>Brian</div>
      <div><br>
        <div>
          <div>On Jan 18, 2017, at 1:03 AM, Jayathirth D V <<a
              moz-do-not-send="true"
              href="mailto:jayathirth.d.v@oracle.com">jayathirth.d.v@oracle.com</a>>
            wrote:</div>
          <br class="Apple-interchange-newline">
          <blockquote type="cite">
            <div style="margin: 0in 0in 0.0001pt; font-size: 11pt;
              font-family: Calibri, sans-serif; font-style: normal;
              font-variant: normal; font-weight: normal; letter-spacing:
              normal; line-height: normal; orphans: auto; text-align:
              start; text-indent: 0px; text-transform: none;
              white-space: normal; widows: auto; word-spacing: 0px;
              -webkit-text-stroke-width: 0px; background-color: rgb(255,
              255, 255);"><span style="color: rgb(31, 73, 125);">Please
                find updated webrev for review :<o:p></o:p></span></div>
            <div style="margin: 0in 0in 0.0001pt; font-size: 11pt;
              font-family: Calibri, sans-serif; font-style: normal;
              font-variant: normal; font-weight: normal; letter-spacing:
              normal; line-height: normal; orphans: auto; text-align:
              start; text-indent: 0px; text-transform: none;
              white-space: normal; widows: auto; word-spacing: 0px;
              -webkit-text-stroke-width: 0px; background-color: rgb(255,
              255, 255);"><span style="color: rgb(31, 73, 125);"><a
                  moz-do-not-send="true"
                  href="http://cr.openjdk.java.net/%7Ejdv/8152561/webrev.01/"
                  style="color: rgb(149, 79, 114); text-decoration:
                  underline;">http://cr.openjdk.java.net/~jdv/8152561/webrev.01/</a></span></div>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
  </body>
</html>