commit 7719269f0d269b1bf206545fcc645e86c9c42e90
author Ed Tanous <etanous@nvidia.com> Tue Jun 24 11:32:12 2025 -0700
committer Ed Tanous <ed@tanous.net> Tue Nov 18 04:08:13 2025 +0000
tree f8fc216d230ccf40a8838a3dfc89c7714d964096
parent 4c9979f9122f4d63a54d692c6e7080426da5e6a6

Remove implicit conversions

Implicit conversions are something that nlohmann library itself is a bad
default, and 3 years ago threatened to change the default.  These
implicit conversions cause a number of crashes that are hard to
reproduce, because they throw an uncaught exception.

Update the code to be able to do no more implicit conversions.

Tested: Entity-manager launches and runs.  Items are detected correctly
and show up on dbus.  Unit tests pass.

Change-Id: Ib23159ae58f5584641427d9be7545bc25a3619af
Signed-off-by: Ed Tanous <etanous@nvidia.com>

meson.build

diff --git a/meson.build b/meson.build
index c364a9e..4df6652 100644
--- a/meson.build
+++ b/meson.build
@@ -6,7 +6,10 @@
     version: '0.1',
     meson_version: '>=1.3.0',
 )
-add_project_arguments('-Wno-psabi', language: 'cpp')
+add_project_arguments(
+    ['-Wno-psabi', '-DJSON_USE_IMPLICIT_CONVERSIONS=0'],
+    language: 'cpp',
+)
 
 boost_args = [
     '-DBOOST_ASIO_NO_DEPRECATED',