commit d0a341b96370e44a815de13205d19552fb3b1650
author Ed Tanous <etanous@nvidia.com> Fri Oct 18 14:20:33 2024 -0700
committer Ed Tanous <ed@tanous.net> Tue Apr 29 03:20:43 2025 +0000
tree f0105306209b40e3133d70df2021cfc5eb2020db
parent 761cdfa58b98cf17568e7dd4349330f1c523b144

Remove is_object

is_object doesn't throw, but generally is_object is used in some kind of
pattern of.

if (x.is_object()){
   x["thing"];
}

operator[] technically throws if it's the wrong type, which bloats
binary sizes.

Replace these with the equivalent get_ptr<object_t>

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

redfish-core/include/sub_request.hpp

diff --git a/redfish-core/include/sub_request.hpp b/redfish-core/include/sub_request.hpp
index df8cf70..c44644a 100644
--- a/redfish-core/include/sub_request.hpp
+++ b/redfish-core/include/sub_request.hpp
@@ -31,7 +31,7 @@
             }
 
             auto oemIt = reqJson.find("Oem");
-            if (oemIt != reqJson.end() && oemIt->is_object())
+            if (oemIt != reqJson.end())
             {
                 const nlohmann::json::object_t* oemObj =
                     oemIt->get_ptr<const nlohmann::json::object_t*>();