json icon indicating copy to clipboard operation
json copied to clipboard

Make serializer dump() non-recursive

Open vii opened this issue 1 year ago • 4 comments

This allows printing very deep trees without running out of stack.

vii avatar Aug 23 '24 02:08 vii

Coverage Status

coverage: 100.0%. remained the same when pulling ccbafb4388288087bc4a9cd6728dec15374e353c on vii:develop into 960b763ecd144f156d05ec61f577b04107290137 on nlohmann:develop.

coveralls avatar Aug 23 '24 05:08 coveralls

Unfortunately has some compile errors on MSVC etc. Can update and fix if you like this direction -

error C7555: use of designated initializers requires at least '/std:c++20'

vii avatar Aug 23 '24 14:08 vii

I very much like this fix here! It was on my todo list as well.

t-b avatar Aug 24 '24 10:08 t-b

This pull request has been marked as stale because it has had no activity for 30 days. While we won’t close it automatically, we encourage you to update or comment if it is still relevant. Keeping pull requests active and up-to-date helps us review and merge changes more efficiently. Thank you for your contributions!

github-actions[bot] avatar Jan 19 '25 00:01 github-actions[bot]