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

2024-08-26 Thread Paul Sandoz
On Wed, 21 Aug 2024 13:24:15 GMT, Maurizio Cimadamore 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

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

2024-08-23 Thread Per Minborg
On Wed, 21 Aug 2024 13:24:15 GMT, Maurizio Cimadamore 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

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

2024-08-21 Thread Maurizio Cimadamore
> 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

RFR: 8338728: Misc issues in memory layout javadoc

2024-08-21 Thread Maurizio Cimadamore
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