build: Windows 10 + msys2 (with winiconv): passing argument 2 of 'iconv' from incompatible pointer type

[vtorri]

build: Windows 10 + msys2 (with winiconv): passing argument 2 of 'iconv' from incompatible pointer type

[TheOneric]

char ** or more specifically char **restrict is the correct type; your iconv headers are broken
See: https://pubs.opengroup.org/onlinepubs/9699919799/functions/iconv.html

(there used to be an error in the spec where the version documented in the iconv.h header didn’t match the version of the iconv’ function itself, but that was fixed ages ago. Some systems like NetBSD retain a const version anyway, but i guess they also configure their compilers to not error on this by default)

win-iconv defaults to a non-compliant setup and you need a build time toggle to make it behave correctly; win-iconv/win-iconv#25

Furthermore sub_recode wasn’t touched in 10 years; this exact setup working fine with 0.17.1 but not 0.17.2 is thus impossible; most likely the winiconv macro define got just lost inbetween or your compiler used to treat this as a non-fatal error before

[vtorri]

actually, winiconv can modify the constness with WINICONV_CONST, now it works.
maybe gcc 14 changed something, but 0.17.1 did compile when i tried it, as its compilation was part of my package manager. So I'm sure it compiled.
anyway, it is fixed, now