<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Misha!<br>
<br>
Thanks for the comments! I'll fix as you suggested and test before
push.<br>
<br>
-- Dima<br>
<br>
<div class="moz-cite-prefix">On 29.02.2016 18:24, Michail Chernov
wrote:<br>
</div>
<blockquote cite="mid:56D462C8.6020503@oracle.com" type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
Hi Dima,<br>
<br>
Test looks good to me.<br>
<br>
Some minor issues:<br>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
1. " * @library /testlibrary " - /testlibrary is not necessary,
test does not depend on any test library classes.<br>
2. " * @modules java.base/sun.misc" - is not necessary too.<br>
<br>
No extra webrev is required.<br>
<br>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
Thanks<br>
Michail<br>
<br>
<br>
<div class="moz-cite-prefix">On 24.02.2016 21:47, Dmitry Fazunenko
wrote:<br>
</div>
<blockquote cite="mid:56CDFABD.6030206@oracle.com" type="cite">Hi
Thomas, <br>
<br>
The new version where all your comments addressed: <br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Edfazunen/8145908/webrev.03/">http://cr.openjdk.java.net/~dfazunen/8145908/webrev.03/</a>
<br>
<br>
To clarify the idea of usage a window within STORAGE I added the
following comment: <br>
<br>
// To cause Remembered Sets change their coarse level the
test uses a window <br>
// within STORAGE. All the BUFFER elements refer to only
STORAGE objects <br>
// from the current window. The window is defined by a
range. <br>
// The first element has got the index: 'windowStart', <br>
// the last one: 'windowStart + windowSize - 1' <br>
// The window is shifting periodically. <br>
<br>
Thanks, <br>
Dima <br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>