Re: RFR: 8286858: Remove dead code in sun.reflect.misc.MethodUtil

2022-05-20 Thread Andrey Turbanov
On Wed, 18 May 2022 02:03:34 GMT, Mandy Chung wrote: >Do you know why there are test failures (they look unrelated)? All failed tests seems Loom-integration related. They fail only x86. Seems they are fixed in master under [JDK-8286476 ](https://bugs.openjdk.java.net/browse/JDK-8286476) -

Re: RFR: 8286858: Remove dead code in sun.reflect.misc.MethodUtil

2022-05-17 Thread Iris Clark
On Sun, 15 May 2022 12:47:10 GMT, Andrey Turbanov wrote: > They are unused and leftover since JDK 7. It's good to remove them. Nice clean up. - Marked as reviewed by iris (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/8715

Re: RFR: 8286858: Remove dead code in sun.reflect.misc.MethodUtil

2022-05-17 Thread Mandy Chung
On Sun, 15 May 2022 12:47:10 GMT, Andrey Turbanov wrote: > They are unused and leftover since JDK 7. It's good to remove them. Looks good. Do you know why there are test failures (they look unrelated)? - Marked as reviewed by mchung (Reviewer). PR: https://git.openjdk.java.net/

RFR: 8286858: Remove dead code in sun.reflect.misc.MethodUtil

2022-05-17 Thread Andrey Turbanov
They are unused and leftover since JDK 7. It's good to remove them. - Commit messages: - [PATCH] Remove unused code in sun.reflect.misc.MethodUtil Changes: https://git.openjdk.java.net/jdk/pull/8715/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8715&range=00 Issue: