rapidjson icon indicating copy to clipboard operation
rapidjson copied to clipboard
verified publisher icon Tencent

Fix MSVC build

Open bobbymcr opened this issue 2 years ago • 2 comments

  • Remove unsupported /std:c++11 flag (only c++14 and above is accepted)
  • Enable MSVC runtime library flag based on CMP0091
  • Ignore .vs folder
  • Add extra assertions in file open tests to detect bad pointers

bobbymcr avatar Sep 01 '23 18:09 bobbymcr

Seems this will break older VC versions?

miloyip avatar Mar 08 '24 02:03 miloyip

Seems this will break older VC versions?

Which version did you see it break? As far as I know this /std arg is only available starting in VS 2017 and only vaiues "c++14" and above options are supported. However I don't know for sure.

bobbymcr avatar Mar 08 '24 05:03 bobbymcr