<html>

  <head>

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

  </head>

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

    <font face="Times New Roman, Times, serif">Looks good.<br>

      <br>

      Jon<br>

    </font><br>

    <div class="moz-cite-prefix">On 04/21/2016 05:40 AM, Thomas Schatzl

      wrote:<br>

    </div>

    <blockquote cite="mid:1461242416.3379.35.camel@oracle.com"

      type="cite">

      <pre wrap="">Hi all,

  could I have reviews for this one-line change that fixes calculation

of the number of words for a given size in bits in

 G1CardLiveData::free_large_bitmap()?

The change is to use the exact same calculation as in

 G1CardLiveData::allocate_large_bitmap() for uniformity.

There is no semantic difference in both since the size_in_bits

parameter is always word-aligned anyway.

CR:

<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8154838">https://bugs.openjdk.java.net/browse/JDK-8154838</a>

Webrev:

<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tschatzl/8154838/webrev/">http://cr.openjdk.java.net/~tschatzl/8154838/webrev/</a>

Testing:

local compilation, local run of gcbasher

Thanks,

  Thomas

</pre>

    </blockquote>

    <br>

  </body>

</html>