Re: RFR: 8283627: Outdated comment in MachineDescriptionTwosComplement.isLP64

2022-08-27 Thread Chris Plummer
On Mon, 22 Aug 2022 15:06:31 GMT, Daniel Skantz wrote: > Update MachineDescriptionTwosComplement.java comment to indicate that LP64 > machines are no longer uncommon. How about this one in MachineDescription.java: /** Indicates whether the underlying machine supports the LP64 data mod

Re: RFR: JDK-8292595: jdwp utf_util getWideString might leak memory [v2]

2022-08-27 Thread Alan Bateman
On Fri, 26 Aug 2022 12:04:57 GMT, Matthias Baesken wrote: >> There seems to be a case where utf_util.c getWideString might leak memory in >> an early return. > > Matthias Baesken has updated the pull request incrementally with one > additional commit since the last revision: > > Introduce UT

RFR: 8283627: Outdated comment in MachineDescriptionTwosComplement.isLP64

2022-08-27 Thread Daniel Skantz
Update MachineDescriptionTwosComplement.java comment to indicate that LP64 machines are no longer uncommon. - Commit messages: - JDK-8283627 Changes: https://git.openjdk.org/jdk/pull/9967/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9967&range=00 Issue: https://bugs.op