Git Repository Merge Request #27: Replace array by pointer allocation for MSVC (rejected)

MSVC does not allow array allocation using const casted integers, so use pointer allocation instead.

[Ethan Merritt]

The patched code segfaults on my machine.
I can see it needs a sizeof(coordval) in the allocation.
Also needs a free() to avoid memory leak.

What version of MSVC is this?  I thought MSVC 2015 was c99 compliant. If it isn't, that's probably a problem elsewhere in the code as well.

[Alexander Täschner]

Corrected the allocation size and added the missing free().

This is the most current MSVC compiler (2022 v 17.6.5) which errors out at this newly added code. I found no other problems elsewhwere in the code (monthly runs at https://github.com/AlexanderTaeschner/gnuplot).

[Alexander Täschner]

Since you asked for C99 compliance: MSVC is complinat to C11 and C17, but there is no plan to support variable length arrays, used in this newly added code, due to security concerns (see https://devblogs.microsoft.com/cppblog/c11-and-c17-standard-support-arriving-in-msvc/#variable-length-arrays).

[Alexander Täschner]

Fixed by 6af1bda4 (Ethan Merritt) - thanks!