Warnings when compiling with VisualStudio 2015

[ChipBurwell]

I've update to version 2.1.1 (from previously using 2.0.10) and now I'm getting the warnings listed below. I don't think any of them are critical, but I'm wondering if there is a simple way to fix this without turning off warnings that I'd like to have on?

1>json.hpp(6620): warning C4996: '_snprintf': This function or variable may be unsafe. Consider using _snprintf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files (x86)\windows kits\10\include\10.0.10240.0\ucrt\stdio.h(1952): note: see declaration of '_snprintf'
1> json.hpp(6594): note: while compiling class template member function 'void nlohmann::detail::serializer<nlohmann::basic_jsonstd::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>::dump_float(double)'
1> json.hpp(6203): note: see reference to function template instantiation 'void nlohmann::detail::serializer<nlohmann::basic_jsonstd::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>::dump_float(double)' being compiled
1> json.hpp(8609): note: see reference to class template instantiation 'nlohmann::detail::serializer<nlohmann::basic_jsonstd::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>' being compiled
1> json.hpp(8607): note: while compiling class template member function 'std::basic_string<char,std::char_traits,std::allocator> nlohmann::basic_jsonstd::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer::dump(const int,const char,const bool) const'
1> json.hpp(13523): note: see reference to function template instantiation 'std::basic_string<char,std::char_traits,std::allocator> nlohmann::basic_jsonstd::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer::dump(const int,const char,const bool) const' being compiled
1> c:\program files (x86)\microsoft visual studio 14.0\vc\include\type_traits(639): note: see reference to class template instantiation 'nlohmann::basic_jsonstd::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer' being compiled
1> c:\program files (x86)\microsoft visual studio 14.0\vc\include\type_traits(668): note: see reference to class template instantiation 'std::is_nothrow_constructible<_Ty,nlohmann::basic_jsonstd::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer &&>' being compiled
1> with
1> [
1> _Ty=nlohmann::json
1> ]
1> json.hpp(13504): note: see reference to class template instantiation 'std::is_nothrow_move_constructiblenlohmann::json' being compiled

[nlohmann]

I can confirm the warnings, but I am not MSVC expert, so I cannot decide whether they are critical. If someone has an idea - PRs are welcome :)

[crea7or]

Regarding '_snprintf' and _CRT_SECURE_NO_WARNINGS: Microsoft have special, modified string functions where strings sizes are passed as additional parameters:

int snprintf( char *buffer, size_t count, const char *format [, argument] ...);
vs
int _snprintf_s( char *buffer, size_t sizeOfBuffer, size_t count, const char *format [, argument] ... );

they are considered as more secure, so this is the reason for this warning.
warning can be suppressed for msvc compilers via:
#define _CRT_SECURE_NO_WARNINGS 1
or
#pragma warning(disable:4996)

[crea7or]

However I can't see this warning(warning C4996) in the current release and dev branches right now (vs 2015 and vs 2017).

[nlohmann]

I won't make another 2.x.x release - are there similar warnings in the develop branch?

[nlohmann]

@crea7or Ignore my previous comment #723 (comment).

If the warning disappeared in the develop branch, I can close this issue.

[ChipBurwell]

I just did a get of the latest develop branch and I can confirm this warning is still happening. I don't think this issue should have been closed. Chip

…

On 10/2/2017 9:22 AM, Niels Lohmann wrote: @crea7or <https://github.com/crea7or> Ignore my previous comment #723 (comment) <#723 (comment)>. If the warning disappeared in the |develop| branch, I can close this issue. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#723 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEoQadF9R52CIL9e945OH2zaqfBdK-W2ks5soQ3KgaJpZM4PLEDK>.

[nlohmann]

Can you please paste the complete warning? Which compiler version are you using?

[crea7or]

There is nothing related to sprintf on a line 6620 in dev or master branches. Other errors does not match the code in branches too.

[ChipBurwell]

These are the warnings I'm getting with code pulled this morning from the develop branch: 5>E:\TruGolf\e6_2\head\Fexy/Utils/json.hpp(6597): warning C4996: '_snprintf': This function or variable may be unsafe. Consider using _snprintf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details. 5>  C:\Program Files (x86)\Windows Kits\10\Include\10.0.10240.0\ucrt\stdio.h(1952): note: see declaration of '_snprintf' 5>  E:\TruGolf\e6_2\head\Fexy/Utils/json.hpp(6571): note: while compiling class template member function 'void nlohmann::detail::serializer<nlohmann::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>>::dump_float(double)' 5>  E:\TruGolf\e6_2\head\Fexy/Utils/json.hpp(6180): note: see reference to function template instantiation 'void nlohmann::detail::serializer<nlohmann::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>>::dump_float(double)' being compiled 5>  E:\TruGolf\e6_2\head\Fexy/Utils/json.hpp(8587): note: see reference to class template instantiation 'nlohmann::detail::serializer<nlohmann::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>>' being compiled 5>  E:\TruGolf\e6_2\head\Fexy/Utils/json.hpp(8585): note: while compiling class template member function 'std::basic_string<char,std::char_traits<char>,std::allocator<char>> nlohmann::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>::dump(const int,const char,const bool) const' 5>  E:\TruGolf\e6_2\head\Fexy/Utils/json.hpp(13512): note: see reference to function template instantiation 'std::basic_string<char,std::char_traits<char>,std::allocator<char>> nlohmann::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>::dump(const int,const char,const bool) const' being compiled 5>  C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\type_traits(639): note: see reference to class template instantiation 'nlohmann::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>' being compiled 5>  C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\type_traits(668): note: see reference to class template instantiation 'std::is_nothrow_constructible<_Ty,nlohmann::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer> &&>' being compiled 5>          with 5>          [ 5>              _Ty=nlohmann::json 5>          ] I'm using Visual Studio VC2015 which uses the VS 140 compilers I believe. I would suggest fixing this by replacing line 6597 which currently is:          std::ptrdiff_t len = snprintf(number_buffer.data(), number_buffer.size(), "%.*g", d, x); with #ifdef _MSC_VER         std::ptrdiff_t len = _snprintf_s( number_buffer.data(), number_buffer.size(), _TRUNCATE, "%.*g", d, x ); #else         std::ptrdiff_t len = snprintf( number_buffer.data(), number_buffer.size(), "%.*g", d, x ); #endif Chip

…

On 10/2/2017 9:52 AM, Niels Lohmann wrote: Can you please paste the complete warning? Which compiler version are you using? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#723 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEoQadR4wK8raL5SibG-qpc3qFFHBSAJks5soRTXgaJpZM4PLEDK>.