RFR: 8027545: Improve object array chunking test in G1's copy_to_survivor_space [v2]

[Thomas Schatzl]

Hi,

On 11.09.20 11:01, Kim Barrett wrote:
[...]
> Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:
> 
>    Respond to reviews by ayang and sjohanss.
 >
> Changes:
>    - all: https://git.openjdk.java.net/jdk/pull/90/files
>    - new: https://git.openjdk.java.net/jdk/pull/90/files/9397514e..1981c392
> 
> Webrevs:
>   - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=90&range=01
>   - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=90&range=00-01
> 
>    Stats: 152 lines in 6 files changed: 122 ins; 14 del; 16 mod
>    Patch: https://git.openjdk.java.net/jdk/pull/90.diff
>    Fetch: git fetch https://git.openjdk.java.net/jdk pull/90/head:pull/90
> 
> PR: https://git.openjdk.java.net/jdk/pull/90
> 

   - partialArrayTaskStepper.hpp:

s/_task_fannout/_task_fanout (or maybe _task_fan_out)

   - since _task_fannout and _task_limit are only assigned in the 
constructor of PartialArrayTaskStepper, maybe make them const?

   - also the asserts in PartialArrayTaskStepper::next_impl could be 
moved to the constructor initialization list but otoh the fannout > 0 
assert does make the calculation below immediately obviously safe.

Looks good otherwise.

Thanks,
   Thomas