<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Per,<br>
<br>
Currently we have<br>
<br>
<meta charset="utf-8">
<pre style="color: rgb(0, 0, 0); font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">3029 product(uintx, NewRatio, 2, \
3030 "Ratio of new/old generation sizes") \</pre>
<br>
it should be<br>
<br>
<meta charset="utf-8">
<pre style="color: rgb(0, 0, 0); font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">3029 product(uintx, NewRatio, 2, \
3030 "Ratio of <b>old/new</b> generation sizes") \</pre>
<br>
Thanks.<br>
Tao<br>
<br>
<br>
On 9/24/13 3:05 AM, Per Liden wrote:
<blockquote cite="mid:524163E8.9090006@oracle.com" type="cite">Hi
Tao,
<br>
<br>
Thanks for reviewing.
<br>
<br>
NewRatio looks ok, are you maybe referring to SurvivorRatio?
<br>
<br>
/Per
<br>
<br>
On 2013-09-23 20:41, Tao Mao wrote:
<br>
<blockquote type="cite">Hi Per,
<br>
<br>
Looks good to me.
<br>
<br>
Please also note that the wording of NewRatio's definition has
the same problem. If interested, please create a new CR and fix
it.
<br>
<br>
Thanks.
<br>
Tao
<br>
<br>
On 9/23/13 5:18 AM, Per Liden wrote:
<br>
<blockquote type="cite">Hi,
<br>
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~pliden/8024632/webrev.01/">http://cr.openjdk.java.net/~pliden/8024632/webrev.01/</a>
<br>
<br>
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8024632">https://bugs.openjdk.java.net/browse/JDK-8024632</a>
<br>
<br>
Summary: Fix for the incorrect description of
InitialSurvivorRatio. The ratio should be described as "young
gen / survivor size", not "eden size / survivor size".
<br>
<br>
cheers,
<br>
/Per
<br>
</blockquote>
</blockquote>
<br>
</blockquote>
</body>
</html>