Internal compiler error when building with MinGW on Windows

[sakiryu]

When i tried to build a console app using the header-only version, got the following error:

In file included from ../fmt/include/fmt/format.h:3536,
from ../main.cpp:4:
../fmt/include/fmt/format-inl.h: In function 'int fmt::v6::internal::snprintf_float(T, int, fmt::v6::internal::float_specs, fmt::v6::internal::buffer&)':
../fmt/include/fmt/format-inl.h:69:25: internal compiler error: Segmentation fault
#define FMT_SNPRINTF snprintf
../fmt/include/fmt/format-inl.h:1163:25: note: in expansion of macro 'FMT_SNPRINTF'
auto snprintf_ptr = FMT_SNPRINTF;

This was caused by the snprintf assignment to snprintf_ptr variable. I was able to build successfully after a few changes:

Line 64, change:
#define FMT_SNPRINTF snprintf

To:
#if defined(__MINGW32__)
int (*psnprintf)(char*, std::size_t, char const*, ...) = std::snprintf;
#define FMT_SNPRINTF psnprintf
#else
#define FMT_SNPRINTF snprintf
#endif

[vitaut]

Does changing

fmt/include/fmt/format-inl.h

Line 1150 in 25d6916

auto snprintf_ptr = FMT_SNPRINTF;

to

int (*snprintf_ptr)(char*, std::size_t, char const*, ...) = FMT_SNPRINTF;

and similarly in

fmt/src/format.cc

Line 22 in 25d6916

auto snprintf_ptr = FMT_SNPRINTF;

also "fix" the internal compiler error?

[sakiryu]

Yes, it works as well.

[vitaut]

Great, thanks for checking. Could you submit a PR with these two small changes and a comment that it's a workaround for MinGW?

[sakiryu]

Sure

[vitaut]

@ryusakki, could you also post your compiler version here for future reference?