<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Igor,<br><div><br></div><div>The change looks good to me, I compared old/new, the port looks good.</div><div><br></div><div>For the reviewers, here is the original test code:</div><div><a href="http://hg.openjdk.java.net/jdk/jdk/file/2e928420389d/src/hotspot/share/memory/virtualspace.cpp#l1071">http://hg.openjdk.java.net/jdk/jdk/file/2e928420389d/src/hotspot/share/memory/virtualspace.cpp#l1071</a><br></div><div><br></div><div>Thanks,</div><div>Jc</div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 16, 2018 at 9:06 AM Igor Ignatyev <<a href="mailto:igor.ignatyev@oracle.com">igor.ignatyev@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><a href="http://cr.openjdk.java.net/~iignatyev//8171097/webrev.00/index.html" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~iignatyev//8171097/webrev.00/index.html</a><br>
> 188 lines changed: 187 ins; 1 del; 0 mod; <br>
<br>
Hi all,<br>
<br>
could you please review this small (and hopefully trivial) patch which converts internal TestReservedSpace_test to gtest? <br>
since the old test is still used by WhiteBox::runMemoryUnitTests, the old code hasn't been removed. it will be removed later when all 4 tests used by runMemoryUnitTests are converted.<br>
<br>
webrev: <a href="http://cr.openjdk.java.net/~iignatyev//8171097/webrev.00/index.html" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~iignatyev//8171097/webrev.00/index.html</a><br>
JBS: <a href="https://bugs.openjdk.java.net/browse/JDK-8171097" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8171097</a><br>
testing:<br>
- converted tests on linux-x64, windows-x64, macosx-x64, solaris-sparcv9 in product and fastdebug variants <br>
- build w/ precompiled-headers enabled and disabled<br>
<br>
PS the patch has been originally created by Kirill Zh, but hasn't been sent out for official review<br>
<br>
Thanks,<br>
-- IgorĀ  </blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>