cpprestsdk icon indicating copy to clipboard operation
cpprestsdk copied to clipboard

Fix likely typo in SafeInt3.hpp, that results in error with clang 15

Open Fighter19 opened this issue 3 years ago • 2 comments
trafficstars

Please double check before merge. Prior to this, it resulted in following error while trying to build with the Emscripten toolchain.

[  1%] Building CXX object Release/src/CMakeFiles/cpprest.dir/http/client/http_client.cpp.o
In file included from /cpprestsdk/Release/src/http/client/http_client.cpp:16:
In file included from /cpprestsdk/Release/src/pch/stdafx.h:73:
In file included from /cpprestsdk/Release/include/cpprest/details/basic_types.h:31:
/cpprestsdk/Release/include/cpprest/details/SafeInt3.hpp:1577:32: error: parameter 'b' set but not used [-Werror,-Wunused-but-set-parameter]
    static void CastThrow(bool b, T& t) SAFEINT_CPP_THROW

Fighter19 avatar Jul 01 '22 12:07 Fighter19

Thanks. This should be merged asap.

blazorin avatar Sep 17 '22 10:09 blazorin

收到,谢谢 

yuxiang-wen avatar Sep 17 '22 10:09 yuxiang-wen