Include cinttypes instead of inttypes.h (#6548)
authorAndres Noetzli <andres.noetzli@gmail.com>
Mon, 17 May 2021 16:52:55 +0000 (09:52 -0700)
committerGitHub <noreply@github.com>
Mon, 17 May 2021 16:52:55 +0000 (16:52 +0000)
commiteed22b44b92d14be51f82e4871c926dee79eaf3a
treedd22a27678c0e47e62c829b6497cab92335fd57a
parent63281fbfe093b1d5e375a378bb59761f77256d08
Include cinttypes instead of inttypes.h (#6548)

This commit changes the includes used by MiniSat. This commit changes
the includes from stdint.h/inttypes.h/limits.h to
cstdint/cinttypes/climits. This ensures that the macros in
cinttypes/inttypes.h, e.g., `PRIi64`, are actually defined. The C99
standard suggested that those macros are only defined for C++ code when
`__STDC_FORMAT_MACROS` is defined. This was never adopted by a C++
standard (https://en.cppreference.com/w/cpp/types/integer). However,
certain versions of mingw-w64 seem to require it with inttypes.h but not
cinttypes.

This fixes the nightly Windows build (tested in the Docker container
used by the nightlies).
src/prop/bvminisat/mtl/IntTypes.h
src/prop/minisat/mtl/IntTypes.h