Re: RFR: 8346609: Improve MemorySegment.toString [v6]

2025-01-07 Thread Maurizio Cimadamore
On Tue, 7 Jan 2025 12:58:12 GMT, Per Minborg wrote: >> This PR proposes to improve the current `MemorySegment.toString()` method >> from: >> >> `MemorySegment{ address: 0x6264c540, byteSize: 8 }` >> >> to >> >> `MemorySegment{ native, address: 0x6264c540, byteSize: 8}` >> >> Tests pa

Re: RFR: 8346609: Improve MemorySegment.toString [v6]

2025-01-07 Thread Per Minborg
> This PR proposes to improve the current `MemorySegment.toString()` method > from: > > `MemorySegment{ address: 0x6264c540, byteSize: 8 }` > > to > > `MemorySegment{ native, address: 0x6264c540, byteSize: 8}` > > Tests passes tier1-tier3 Per Minborg has updated the pull request incre

Re: RFR: 8346609: Improve MemorySegment.toString [v5]

2025-01-07 Thread Per Minborg
> This PR proposes to improve the current `MemorySegment.toString()` method > from: > > `MemorySegment{ address: 0x6264c540, byteSize: 8 }` > > to > > `MemorySegment{ native, address: 0x6264c540, byteSize: 8}` > > Tests passes tier1-tier3 Per Minborg has updated the pull request incre

Re: RFR: 8346609: Improve MemorySegment.toString [v4]

2025-01-07 Thread Per Minborg
> This PR proposes to improve the current `MemorySegment.toString()` method > from: > > `MemorySegment{ address: 0x6264c540, byteSize: 8 }` > > to > > `MemorySegment{ native, address: 0x6264c540, byteSize: 8}` > > Tests passes tier1-tier3 Per Minborg has updated the pull request incre

Re: RFR: 8346609: Improve MemorySegment.toString [v3]

2025-01-07 Thread Per Minborg
> This PR proposes to improve the current `MemorySegment.toString()` method > from: > > `MemorySegment{ address: 0x6264c540, byteSize: 8 }` > > to > > `MemorySegment{ native, address: 0x6264c540, byteSize: 8}` > > Tests passes tier1-tier3 Per Minborg has updated the pull request incre

Re: RFR: 8346609: Improve MemorySegment.toString [v2]

2025-01-07 Thread Maurizio Cimadamore
On Tue, 7 Jan 2025 11:30:13 GMT, Per Minborg wrote: >> This PR proposes to improve the current `MemorySegment.toString()` method >> from: >> >> `MemorySegment{ address: 0x6264c540, byteSize: 8 }` >> >> to >> >> `MemorySegment{ native, address: 0x6264c540, byteSize: 8}` >> >> Tests pa

Re: RFR: 8346609: Improve MemorySegment.toString [v2]

2025-01-07 Thread Per Minborg
> This PR proposes to improve the current `MemorySegment.toString()` method > from: > > `MemorySegment{ address: 0x6264c540, byteSize: 8 }` > > to > > `MemorySegment{ native, address: 0x6264c540, byteSize: 8}` > > Tests passes tier1-tier3 Per Minborg has updated the pull request with

Re: RFR: 8346609: Improve MemorySegment.toString

2024-12-20 Thread Maurizio Cimadamore
On Thu, 19 Dec 2024 17:41:29 GMT, Jorn Vernee wrote: > I would find it reasonable to add `isAlive=true/false` to that, but it's not > a property of the MS itself, but of the scope that it is attached to, so I'm > not sure. And even for liveness, the fact that this info could be updated concurr

Re: RFR: 8346609: Improve MemorySegment.toString

2024-12-19 Thread Jorn Vernee
On Thu, 19 Dec 2024 09:48:23 GMT, Per Minborg wrote: > This PR proposes to improve the current `MemorySegment.toString()` method > from: > > `MemorySegment{ address: 0x6264c540, byteSize: 8 }` > > to > > `MemorySegment{ native, address: 0x6264c540, byteSize: 8, confined, not > alive,

RFR: 8346609: Improve MemorySegment.toString

2024-12-19 Thread Per Minborg
This PR proposes to improve the current `MemorySegment.toString()` method from: `MemorySegment{ address: 0x6264c540, byteSize: 8 }` to `MemorySegment{ native, address: 0x6264c540, byteSize: 8, confined, not alive, read-only, mapped }` Tests passes tier1-tier3 - Commit mes