Fixes to #135

[TkTech]

No description provided.

[TkTech]

@lemire this is updating your #135 PR. Is simdjson still tested under C++11? It's currently broken and requires C++17 if the header is used multiple times.

 SIMDJSON_CONSTEXPR_LAMBDA void escape_json_char(char c, char *&out) { ... }

I believe what's happening here is that in C++17 SIMDJSON_CONSTEXPR_LAMBDA is constexpr, which will inline. So importing it twice is fine. On C++11, the macro expands to nothing so it's just a regular header-defined function which will conflict with any other object that also imported simdjson.h.

[lemire]

@TkTech Ah. Makes sense. This looks like a simdjson bug.

[lemire]

I’ll work on a patch

[lemire]

Once the PR simdjson/simdjson#2606 goes green, I will issue a release.

[lemire]

@TkTech Please update to 4.3.1, there should be no issue:

https://github.com/simdjson/simdjson/releases/tag/v4.3.1