<html>

  <head>

    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">

  </head>

  <body text="#000000" bgcolor="#FFFFFF">

    Hi Sangheon,<br>

    <br>

    <div class="moz-cite-prefix">On 2015-02-13 07:54, Sangheon Kim

      wrote:<br>

    </div>

    <blockquote cite="mid:54DD9FA0.2000107@oracle.com" type="cite">

      <meta http-equiv="content-type" content="text/html; charset=utf-8">

      <pre wrap="">Hi All,

Please review this small enhancement for better error messages when assert fails.

This change would be helpful for <a moz-do-not-send="true" id="key-val" rel="4766142" href="https://bugs.openjdk.java.net/browse/JDK-8071930">JDK-8071930</a> which is hard to reproduce.

I will need a sponsor for this change.

CR:

<a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8073115">https://bugs.openjdk.java.net/browse/JDK-8073115</a>

Webrev:

<a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Esangheki/8073115/webrev.00/">http://cr.openjdk.java.net/~sangheki/8073115/webrev.00/</a></pre>

    </blockquote>

    <br>

<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~sangheki/8073115/webrev.00/src/share/vm/gc_implementation/parallelScavenge/objectStartArray.hpp.udiff.html">http://cr.openjdk.java.net/~sangheki/8073115/webrev.00/src/share/vm/gc_implementation/parallelScavenge/objectStartArray.hpp.udiff.html</a><br>

    <br>

    You are duplicating the same assert three times. Could you create a

    new define, say #define assert_covered_region_contains(addr) ...,

    and remove the duplication? This will also have the benefit of

    lowering the line noise in the functions you change.<br>

    <br>

    Thanks,<br>

    StefanK<br>

    <br>

    <blockquote cite="mid:54DD9FA0.2000107@oracle.com" type="cite">

      <pre wrap="">

Testing:

JPRT

Thanks,

Sangheon

</pre>

    </blockquote>

    <br>

  </body>

</html>