Tencent / rapidjson

A fast JSON parser/generator for C++ with both SAX/DOM style API

http://rapidjson.org/

Other

14k stars 3.5k forks source link

fix Visual Studio 2022 (using /std:c++20) warning C5232: in C… #2242

Closed gvollant closed 3 months ago

gvollant commented 6 months ago

Fix issue #2241 : Visual Studio 2022 (using /std:c++20) new warning warning C5232: in C++20 this comparison calls ... recursively

tencent-adm commented 6 months ago

All committers have signed the CLA.

gvollant commented 6 months ago

it seem I made a clone of #2197

Neustradamus commented 5 months ago

@tencent team,

Have you seen this PR?

@gvollant: Can you look my emails?

bobbymcr commented 3 months ago

Thanks @gvollant , looks like we are trying to address the same problem! Is anyone from @tencent-adm able to get some of these changes merged? I have 3 PRs currently waiting, actually, which should help get the right coverage for MSVC / C++20:

Tencent / rapidjson

fix Visual Studio 2022 (using /std:c++20) warning C5232: in C… #2242

2197

2198

2199