Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags [v3]

2023-11-16 Thread Leonid Mesnik
On Thu, 16 Nov 2023 18:20:04 GMT, Mandy Chung wrote: >> This PR includes test fixes for the following issues: >> >> 8319567: Update java/lang/invoke tests to support vm flags >> 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java >> to accept vm flags >> 8319672: Several

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags [v2]

2023-11-16 Thread Mandy Chung
On Thu, 16 Nov 2023 18:09:20 GMT, Leonid Mesnik wrote: >> Mandy Chung has updated the pull request incrementally with one additional >> commit since the last revision: >> >> review feedback > > test/jdk/jdk/modules/incubator/DefaultImage.java line 113: > >> 111: PrintStream ps = new

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags [v3]

2023-11-16 Thread Mandy Chung
> This PR includes test fixes for the following issues: > > 8319567: Update java/lang/invoke tests to support vm flags > 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java > to accept vm flags > 8319672: Several classloader tests ignore VM flags > 8319676: A couple of jdk

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags [v2]

2023-11-16 Thread Leonid Mesnik
On Wed, 15 Nov 2023 02:39:56 GMT, Mandy Chung wrote: >> This PR includes test fixes for the following issues: >> >> 8319567: Update java/lang/invoke tests to support vm flags >> 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java >> to accept vm flags >> 8319672: Several

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags [v2]

2023-11-15 Thread Jorn Vernee
On Wed, 15 Nov 2023 02:39:56 GMT, Mandy Chung wrote: >> This PR includes test fixes for the following issues: >> >> 8319567: Update java/lang/invoke tests to support vm flags >> 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java >> to accept vm flags >> 8319672: Several

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags [v2]

2023-11-14 Thread Mandy Chung
> This PR includes test fixes for the following issues: > > 8319567: Update java/lang/invoke tests to support vm flags > 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java > to accept vm flags > 8319672: Several classloader tests ignore VM flags > 8319676: A couple of jdk

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags [v2]

2023-11-14 Thread Mandy Chung
On Wed, 15 Nov 2023 00:17:56 GMT, Jorn Vernee wrote: >> Mandy Chung has updated the pull request incrementally with one additional >> commit since the last revision: >> >> review feedback > > test/jdk/java/lang/ClassLoader/getResource/GetResource.java line 166: > >> 164: Map env = pb

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags

2023-11-14 Thread Mandy Chung
On Wed, 15 Nov 2023 00:31:24 GMT, Jorn Vernee wrote: >> This PR includes test fixes for the following issues: >> >> 8319567: Update java/lang/invoke tests to support vm flags >> 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java >> to accept vm flags >> 8319672: Several

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags

2023-11-14 Thread Mandy Chung
On Wed, 15 Nov 2023 00:45:45 GMT, Jorn Vernee wrote: >> This PR includes test fixes for the following issues: >> >> 8319567: Update java/lang/invoke tests to support vm flags >> 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java >> to accept vm flags >> 8319672: Several

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags

2023-11-14 Thread Jorn Vernee
On Tue, 14 Nov 2023 22:45:56 GMT, Mandy Chung wrote: > This PR includes test fixes for the following issues: > > 8319567: Update java/lang/invoke tests to support vm flags > 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java > to accept vm flags > 8319672: Several class

Re: RFR: 8319567: Update java/lang/invoke tests to support vm flags

2023-11-14 Thread Jorn Vernee
On Tue, 14 Nov 2023 22:45:56 GMT, Mandy Chung wrote: > This PR includes test fixes for the following issues: > > 8319567: Update java/lang/invoke tests to support vm flags > 8319568: Update java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java > to accept vm flags > 8319672: Several class