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