__unreacahble() assumes MSVC on Windows

[GrimMaple]

It seems that the code below:

c-utils/include/utils/utils.h

Line 145 in c51a188

#define __unreachable() __assume(0)

assumes that, if on windows, we're building with MSVC. That isn't always the case, as the build of libosdp fails with __assume() begin undefined when I build with MSYS2 UCRT64 (clang)

[GrimMaple]

Possible fix is adding && defined(_MSC_VER) to the #if on top of it

[GrimMaple]

A similar issue is here:

c-utils/src/logger.c

Line 67 in c51a188

if (ctx->file && isatty(fileno(ctx->file))) {

Building from MSYS2 UCRT64 says isatty is undefined; expected to use _isatty from #include <io.h>

[sidcha]

@GrimMaple I do not have a way to test this easily. Can you please make these changes, test it locally and create a pull request?

[GrimMaple]

@GrimMaple I do not have a way to test this easily. Can you please make these changes, test it locally and create a pull request?

Yeah no problem, once I get some free time I'll make a PR