Require C++11 Minimum #35

carrus85 · 2024-08-07T22:50:45Z

Description

Fixes some warnings around still using sprintf a few places.
Adds optional C++11 support Under USE_CPP11 CMake Flag, which currently just removes the requirement for boost shared_ptr (which won't work on boost v1.86 or higher under C++0x anyway).
If not building with C++11, ensures we actually look up the boost include paths (instead of implicitly assuming their location).
Adds some missing header includes that we're being transitively provided via boost

carrus85 · 2024-08-07T22:51:04Z

Testing pending... just getting the PR up for some 👀

Not including this will cause an error if boost isn't included.

carrus85 changed the title ~~Get code compiling cleaning on M1 Mac.~~ Get code compiling cleanly on M1 Mac. Aug 7, 2024

carrus85 added 3 commits August 8, 2024 16:05

fix: use snprintf instead of sprintf (removes warnings)

7682175

fix: Add missing cassert include

36f0ab5

Not including this will cause an error if boost isn't included.

feat(c++11): Require C++11 minimum

4e41a29

carrus85 force-pushed the m1-mac-support branch from fec7afb to 4e41a29 Compare August 8, 2024 22:09

carrus85 changed the title ~~Get code compiling cleanly on M1 Mac.~~ Require C++11 Minimum Aug 8, 2024