Niels Lohmann

icon indicating a website link https://nlohmann.me icon indicating an email [email protected]

icon location Berlin, Germany icon location You may know me from my JSON library for C++.

Results 421 comments of Niels Lohmann

Fix json::update() with merge_objects==true error

Maybe related: #4813

[MSVC][build] JSON failed with error C2672: 'nlohmann::json_abi_v3_12_0::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,

Not using MSVC myself, so I'd be glad if anyone could help.

[MSVC][build] JSON failed with error C2672: 'nlohmann::json_abi_v3_12_0::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,

@QuellaZhang Just so I understand: the code does compile with previous versions of MSVC, but fails to do so with a newer version unless `/permisisve-` is used?

std::map and std::unordered_map serialization broken for keys of type std::u16string

Using the `to_json` does not work, because the keys only need to be converted to object keys, but not `basic_json`. We use this code to determine if a type can...

std::map and std::unordered_map serialization broken for keys of type std::u16string

Yes, this looks reasonable.

Tests don't build with VS 2026

Strange. It's this test: ```cpp #ifdef JSON_HAS_CPP_17 #if JSON_HAS_FILESYSTEM || JSON_HAS_EXPERIMENTAL_FILESYSTEM TEST_CASE("std::filesystem::path") { SECTION("ascii") { json const j_string = "Path"; auto p = j_string.template get(); json const j_path = p;...

Tests don't build with VS 2026

Strange. `test-conversions_cpp17` should be built with C++17, and since the test fails in line 1667, `JSON_HAS_CPP_17` should be set and `JSON_HAS_FILESYSTEM` or `JSON_HAS_EXPERIMENTAL_FILESYSTEM` is set.

Make escape/unescape functions linear and not memory-storming.

Clang-Tidy fails due to an update. I have not found the time to look into it - probably just some exclusions needed.

Make escape/unescape functions linear and not memory-storming.

> Clang-Tidy fails due to an update. I have not found the time to look into it - probably just some exclusions needed. Oh, I see there are indeed some...

Make escape/unescape functions linear and not memory-storming.

🔴 CI is red due to unrelated issues, see https://github.com/nlohmann/json/issues/4869.