Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-09 Thread Stuart Marks
On Tue, 9 Mar 2021 19:51:42 GMT, Stuart Marks wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Respond to review feedback. > > src/java.base/share/classes/java/lang/reflect/Method.java line 579: > >> 577: *

Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-09 Thread Rémi Forax
On Tue, 9 Mar 2021 19:13:24 GMT, Joe Darcy wrote: >> src/java.base/share/classes/java/lang/reflect/Method.java line 589: >> >>> 587: * different return type, the virtual machine does not. >>> 588: * A >>> 589: * common case where covariant overrides are used is for a {@link >>

Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-09 Thread Joe Darcy
On 3/9/2021 12:03 PM, Stuart Marks wrote: On Tue, 9 Mar 2021 19:13:24 GMT, Joe Darcy wrote: src/java.base/share/classes/java/lang/reflect/Method.java line 589: 587: * different return type, the virtual machine does not. 588: * A 589: * common case where covariant overrides

Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-09 Thread Stuart Marks
On Tue, 9 Mar 2021 03:27:29 GMT, Joe Darcy wrote: >> The existing documentation of Method.isBridge isn't terribly helpful to the >> reader. This RFE proposes to given a common example of how bridge methods >> are used. The JLS does *not* have a section discussing bridge methods in >> detail;

Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-09 Thread Stuart Marks
On Tue, 9 Mar 2021 19:13:24 GMT, Joe Darcy wrote: >> src/java.base/share/classes/java/lang/reflect/Method.java line 589: >> >>> 587: * different return type, the virtual machine does not. >>> 588: * A >>> 589: * common case where covariant overrides are used is for a {@link >>

Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-09 Thread Stuart Marks
On Tue, 9 Mar 2021 03:27:29 GMT, Joe Darcy wrote: >> The existing documentation of Method.isBridge isn't terribly helpful to the >> reader. This RFE proposes to given a common example of how bridge methods >> are used. The JLS does *not* have a section discussing bridge methods in >> detail;

Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-09 Thread Joe Darcy
On Tue, 9 Mar 2021 07:19:27 GMT, Rémi Forax wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Respond to review feedback. > > src/java.base/share/classes/java/lang/reflect/Method.java line 589: > >> 587: *

Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-08 Thread Rémi Forax
On Tue, 9 Mar 2021 03:27:29 GMT, Joe Darcy wrote: >> The existing documentation of Method.isBridge isn't terribly helpful to the >> reader. This RFE proposes to given a common example of how bridge methods >> are used. The JLS does *not* have a section discussing bridge methods in >> detail;

Re: RFR: 8263102: Expand documention of Method.isBridge [v3]

2021-03-08 Thread Joe Darcy
> The existing documentation of Method.isBridge isn't terribly helpful to the > reader. This RFE proposes to given a common example of how bridge methods are > used. The JLS does *not* have a section discussing bridge methods in detail; > bridge methods are a compilation technique for lowering