<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Hi Joe,<br>
<br>
On 2015-12-09 19:31, Joseph Provino wrote:<br>
</div>
<blockquote cite="mid:5668739E.20202@oracle.com" type="cite">Please
review this very small change. It's removal of unneeded code.
<br>
<br>
CR: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8139871">https://bugs.openjdk.java.net/browse/JDK-8139871</a>
<br>
<br>
Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~jprovino/8139871/webrev.00">http://cr.openjdk.java.net/~jprovino/8139871/webrev.00</a>
<br>
</blockquote>
<br>
The mark_init_elapsed_time_ms parameter seems unused now:<br>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<pre> void G1CollectorPolicy::record_concurrent_mark_init_end(double
mark_init_elapsed_time_ms) {
collector_state()->set_during_marking(true);
assert(!collector_state()->initiate_conc_mark_if_possible(), "we should have cleared it by now");
collector_state()->set_during_initial_mark_pause(false);
<span class="removed">- _cur_mark_stop_world_time_ms = mark_init_elapsed_time_ms;</span>
}
Otherwise, this looks good.
</pre>
<br>
For future reviews, could you use the official webrev script at:<br>
<a class="moz-txt-link-freetext" href="http://hg.openjdk.java.net/code-tools/webrev/">http://hg.openjdk.java.net/code-tools/webrev/</a><br>
<br>
The udiffs view is broken in the version your are using.<br>
<br>
Thanks,<br>
StefanK<br>
<blockquote cite="mid:5668739E.20202@oracle.com" type="cite">
<br>
joe
<br>
</blockquote>
<br>
</body>
</html>