RFR: 8308410: broken compilation of test\jdk\tools\launcher\exeJliLaunchTest.c

[Adam Sotona]

On Fri, 19 May 2023 12:35:47 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> JDK-8303669 patch to test\jdk\tools\launcher\exeJliLaunchTest.c broke compilation on windows.
>> Unfortunately MSVC does not support variable length arrays.
>> This patch fixes test\jdk\tools\launcher\exeJliLaunchTest.c to use dynamic array allocation.
>> 
>> Please review.
>> 
>> Thanks,
>> Adam
>
> test/jdk/tools/launcher/exeJliLaunchTest.c line 39:
> 
>> 37: {
>> 38:     //avoid null-terminated array of arguments to test JDK-8303669
>> 39:     char **argv = malloc(sizeof(char *) * argc);
> 
> If this is building on all platforms then okay but I would have expected you'd need a cast here.

I've learned that there are completely opposite rules for C and C++.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14060#discussion_r1198933225