RFR(XXS): 8214108: Incorrect Function parameter lists in some jtreg tests

[Gary Adams]

Looks OK to me.

It appears that there were problems in the past with ex03t001
using different GC implementations.

I'd suggest filing a new bug to track moving the test off the ProblemList.
It could be combined with restoring a number of other tests in the
next release timeframe.

On 11/20/18, 9:41 AM, Simon Tooke wrote:
> Hello,
>
> I'm not sure if this belongs in the serviceability group, but the tests
> are for jvmti, so it seemed appropriate.
> Please point me elsewhere if not.
>
> Two tests define a ClassUnload() callback function with an incorrect
> parameter,"jvmtiEnv jvmti_env", instead of "jvmtiEnv* jvmti_env".  
>
> This will lead to an invalid cast warning on some compilers, but
> currently has no impact on the tests as ex03t001 is disabled, and the
> body of the other ClassUnload() is empty. These issues were found by
> compiling with GCC 8.1.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8214108
> webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/stooke/JDK-8214108/01/webrev/
>
> For what it's worth, ex03t001 does pass if enabled, but I don't know if the test is correct at this point; it's been disabled for a while.
>
> Please let me know what you think,
>
> Thanks,
> -Simon
>