<html>

  <head>

    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    Hi Thomas,<br>

    <br>

    Thanks for the review!<br>

    I will fix all the mistakes you've pointed to<br>

    <br>

    Jane<br>

    <br>

    <div class="moz-cite-prefix">On 23.04.2015 13:29, Thomas Schatzl

      wrote:<br>

    </div>

    <blockquote cite="mid:1429784970.3316.49.camel@oracle.com"

      type="cite">

      <pre wrap="">Hi,

On Mon, 2015-04-20 at 16:27 +0300, Evgeniya Stepanova wrote:

</pre>

      <blockquote type="cite">

        <pre wrap="">The webrev for the diff could be found at

<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~eistepan/6407976/webrev.01.diff/">http://cr.openjdk.java.net/~eistepan/6407976/webrev.01.diff/</a> 

But changes are visible only in patch files, since there are only

white-spaces 

changes.

</pre>

      </blockquote>

      <pre wrap="">

  - g1CollectedHeap.cpp, line 3582, the format specifier in the line

"st->print("%3d ", i)" should be %u

  - in g1CollectedHeap.cpp, in the constructor for

RebuildRSOutOfRegionClosure, the _worker_i member of the class can be

removed, it seems unused.

  - concurrentMarkSweepGeneration.cpp:3153, please align the comment to

the others

  - concurrentMarkSweepGeneration.cpp:3645, is it possible to fix the

indentation of the parameters passed to

AdaptiveSizePolicy::calc_active_conc_workers()

  - concurrentMark.cpp:2320-2323, g1CollectedHeap.cpp:5021-5025,

parNewGeneration.cpp:907-910, adaptiveSizePolicy.cpp:100-103,

adaptiveSizePolicy.cpp:212-216, adaptiveSizePolicy.hpp:346-350, 361-368

parameter indentation wrong.

Although you mentioned that the indentation/spacing problems were fixed

with this webrev, they seem to persist. The raw files uploaded as well

as the complete patchset file still do not contain the expected changes.

Please make sure that the local webrev output contains these changes

(delete any existing webrev folder in the place where the new webrev

folder will be located), and when uploading the webrev, also check

whether they were updated.

Thanks,

  Thomas

</pre>

    </blockquote>

    <br>

    <div class="moz-signature">-- <br>

      <i>Evgeniya Stepanova</i></div>

  </body>

</html>