RFR: 8338535: GenShen: some style improvements to source code implementation [v3]
Kelvin Nilsen
kdnilsen at openjdk.org
Wed Aug 21 20:18:30 UTC 2024
On Tue, 20 Aug 2024 08:58:43 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Kelvin Nilsen has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Code touchup
>>
>> Remove 2 unneeded blank lines
>> Move and simplify a comment from call point to called method
>> implementation
>
> src/hotspot/share/gc/shenandoah/heuristics/shenandoahAdaptiveHeuristics.hpp line 73:
>
>> 71:
>> 72: virtual void choose_collection_set_from_regiondata(ShenandoahCollectionSet* cset,
>> 73: RegionData data[], size_t size,
>
> Not at all sure this is a great style. It is also not consistent with the rest of Hotspot: searching for `[],` or `[] ` in `.cpp` files yields no hits. Leave it be at `RegionData*`.
Will revert these changes.
-------------
PR Review Comment: https://git.openjdk.org/shenandoah/pull/475#discussion_r1725703307
More information about the shenandoah-dev
mailing list