Re: RFR: JDK-8280168: Add Objects.toIdentityString [v7]

2022-01-25 Thread Roger Riggs
On Mon, 24 Jan 2022 21:31:37 GMT, Joe Darcy wrote: >> While it is strongly recommend to not use the default toString for a class, >> at times it is the least-bad alternative. When that alternative needs to be >> used, it would be helpful to have the implementation already available, such >> as

Re: RFR: JDK-8280168: Add Objects.toIdentityString [v7]

2022-01-25 Thread Alan Bateman
On Mon, 24 Jan 2022 21:31:37 GMT, Joe Darcy wrote: >> While it is strongly recommend to not use the default toString for a class, >> at times it is the least-bad alternative. When that alternative needs to be >> used, it would be helpful to have the implementation already available, such >> as

Re: RFR: JDK-8280168: Add Objects.toIdentityString [v7]

2022-01-24 Thread Mandy Chung
On Mon, 24 Jan 2022 21:31:37 GMT, Joe Darcy wrote: >> While it is strongly recommend to not use the default toString for a class, >> at times it is the least-bad alternative. When that alternative needs to be >> used, it would be helpful to have the implementation already available, such >> as

Re: RFR: JDK-8280168: Add Objects.toIdentityString [v7]

2022-01-24 Thread Stuart Marks
On Mon, 24 Jan 2022 21:31:37 GMT, Joe Darcy wrote: >> While it is strongly recommend to not use the default toString for a class, >> at times it is the least-bad alternative. When that alternative needs to be >> used, it would be helpful to have the implementation already available, such >> as

Re: RFR: JDK-8280168: Add Objects.toIdentityString [v7]

2022-01-24 Thread Joe Darcy
On Mon, 24 Jan 2022 21:51:13 GMT, Rémi Forax wrote: > toIdentityString is a better name than toDefaultString. > > It's fine for me but given that "identity" has a slightly different meaning > in the context of Valhalla that in System.identityHashCode(), it may be good > to ask Brian about that

Re: RFR: JDK-8280168: Add Objects.toIdentityString [v7]

2022-01-24 Thread Rémi Forax
On Mon, 24 Jan 2022 21:31:37 GMT, Joe Darcy wrote: >> While it is strongly recommend to not use the default toString for a class, >> at times it is the least-bad alternative. When that alternative needs to be >> used, it would be helpful to have the implementation already available, such >> as

Re: RFR: JDK-8280168: Add Objects.toIdentityString [v7]

2022-01-24 Thread Joe Darcy
> While it is strongly recommend to not use the default toString for a class, > at times it is the least-bad alternative. When that alternative needs to be > used, it would be helpful to have the implementation already available, such > as in Objects.toDefaultString(). This method is analagous t