On Sun, 18 May 2025 15:36:03 GMT, Guoxiong Li <g...@openjdk.org> wrote:

>> Albert Mingkun Yang has updated the pull request with a new target base due 
>> to a merge or a rebase. The pull request now contains five commits:
>> 
>>  - review
>>  - Merge branch 'master' into pgc-size-policy
>>  - review
>>  - Merge branch 'master' into pgc-size-policy
>>  - pgc-size-policy
>
> src/hotspot/share/gc/parallel/psYoungGen.cpp line 268:
> 
>> 266:   size_t original_committed_size = virtual_space()->committed_size();
>> 267: 
>> 268:   while (true) {
> 
> The `while` statement only runs once. May we find a better way to refactor 
> the code?

@lgxbslgx I did some restructuring based on some offline discussion -- 
hopefully, the new style is more readable.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25000#discussion_r2154053199

Reply via email to