Re: RFR: 8266783: java/lang/reflect/Proxy/DefaultMethods.java fails with jtreg 6

2021-05-11 Thread Iris Clark
On Tue, 11 May 2021 16:48:28 GMT, Mandy Chung wrote: > Data provider with varargs does not work on TestNG 7.4.0. Fix the test to > create an object array instead of varargs to workaround this issue. Marked as reviewed by iris (Reviewer). - PR:

Re: RFR: 8266783: java/lang/reflect/Proxy/DefaultMethods.java fails with jtreg 6

2021-05-11 Thread Mandy Chung
On Tue, 11 May 2021 16:48:28 GMT, Mandy Chung wrote: > Data provider with varargs does not work on TestNG 7.4.0. Fix the test to > create an object array instead of varargs to workaround this issue. Yes, fixing these is indeed looking more comfortable  - PR:

Re: RFR: 8266783: java\lang\reflect\Proxy\DefaultMethods.java fails with jtreg 6

2021-05-11 Thread Mark Reinhold
On Tue, 11 May 2021 16:48:28 GMT, Mandy Chung wrote: > Data provider with varargs does not work on TestNG 7.4.0. Fix the test to > create an object array instead of varargs to workaround this issue. All those backslashes hurt my eyes. Could we please use forward slashes, both in the commit

RFR: 8266783: java\lang\reflect\Proxy\DefaultMethods.java fails with jtreg 6

2021-05-11 Thread Mandy Chung
Data provider with varargs does not work on TestNG 7.4.0. Fix the test to create an object array instead of varargs to workaround this issue. - Commit messages: - 8266783: java\lang\reflect\Proxy\DefaultMethods.java fails with jtreg 6 Changes: