RFR: 8345687: Improve the implementation of SegmentFactories::allocateSegment [v6]

Quan Anh Mai qamai at openjdk.org
Thu Mar 13 09:07:58 UTC 2025


On Wed, 12 Mar 2025 11:15:29 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains nine additional commits since the last revision:
>> 
>>  - Merge branch 'master' into segmentallocate
>>  - revert changes to CallocArena
>>  - Merge branch 'master' into segmentallocate
>>  - copyright
>>  - Merge branch 'master' into segmentallocate
>>  - wrong init
>>  - move segment instance creation to SegmentFactories
>>  - address review
>>  - improve the implementation of SegmentFactories::allocateSegment
>
> test/micro/org/openjdk/bench/java/lang/foreign/AllocTest.java line 26:
> 
>> 24: package org.openjdk.bench.java.lang.foreign;
>> 25: 
>> 26: import java.lang.foreign.*;
> 
> I believe it would be better to leave the imports alone -- as all the other benchmarks also use the "more explicit" style.

I have expanded the import section, please reapprove this PR, thanks a lot.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22610#discussion_r1993069941


More information about the core-libs-dev mailing list