<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
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 class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~dfazunen/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>
</body>
</html>