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

[Kim Barrett]

> On Sep 10, 2020, at 10:08 AM, Kim Barrett <kbarrett at openjdk.java.net> wrote:
> 
> On Wed, 9 Sep 2020 12:49:56 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:
>> 
>> src/hotspot/share/gc/g1/g1ParScanThreadState.cpp line 68:
>> 
>>> 66:     _old_gen_is_full(false),
>>> 67:     _partial_objarray_chunk_size(ParGCArrayScanChunk),
>>> 68:     _partial_array_stepper(n_workers),
>> 
>> What do you think about saving the chunk size in the stepper instead? Then we don't need to pass it in to `start()` and
>> `next()`. To avoid needing it for the call to `oop_iterate_range()` we could instead have the `Step` include both the
>> start and end index.
> 
> See above discussion about naming and factoring.  The same stepper can support multiple array types if the chunk size
> is external to the stepper.

Well, that discussion is only “above" in the github UI tool.

Now that I see what happens when I reply there, maybe I won’t do that so much.
That’s seriously chatty and massively quoting redundant.