Default CMake option settings should not cause failure on systems with no 64-bit integer type

[dsabala]

This library targets embedded systems, especially those with low amount of memory.
I am working on such a project with Cortex-M MCU and ARM GCC 13.3.
However I will continue to use mpaland library since with default settings this library throws this error

228 | #error "No basic integer type has a size of 64 bits exactly"

Please consider setting the default options for your most frequent use cases.

[eyalroz]

You're essentially telling me that enabling MSVC-style specifiers should be turned off by default; or that it should be willing to accept the lack of a 64-bit type.

[dsabala]

There is a bug on line 190 in printf.c

It is

#ifdef PRINTF_SUPPORT_MSVC_STYLE_INTEGER_SPECIFIERS

It should be something like this

#if PRINTF_SUPPORT_MSVC_STYLE_INTEGER_SPECIFIERS == 1

Now:

set(SUPPORT_MSVC_STYLE_INTEGER_SPECIFIERS OFF)
add_subdirectory(eyalrozprintf)

The SUPPORT_MSVC_STYLE_INTEGER_SPECIFIERS setting is ignored, which initially gives a bad impression when choosing a library. I'm very curious, what is the use of the MSVC compiler in a world of limited memory? Can MSVC be used, for example, to develop Cortex-M firmware?

You're essentially telling me that enabling MSVC-style specifiers should be turned off by default

I think so, probably the vast majority of library users do not use MSVC.

[eyalroz]

this bug #205 , already fixed on the development branch. Sorry about that.

The feature is not about the MSVC compiler, it's about the specifiers, %I8, %I16, %I32, %I64. See the README.

Anyway, I will get around to this, but probably not before January I'm afraid.

[dsabala]

Thanks for explanation, today I learned: interesting alternative for using inttypes.h header

[eyalroz]

This is still a valid issue... I do need to change the default, for now - until I can arrange for the MSVC specifiers to support systems with no known 64-bit type (e.g. by only enabling specifiers for which we can find the type.

[eyalroz]

@dsabala : Would you mind pulling the development branch, and building with MSVC specifiers turned on (they're currently off by default)? I changed the preprocessor code to hopefully work even when not all desired sizes for these specifiers are available.