search

chadaustin / sajson

Lightweight, extremely high-performance JSON parser for C++11
MIT License
568 stars 43 forks source link

snprintf definition #29

Closed SDraw closed 7 years ago

SDraw commented 7 years ago

'snprintf' was added in MSVS 2015, but it had name '_snprintf' before. https://msdn.microsoft.com/ru-ru/library/2ts7cx93(v=vs.140).aspx

chadaustin commented 7 years ago

Nice find! I think fixing this is important, but I don't think sajson should #define an unnamespaced symbol like that, especially because it's a header-only library. Would you mind adding a macro which invokes the appropriate snprintf function? Thanks!

SDraw commented 7 years ago

You're right. How is it now?

chadaustin commented 7 years ago

Thank you!