<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Mandy,<br>
<br>
Thank you for the review!<br>
Jane<br>
<div class="moz-cite-prefix">On 01.04.2015 17:40, Mandy Chung wrote:<br>
</div>
<blockquote cite="mid:551C0356.6060201@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
<br>
<div class="moz-cite-prefix">On 4/1/2015 3:49 AM, Evgeniya
Stepanova wrote:<br>
</div>
<blockquote cite="mid:551BCD40.5010601@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8"
http-equiv="Content-Type">
Hi Mandy!<br>
<br>
Thanks for looking at this<br>
One of the testlibrary classes used is depends on
java.management. Added tag @modules<br>
<br>
new webrev: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Eeistepan/8026043/webrev.01">http://cr.openjdk.java.net/~eistepan/8026043/webrev.01</a><br>
<br>
</blockquote>
<br>
Thanks. Looks okay to me.<br>
<br>
Nit: There are a few long lines that you may consider breaking
them.<br>
No need for an updated webrev.<br>
<br>
Mandy<br>
<br>
<blockquote cite="mid:551BCD40.5010601@oracle.com" type="cite">
Thanks,<br>
Jane<br>
<br>
<div class="moz-cite-prefix">On 31.03.2015 19:25, Mandy Chung
wrote:<br>
</div>
<blockquote cite="mid:551ACA7E.8010502@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8"
http-equiv="Content-Type">
<div class="moz-cite-prefix">Does this need any @modules tag?<br>
<br>
Mandy<br>
<br>
On 3/31/15 7:10 AM, Evgeniya Stepanova wrote:<br>
</div>
<blockquote cite="mid:551AAAD1.90301@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html;
charset=UTF-8">
Hi all,<br>
<br>
Could you please review fix for 8026043? It is a regression
test for 8000831: Heap verification output
incorrect/incomplete. Test check that gc log contains
expected information<br>
Originally patch was performed by Filipp Zhinkin in [1] ,
but review process was never finished.<br>
Test adapted to use current testlibrary functionality.<br>
<br>
bug: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8026043">https://bugs.openjdk.java.net/browse/JDK-8026043</a><br>
webrev: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Eeistepan/8026043/webrev.00/">http://cr.openjdk.java.net/~eistepan/8026043/webrev.00/</a><br>
[1] <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2013-October/008772.html">http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2013-October/008772.html</a><br>
<br>
Thanks,<br>
Jane<br>
<div class="moz-signature"><br>
</div>
</blockquote>
<br>
</blockquote>
<br>
<div class="moz-signature">-- <br>
<i>Evgeniya Stepanova</i></div>
</blockquote>
<br>
</blockquote>
<br>
<div class="moz-signature">-- <br>
<i>Evgeniya Stepanova</i></div>
</body>
</html>