compiler warnings about cfish HOT 10 CLOSED

Did you update your MSYS installation after #29 ?

It seems MSYS/MinGW doesn't know what it wants.

from cfish.

Ah, I thought these warning were caused by the changes I made then, but that's not true.

from cfish.

The problem is that fixing this by using the %z modifier for both Linux and Windows breaks cross-compilation for Windows with MinGW on Linux. (I could just cast the size_t values to int and use %d to work around the problem, but there is an annoying inconsistency in MinGW.)

from cfish.

Adding a compilation flag may have done the trick. At least it works now for MinGW on Linux.
1e615f5

from cfish.

nope, it's not fixed and now I get additional warnings.....

a) benchmark.c:209:35: warning: unknown conversion type character 'z' in format [-Wformat=]
fprintf(stderr, "\nPosition: %zu/%zu\n", ++j, num_fens - num_opts);

b) benchmark.c:209:21: warning: too many arguments for format [-Wformat-extra-args]
fprintf(stderr, "\nPosition: %zu/%zu\n", ++j, num_fens - num_opts);

c) benchmark.c:223:19: warning: ISO C does not support the 'I64' ms_printf length modifier [-Wformat=]

from cfish.

Do you get those warnings with COMP=mingw or with the (default) COMP=gcc?

I suppose you're also getting warnings in at least search,c?

from cfish.

COMP=gcc (7.3.0)
I get warnings in:
benchmark.c
position.c
search.c
tt.c

from cfish.

OK, please try again with the latest commit.

from cfish.

works fine, no warnings anymore........but final executable increased size from 196k to 221k

from cfish.

Yes, because it now links in a standard-compliant version of printf.
Without it you need to use %I, but then the compiler complains.

from cfish.