<div dir="auto"><div>Thank JC for the fast response, and sorry for the oversight.</div><div dir="auto"><br></div><div dir="auto">-Man<br><br><div class="gmail_quote" dir="auto"><div dir="ltr">On Sat, Dec 8, 2018, 13:18 JC Beyler <<a href="mailto:jcbeyler@google.com">jcbeyler@google.com</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr">Hi Roman,<div><br></div><div>I filed a bug here: <a href="https://bugs.openjdk.java.net/browse/JDK-8215043" target="_blank" rel="noreferrer">https://bugs.openjdk.java.net/browse/JDK-8215043</a></div><div><br></div><div>and I have a webrev that I'll send up for review in a separate email.</div><div><br></div><div>Sorry about that,</div><div>Jc</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 7, 2018 at 12:02 PM Roman Kennke <<a href="mailto:rkennke@redhat.com" target="_blank" rel="noreferrer">rkennke@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It is already pushed, but I noticed you have removed the definition of<br>
Abstract_VM_Version::parallel_worker_threads() but the declaration is<br>
still there. One can code against this but then get linker failure.<br>
Maybe also remove:<br>
<br>
static unsigned int parallel_worker_threads();<br>
<br>
in vm_version.hpp?<br>
<br>
Roman<br>
<br>
> Could anyone review this cleanup change?<br>
> In addition to straight code movement, it removes unnecessary dependency<br>
> on adaptiveSizePolicy.hpp in G1 code, and removed unnecessary include<br>
> statements in adaptiveSizePolicy.hpp/cpp.<br>
> <br>
> Webrev: <a href="https://cr.openjdk.java.net/~manc/8213224/webrev.00/" rel="noreferrer noreferrer" target="_blank">https://cr.openjdk.java.net/~manc/8213224/webrev.00/</a><br>
> RFE: <a href="https://bugs.openjdk.java.net/browse/JDK-8213224" rel="noreferrer noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8213224</a><br>
> Tested on submit repo with the help from JC (CCed).<br>
> <br>
> Thanks,<br>
> Man<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="m_1651586518367309123gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>
</blockquote></div></div></div>