Hi Jc,
I'd prefer to see the fix splitted to smaller parts.
The changes are quite simple, but it's just too many files, so it's
impossible (at least for me) to review them accurately.
--alex
On 09/07/2018 10:53, JC Beyler wrote:
Hi all,
I'm taking the risk to perhaps do something too huge but as it is the
same as the previous ones (and the last one!):
It's 4k of changes, for that I apologize. I can divide it up in 2/3
parts again if you want or we can just do it in one go since now some
reviewers know what to expect. There still is a bit of refactoring to
the vmTestbase but this will in essence finish:
- Removing the JNI_ENV* and JVMTI_ENV* macros for the vmTestbase
Webrev: http://cr.openjdk.java.net/~jcbeyler/8210385/webrev.00/
<http://cr.openjdk.java.net/%7Ejcbeyler/8210385/webrev.00/>
Bug: https://bugs.openjdk.java.net/browse/JDK-8210385
Thanks for your reviews and let me know if you would prefer me to cut it
up like I did the Get[A-F] & Get[G-Z] ones,
Jc
Ps: for the curious, then we can start cleaning up the native code a bit
more sanely (example: JDK-8191519) and there will be one more webrev
removing #ifdef cplusplus lingering in the vmTestbase (JDK-8210481).
