Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-10 Thread Michael McMahon
On Tue, 9 Mar 2021 19:56:25 GMT, Patrick Concannon wrote: >> Hi, >> >> Could someone please review my code for updating the code in the `java.net` >> and `java.nio` packages to make use of the `instanceof` pattern variable? >> >> Kind regards, >> Patrick > > Patrick Concannon has updated the

Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-10 Thread Chris Hegarty
On Tue, 9 Mar 2021 20:52:19 GMT, Mark Sheppard wrote: >> The disadvantage is that it would add another level of braces. > > maybe ... I find it would provide an easier flow to code ... an "easy on the > eye" call flow linkage - the variable is declared and then used instead > of the disc

Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-10 Thread Chris Hegarty
On Tue, 9 Mar 2021 19:56:25 GMT, Patrick Concannon wrote: >> Hi, >> >> Could someone please review my code for updating the code in the `java.net` >> and `java.nio` packages to make use of the `instanceof` pattern variable? >> >> Kind regards, >> Patrick > > Patrick Concannon has updated the

Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-09 Thread Mark Sheppard
On Tue, 9 Mar 2021 20:26:19 GMT, Daniel Fuchs wrote: >> src/java.base/share/classes/java/net/NetMulticastSocket.java line 219: >> >>> 217: if (addr == null) >>> 218: addr = new InetSocketAddress(0); >>> 219: if (!(addr instanceof InetSocketAddress epoint)) >> >> in t

Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-09 Thread Daniel Fuchs
On Tue, 9 Mar 2021 19:56:25 GMT, Patrick Concannon wrote: >> Hi, >> >> Could someone please review my code for updating the code in the `java.net` >> and `java.nio` packages to make use of the `instanceof` pattern variable? >> >> Kind regards, >> Patrick > > Patrick Concannon has updated the

Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-09 Thread Daniel Fuchs
On Tue, 9 Mar 2021 20:07:14 GMT, Mark Sheppard wrote: >> Patrick Concannon has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8263233: Refactored equals method further > > src/java.base/share/classes/java/net/NetMulticastSocket.java line 21

Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-09 Thread Patrick Concannon
On Tue, 9 Mar 2021 18:22:17 GMT, Chris Hegarty wrote: >> Patrick Concannon has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8263233: Refactored equals method further > > src/java.base/share/classes/java/net/InterfaceAddress.java line 108:

Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-09 Thread Mark Sheppard
On Tue, 9 Mar 2021 19:56:25 GMT, Patrick Concannon wrote: >> Hi, >> >> Could someone please review my code for updating the code in the `java.net` >> and `java.nio` packages to make use of the `instanceof` pattern variable? >> >> Kind regards, >> Patrick > > Patrick Concannon has updated the

Re: RFR: 8263233: Update java.net and java.nio to use instanceof pattern variable [v3]

2021-03-09 Thread Patrick Concannon
> Hi, > > Could someone please review my code for updating the code in the `java.net` > and `java.nio` packages to make use of the `instanceof` pattern variable? > > Kind regards, > Patrick Patrick Concannon has updated the pull request incrementally with one additional commit since the last r