<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">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">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" 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" 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="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>