<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Looks good.  The IDE question still stands.  If you are using an IDE this shouldn't have happened.<div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Feb 10, 2016, at 8:41 PM, Konstantin Shefov <<a href="mailto:konstantin.shefov@oracle.com" class="">konstantin.shefov@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type" class="">
  
  <div bgcolor="#FFFFFF" text="#000000" class="">
    <tt class="">Doug, Christian, Vladimir, thank you for reviewing.<br class="">
      <br class="">
      Here is a patch with space:<br class="">
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~kshefov/8149472/webrev.01/">http://cr.openjdk.java.net/~kshefov/8149472/webrev.01/</a><br class="">
      <br class="">
      -Konstantin<br class="">
    </tt><br class="">
    <div class="moz-cite-prefix">10.02.2016 23:02, Christian Thalinger
      пишет:<br class="">
    </div>
    <blockquote cite="mid:B517128F-5765-4628-B9B8-EA4B86BB267B@oracle.com" type="cite" class="">
      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" class="">
      +            return Objects.equals(x,y);<br class="">
      <br class="">
      There is a missing space.  Are you using an IDE?
      <div class=""><br class="">
        <div class="">
          <blockquote type="cite" class="">
            <div class="">On Feb 10, 2016, at 1:20 AM, Konstantin Shefov
              <<a moz-do-not-send="true" href="mailto:konstantin.shefov@oracle.com" class="">konstantin.shefov@oracle.com</a>>
              wrote:</div>
            <br class="Apple-interchange-newline">
            <div class="">
              <meta http-equiv="content-type" content="text/html;
                charset=UTF-8" class="">
              <div text="#000000" bgcolor="#FFFFFF" class=""> Hello<br class="">
                <br class="">
                Please review a bug fix in JVMCI
                jdk.vm.ci.hotspot.HotSpotConstantReflectionProvider::constantEquals(Constant

                x, Constant y) method.
                <meta http-equiv="content-type" content="text/html;
                  charset=UTF-8" class="">
                <br class="">
                <br class="">
                <meta http-equiv="content-type" content="text/html;
                  charset=UTF-8" class="">
                A NullPointerException is thrown, when one calls this
                method with the first argument x equal to null. <br class="">
                <br class="">
                Additional check should be added for null as the first
                argument.<br class="">
                <br class="">
                Bug: <a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8149472">https://bugs.openjdk.java.net/browse/JDK-8149472</a>
                <br class="">
                Webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Ekshefov/8149472/webrev.00/">http://cr.openjdk.java.net/~kshefov/8149472/webrev.00/</a><br class="">
                <br class="">
                Thanks<br class="">
                -Konstantin<br class="">
                <br class="">
              </div>
            </div>
          </blockquote>
        </div>
        <br class="">
      </div>
    </blockquote>
    <br class="">
  </div>

</div></blockquote></div><br class=""></div></body></html>