<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">Approved.<br>
      <br>
      -phil.<br>
      <br>
      On 8/12/14 2:56 PM, Pushan Hinduja wrote:<br>
    </div>
    <blockquote
cite="mid:CAErB7HmhdV5uG3JorVn26BXKhks8ynviLc1AjrjxpogQ-2QO+g@mail.gmail.com"
      type="cite">
      <div dir="ltr">Hello, 
        <div><br>
        </div>
        <div>Here is the updated Webrev for the fix: </div>
        <div><a moz-do-not-send="true"
            href="http://cr.openjdk.java.net/%7Essides/8054878/8054878.1/"
            target="_blank"
            style="font-family:arial,sans-serif;font-size:13px">http://cr.openjdk.java.net/~ssides/8054878/8054878.1/</a><br>
        </div>
        <div><br>
        </div>
        <div>Thank you</div>
        <div>--</div>
        <div>Pushan Hinduja</div>
      </div>
      <div class="gmail_extra"><br>
        <br>
        <div class="gmail_quote">On Tue, Aug 12, 2014 at 1:01 PM, Phil
          Race <span dir="ltr"><<a moz-do-not-send="true"
              href="mailto:philip.race@oracle.com" target="_blank">philip.race@oracle.com</a>></span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <div text="#000000" bgcolor="#FFFFFF">
              <div>
                <pre><span> 309      * @exception DOMException thrown if the node is not <code>null</code> or</span>
<span> 310      * not an instance of <code>IIOMetadtaNode</code>

Seems to be a logic problem with the negatives here.
</span><span>I think what you want is and not or
</span><span>
Also you misspelt</span> IIOMetaDataNode - missing 'a'

I also prefer @throws as it makes it easier to write like this :-

@throws DOMException if {@code node} is not {@code null} and not an instance
of {@code IIOMetadataNode}

-phil.
<span class="HOEnZb"><font color="#888888"><span></span></font></span></pre>
                <div>
                  <div class="h5"> <br>
                    On 8/12/14 11:52 AM, Pushan Hinduja wrote:<br>
                  </div>
                </div>
              </div>
              <div>
                <div class="h5">
                  <blockquote type="cite">
                    <div dir="ltr">Hello, 
                      <div><br>
                      </div>
                      <div>Could you please review the fix for the
                        following bug: </div>
                      <div><a moz-do-not-send="true"
                          href="https://bugs.openjdk.java.net/browse/JDK-8054878"
                          target="_blank">https://bugs.openjdk.java.net/browse/JDK-8054878</a></div>
                      <div><br>
                      </div>
                      <div>The corresponding Webrev is located at: </div>
                      <div><a moz-do-not-send="true"
                          href="http://cr.openjdk.java.net/%7Essides/8054878/8054878.0/"
style="font-family:arial,sans-serif;font-size:13px" target="_blank">http://cr.openjdk.java.net/~ssides/8054878/8054878.0/</a><br>
                      </div>
                      <div>
                        <div><br>
                        </div>
                        <div>This addresses missing @exception or @throw
                          tags in certain files in javax.imageio</div>
                        <div><br>
                        </div>
                        <div>Best Regards, </div>
                        -- <br>
                        <div dir="ltr">
                          <div>Pushan Hinduja<br>
                          </div>
                        </div>
                      </div>
                    </div>
                  </blockquote>
                  <br>
                </div>
              </div>
            </div>
          </blockquote>
        </div>
        <br>
        <br clear="all">
        <div><br>
        </div>
        -- <br>
        <div dir="ltr">
          <div>Pushan Hinduja<br>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
  </body>
</html>