<div dir="ltr"><div dir="ltr">Hi all,<div><br></div><div>I created the webrev with the metadata here:</div><div><a href="http://cr.openjdk.java.net/~jcbeyler/8210192/webrev.02/" target="_blank">http://cr.openjdk.java.net/~jcbeyler/8210192/webrev.02/</a><br></div><div><br></div><div>I pushed the fix since Thomas & Stefan reviewed it. I added Man as the contributor and ensured it built and passed the new test on my side (which with Stefan's testing seemed to be sufficient).</div><div><br></div><div>Thanks Man for the fix and Thomas and Stefan for the reviews :)</div><div>Jc</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 4, 2018 at 6:25 AM Thomas Schatzl <<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
On Mon, 2018-09-03 at 18:37 -0700, Man Cao wrote:<br>
> Hi all,<br>
> <br>
> Thanks, Stefan! I have added a test for the hsperf counter. The test<br>
> is based on test/hotspot/jtreg/gc/TestGenerationPerfCounter.java. <br>
> New webrev:<br>
> <a href="http://cr.openjdk.java.net/~jcbeyler/8210192/webrev.01/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jcbeyler/8210192/webrev.01/</a><br>
> <br>
> I have run some basic tests with "make run-test", and those related<br>
> to test/hotspot/jtreg/gc/testlibrary/PerfCounter.java are still<br>
> passing.<br>
> <br>
<br>
  looks good.<br>
<br>
Thomas<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="m_-6990078089944674765gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>