<div dir="ltr"><div>Thanks for review and sponsoring!</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span style="font-size:12.8px">Note that at the moment are past the JDK9 FC date, and we should not</span><br style="font-size:12.8px"><span style="font-size:12.8px">integrate enhancements right now. There is no process at the moment if</span><br style="font-size:12.8px"><span style="font-size:12.8px">and how these can be integrated. So it may take a while to push the</span><br style="font-size:12.8px"><span style="font-size:12.8px">change.</span></blockquote><div><br></div><div>Should I wait until JDK10 forest and ping someone? Should I update webrev with includes cleanup now or wait until new forest? </div><div>Does it make sense to keep making reviews for simple bugs (e.g. like JDK-8151045) or it's better to wait until new forest (note that I'm not openjdk author yet, so these questions could have obvious answers)? </div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>--</div>Best regards,<div>Tolstopyatov Vsevolod</div></div></div></div>
<br><div class="gmail_quote">On Wed, Jun 8, 2016 at 12:51 AM, Derek White <span dir="ltr"><<a href="mailto:derek.white@oracle.com" target="_blank">derek.white@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 bgcolor="#FFFFFF" text="#000000"><div><div class="h5">
    <div>On 5/30/16 8:05 AM, Всеволод
      Толстопятов wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">
        <div>Hi all,</div>
        <div>Please review and sponsor this change for JDK-8142749.</div>
        <div>Note that not only HeapRegion::_predicted_bytes_to_copy
          related code was removed, but also some unused (since the
          first commit) method/class parameters and
          HeapRegion::object_iterate_mem_careful method. All changed
          files contain at least several unused includes, but I'm not
          sure it's applicable to remove them in this patch.</div>
        <div><br>
        </div>
        CR: <a href="https://bugs.openjdk.java.net/browse/JDK-8142749" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8142749</a>
        <div>Webrev: <a href="http://cr.openjdk.java.net/%7Efzhinkin/vtolstopyatov/8142749/webrev.00/" target="_blank">http://cr.openjdk.java.net/~fzhinkin/vtolstopyatov/8142749/webrev.00/</a></div>
        <div>Testing: hotspot_gc tests</div>
        <div><br clear="all">
          <div>
            <div data-smartmail="gmail_signature">
              <div dir="ltr">
                <div>--</div>
                Best regards,
                <div>Tolstopyatov Vsevolod</div>
              </div>
            </div>
          </div>
        </div>
      </div>
    </blockquote>
    </div></div><p>Looks good!</p><span class="HOEnZb"><font color="#888888">
    <p><br>
    </p>
    <p> - Derek<br>
    </p>
  </font></span></div>

</blockquote></div><br></div>