yes, I've added ptr_converter to the CMake now

address comments

rebase onto new changes, specifically converters getting return values.

add converter test

rebase on new changes and move length modifier handling to a shared function.

fix nit before land

address comments and move macro to a shared location

I split the file handling rewrite into its own separate patch after addressing the comments (https://reviews.llvm.org/D127773), this is now just adding true printf.

move shared fprintf functionality into internal function, as well as other cleanups

LGTM

While the idea of having a seperate "safe" version of printf is not inherently bad, it's already been tried to little success. In the standard right now is the definitions for the _s versions of printf functions. These have bounds checking and disable %n, but are also not widely used. Convincing developers to switch is hard, so my current plan is to use the compile-time switch I've added to disable %n by default in the existing printf functions. This provides the security enhancements by default for all of the people who don't care/didn't know that %n existed, while still allowing people to turn it back on if they are really sure they know what they're doing.

LGTM

rebase

overall, LGTM with a couple nits. Also, if you're going to add a clz header to FPUtil then please add a TODO for me to change str_to_float.h over to using it, or alternately change it yourself.

add tests and header guards

rebase before landing

address comments

address comments and add simple converter test

LGTM

move tests into sprintf_tests and split converter_test changes into a separate patch

split the buffering modes into different functions, as well as cleaning up each implementation to be simpler in its specific case.

change to a simpler algorithm

address comments

LGTM

switch to using limits.h for type size masks, additionally adjust the buffer size to be more exact.

switch to unlocked writes with the lock aquisition happening before the parsing. Additionally track errors better.

move writers to be object libraries

rewrite to be closer to the original

move to constexpr variables for test strings

LGTM

This patch doesn't have comprehensive unit tests yet, but should be good to go once I add those.

address comments

add handling for FILE errors

LGTM

add sprintf integer tests