Re: RFR: 8298177: Various java.lang.invoke cleanups [v4]

2022-12-07 Thread Claes Redestad
On Wed, 7 Dec 2022 14:08:28 GMT, Claes Redestad wrote: >> Various code cleanups around java.lang.invoke code. Started out with dead >> code removal in `jli.MemberName`, then piled on to fix a set of minor >> inefficiencies (excessive vararg array allocations, unnecessary defensive >> cloning o

Re: RFR: 8298177: Various java.lang.invoke cleanups [v4]

2022-12-07 Thread Jorn Vernee
On Wed, 7 Dec 2022 14:08:28 GMT, Claes Redestad wrote: >> Various code cleanups around java.lang.invoke code. Started out with dead >> code removal in `jli.MemberName`, then piled on to fix a set of minor >> inefficiencies (excessive vararg array allocations, unnecessary defensive >> cloning o

Re: RFR: 8298177: Various java.lang.invoke cleanups [v4]

2022-12-07 Thread Claes Redestad
> Various code cleanups around java.lang.invoke code. Started out with dead > code removal in `jli.MemberName`, then piled on to fix a set of minor > inefficiencies (excessive vararg array allocations, unnecessary defensive > cloning of parameter arrays etc). Claes Redestad has updated the pull

Re: RFR: 8298177: Various java.lang.invoke cleanups [v3]

2022-12-06 Thread Jorn Vernee
On Tue, 6 Dec 2022 16:25:12 GMT, Claes Redestad wrote: >> src/java.base/share/classes/java/lang/invoke/MemberName.java line 578: >> >>> 576: throw new LinkageError(m.toString()); >>> 577: } >>> 578: assert(isResolved()); >> >> I don't see why this can be removed. Can

Re: RFR: 8298177: Various java.lang.invoke cleanups [v3]

2022-12-06 Thread Jorn Vernee
On Tue, 6 Dec 2022 18:06:36 GMT, Claes Redestad wrote: >> Various code cleanups around java.lang.invoke code. Started out with dead >> code removal in `jli.MemberName`, then piled on to fix a set of minor >> inefficiencies (excessive vararg array allocations, unnecessary defensive >> cloning o

Re: RFR: 8298177: Various java.lang.invoke cleanups [v3]

2022-12-06 Thread Claes Redestad
> Various code cleanups around java.lang.invoke code. Started out with dead > code removal in `jli.MemberName`, then piled on to fix a set of minor > inefficiencies (excessive vararg array allocations, unnecessary defensive > cloning of parameter arrays etc). Claes Redestad has updated the pull

Re: RFR: 8298177: Various java.lang.invoke cleanups [v2]

2022-12-06 Thread Claes Redestad
> Various code cleanups around java.lang.invoke code. Started out with dead > code removal in `jli.MemberName`, then piled on to fix a set of minor > inefficiencies (excessive vararg array allocations, unnecessary defensive > cloning of parameter arrays etc). Claes Redestad has updated the pull

Re: RFR: 8298177: Various java.lang.invoke cleanups

2022-12-06 Thread Claes Redestad
On Tue, 6 Dec 2022 16:13:12 GMT, Jorn Vernee wrote: >> Various code cleanups around java.lang.invoke code. Started out with dead >> code removal in `jli.MemberName`, then piled on to fix a set of minor >> inefficiencies (excessive vararg array allocations, unnecessary defensive >> cloning of p

Re: RFR: 8298177: Various java.lang.invoke cleanups

2022-12-06 Thread Jorn Vernee
On Tue, 6 Dec 2022 15:22:27 GMT, Claes Redestad wrote: > Various code cleanups around java.lang.invoke code. Started out with dead > code removal in `jli.MemberName`, then piled on to fix a set of minor > inefficiencies (excessive vararg array allocations, unnecessary defensive > cloning of pa

RFR: 8298177: Various java.lang.invoke cleanups

2022-12-06 Thread Claes Redestad
Various code cleanups around java.lang.invoke code. Started out with dead code removal in `jli.MemberName`, then piled on to fix a set of minor inefficiencies (excessive vararg array allocations, unnecessary defensive cloning of parameter arrays etc). - Commit messages: - Restore