Re: RFR (trivial) 8249940: Remove unnecessary includes of jni_util.h in native tests

2020-07-22 Thread David Holmes
Thanks Mandy! David On 23/07/2020 2:22 pm, Mandy Chung wrote: Hi David, Looks good. Mandy On 7/22/20 4:00 PM, David Holmes wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8249940 webrev: http://cr.openjdk.java.net/~dholmes/8249940/webrev/ A number of native tests in hotspot and jdk in

Re: RFR (trivial) 8249940: Remove unnecessary includes of jni_util.h in native tests

2020-07-22 Thread Mandy Chung
Hi David, Looks good. Mandy On 7/22/20 4:00 PM, David Holmes wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8249940 webrev: http://cr.openjdk.java.net/~dholmes/8249940/webrev/ A number of native tests in hotspot and jdk include the jni_util.h header file which is part of the sources fo

Re: RFR (trivial) 8249940: Remove unnecessary includes of jni_util.h in native tests

2020-07-22 Thread David Holmes
Thanks Igor! David On 23/07/2020 9:09 am, Igor Ignatyev wrote: Hi David, looks good to me. -- Igor On Jul 22, 2020, at 4:00 PM, David Holmes wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8249940 webrev: http://cr.openjdk.java.net/~dholmes/8249940/webrev/ A number of native tests i

Re: RFR (trivial) 8249940: Remove unnecessary includes of jni_util.h in native tests

2020-07-22 Thread Igor Ignatyev
Hi David, looks good to me. -- Igor > On Jul 22, 2020, at 4:00 PM, David Holmes wrote: > > Bug: https://bugs.openjdk.java.net/browse/JDK-8249940 > webrev: http://cr.openjdk.java.net/~dholmes/8249940/webrev/ > > A number of native tests in hotspot and jdk include the jni_util.h header > file

RFR (trivial) 8249940: Remove unnecessary includes of jni_util.h in native tests

2020-07-22 Thread David Holmes
Bug: https://bugs.openjdk.java.net/browse/JDK-8249940 webrev: http://cr.openjdk.java.net/~dholmes/8249940/webrev/ A number of native tests in hotspot and jdk include the jni_util.h header file which is part of the sources for libjava and not part of the testing framework, nor an exported interf