<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix"><br>
Hi again,<br>
<br>
Christian has pushed his test library change and it has propagated
to the hotspot-gc repository. So, this change is ready to be
pushed. Just need some reviews. Any takers? :)<br>
<br>
Thanks,<br>
Bengt<br>
<br>
On 1/16/13 1:58 PM, Bengt Rutisson wrote:<br>
</div>
<blockquote cite="mid:50F6A3FC.8030000@oracle.com" type="cite">
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
<br>
Hi all,<br>
<br>
Could I have a couple of reviews for this change?<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ebrutisso/8006398/webrev.00/">http://cr.openjdk.java.net/~brutisso/8006398/webrev.00/</a><br>
<br>
Recently we deprecated some GC combinations. Those should now
print a warning at startup. Other GC combinations should not print
any warnings.<br>
<br>
With the new process handling support that Christian Törnqvist is
adding to the JTREG tests for hotspot it is very easy to write
test that start a VM and checks the output.<br>
<br>
This changes makes use of Christian's testlibrary to verify that
warnings are printed as expected.<br>
<br>
I'm also adding the "gc" keyword to JTREG to make it possible to
filter out GC tests. We should probably use this for all test in
the the /gc folder, but I think that should be done as a separate
change.<br>
<br>
The webrev above is based on Christian's webrev to add the
testlibrary:<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Ebrutisso/8006413/webrev.00/">http://cr.openjdk.java.net/~brutisso/8006413/webrev.00/</a><br>
<br>
Thanks,<br>
Bengt<br>
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
<br>
</blockquote>
<br>
</body>
</html>