Re: RFR: 8330815: Use pattern matching for instanceof in KeepAliveCache [v2]

2024-04-23 Thread Daniel Jeliński
On Tue, 23 Apr 2024 10:49:42 GMT, Christoph Langer wrote: >> In su.net.www.http.KeepAliveCache we could use pattern matching for >> instanceof. > > Christoph Langer has updated the pull request incrementally with one > additional commit since the last revision: > > Add Daniel's more awesome

Re: RFR: 8330815: Use pattern matching for instanceof in KeepAliveCache [v2]

2024-04-23 Thread Christoph Langer
On Tue, 23 Apr 2024 08:54:17 GMT, Daniel Jeliński wrote: > I'd prefer this instead: > > ``` > if (!(obj instanceof KeepAliveKey kae)) > return false; > ``` > > because it's more awesome :) but what you propose looks good too. Yes, you're right. Adapted. 😄 - PR

Re: RFR: 8330815: Use pattern matching for instanceof in KeepAliveCache [v2]

2024-04-23 Thread Christoph Langer
> In su.net.www.http.KeepAliveCache we could use pattern matching for > instanceof. Christoph Langer has updated the pull request incrementally with one additional commit since the last revision: Add Daniel's more awesome suggestion - Changes: - all: https://git.openjdk.org/jd

Re: RFR: 8330815: Use pattern matching for instanceof in KeepAliveCache

2024-04-23 Thread Daniel Jeliński
On Mon, 22 Apr 2024 11:23:59 GMT, Christoph Langer wrote: > In su.net.www.http.KeepAliveCache we could use pattern matching for > instanceof. I'd prefer this instead: if (!(obj instanceof KeepAliveKey kae)) return false; because it's more awesome :) but what you propose lo

Re: RFR: 8330815: Use pattern matching for instanceof in KeepAliveCache

2024-04-23 Thread Jaikiran Pai
On Mon, 22 Apr 2024 11:23:59 GMT, Christoph Langer wrote: > In su.net.www.http.KeepAliveCache we could use pattern matching for > instanceof. Hello Christoph, this cleanup change looks good to me. - Marked as reviewed by jpai (Reviewer). PR Review: https://git.openjdk.org/jdk/pul