RFR: 8323159: Consider adding some text re. memory zeroing in Arena::allocate [v2]

Jorn Vernee jvernee at openjdk.org
Fri Jan 12 11:23:22 UTC 2024


On Thu, 11 Jan 2024 07:59:37 GMT, Per Minborg <pminborg at openjdk.org> wrote:

>> This PR proposes to add a clarification that an `Arena` always returns zeroed-out segments for `Arena::allocate` methods.
>> 
>> Note that other overloaded methods refer to the abstract `Arena::allocate` method via implementation notes.
>
> Per Minborg has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Move and change text on zeroing out

src/java.base/share/classes/java/lang/foreign/Arena.java line 215:

> 213:  * {@linkplain #ofConfined()} and, {@linkplain #ofShared()} will return segments
> 214:  * that are zeroed out when invoking {@linkplain Arena#allocate(long, long) allocate()}.
> 215:  *

I'd expect to find this in the javadoc of each factory method (not in an implSpec). The note says something about those particular methods, not about the Arena type as a whole.

I also think "zero intialized" sounds a bit more professional than "zeroed out". I suggest the following phrasing: 

    Memory segments {@linkPlain #allocate(long, long) allocated} by the returned arena are zero initialized

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17308#discussion_r1450270020


More information about the core-libs-dev mailing list