Add support of multi-dim C-style array member of struct.

nlohmann / json

JSON for Modern C++

https://json.nlohmann.me

MIT License

41.3k stars 6.58k forks source link

Add support of multi-dim C-style array member of struct. #4262

Open peng-wang-cn opened 6 months ago

peng-wang-cn commented 6 months ago

Support up to 4 dimensional array.
Modify pretty-print logic to print multi-dim array's inner most dimension in the same line for better readability.

Refer to discussion #4248 .

coveralls commented 6 months ago

coverage: 100.0%. remained the same when pulling 68c0097234b4ba71058a0dae70fe26591b8aa04d on peng-wang-cn:multidim-array into a259ecc51e1951e12f757ce17db958e9881e9c6c on nlohmann:develop.

peng-wang-cn commented 6 months ago

It looks like the failed tests are due to a clang bug #75943, which was fixed by PR #76007, 2 weeks ago. So @nlohmann , what do we do with this?

nlohmann commented 6 months ago

It looks like the failed tests are due to a clang bug #75943, which was fixed by PR #76007, 2 weeks ago. So @nlohmann , what do we do with this?

Please fix such that there is no error with Clangs out there that still have that bug.

peng-wang-cn commented 5 months ago

It looks like the failed tests are due to a clang bug #75943, which was fixed by PR #76007, 2 weeks ago. So @nlohmann , what do we do with this?

Please fix such that there is no error with Clangs out there that still have that bug.

Using docker image tuxmake/clang-nightly:20240109, ci_test_clang will pass. Can we update the CI config to use images from tuxmake/clang-nightly? @nlohmann