RFR: 8338728: Misc issues in memory layout javadoc [v2]

Paul Sandoz psandoz at openjdk.org
Mon Aug 26 22:44:03 UTC 2024


On Wed, 21 Aug 2024 13:24:15 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> This PR fixes two minor issues in the `MemoryLayout` javadoc:
>> * the section describing dereference path talks about `P` and `P'` but then only uses `P` in the code;
>> * the `ceilDiv` math on the `PathElement::sequenceElement(long, long)` is wrong, as the division returns a negative number for F < 0, which is incorrect
>
> Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   * Clarify javadoc for var handles with dereference path elements
>   * Add test where dereference path element is last element in path

Marked as reviewed by psandoz (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/20659#pullrequestreview-2261775558


More information about the core-libs-dev mailing list