RFR: 8357915: SecureRandom nextLong memory usage [v2]
Bradford Wetmore
wetmore at openjdk.org
Thu Jul 3 21:05:40 UTC 2025
On Mon, 30 Jun 2025 15:20:33 GMT, Sean Mullan <mullan at openjdk.org> wrote:
>> This won't appear in the list of methods declared by `SecureRandom`. An analogy is [`ConcurrentHashMap::forEach(BiConsumer)`](https://docs.oracle.com/en/java/javase/24/docs/api/java.base/java/util/concurrent/ConcurrentHashMap.html#methods-inherited-from-class-java.util.concurrent.ConcurrentMap): this method is [overridden in `ConcurrentHashMap`](https://github.com/openjdk/jdk/blob/jdk24/src/java.base/share/classes/java/util/concurrent/ConcurrentHashMap.java#L1613) but is described by Javadoc as "Methods declared in interface java.util.concurrent.ConcurrentMap".
>>
>> Since this method won't appear, we don't need an `@inheritDoc` tag.
>
> I don't know, that seems less obvious to me, as I can't tell if the class is overriding the method or not.
`@inheritDoc` sounds good to me.
This is better than using `ByteBuffer.getLong()`, which uses the current byte ordering instead of big endian.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26005#discussion_r2183672363
More information about the security-dev
mailing list