tree f376af88690e6e319bd828ff95a1d15d88d256f6
parent ed34a4adf3cc28c92c6d1d9262f47d1f35595d71
author Ed Tanous <edtanous@google.com> 1675455826 -0800
committer Ed Tanous <ed@tanous.net> 1676056495 +0000

Change readJson from is_object to get_ptr

While is_object is useful in contexts where we don't need the underlying
value, in cases where we do, nlohmann::json::begin() can throw when
generating the nlohmann::iterator_proxy object.  This causes unwind
tables to be generated, so as a general rule, we should be prefering
get_ptr in these cases.

Tested: Good unit test coverage for readJson.  Unit tests only.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I5ecb2dca95cc06246e983f9e4190b2dd46d1a6b9