this was actually cleaner

lia-viam · lia-viam · commit e79b8672df8a · 2024-10-04T17:21:11.000-04:00
diff --git a/src/viam/examples/modules/complex/summation/impl.cpp b/src/viam/examples/modules/complex/summation/impl.cpp
@@ -17,11 +17,8 @@ bool find_subtract(ResourceConfig cfg) {
     if (subtract == cfg.attributes().end()) {
         return false;
     }
-    const bool* const subtract_bool = subtract->second.get<bool>();
-    if (!subtract_bool) {
-        return false;
-    }
-    return *subtract_bool;
+
+    return subtract->second.is_a<bool>() && subtract->second.get_unchecked<bool>();
 }
 
 void MySummation::reconfigure(const Dependencies& deps, const ResourceConfig& cfg) {

Original file line number	Diff line number	Diff line change
`@@ -17,11 +17,8 @@ bool find_subtract(ResourceConfig cfg) {`
`17`	`17`	`if (subtract == cfg.attributes().end()) {`
`18`	`18`	`return false;`
`19`	`19`	`}`
`20`		`- const bool* const subtract_bool = subtract->second.get<bool>();`
`21`		`- if (!subtract_bool) {`
`22`		`- return false;`
`23`		`- }`
`24`		`- return *subtract_bool;`
	`20`	`+`
	`21`	`+ return subtract->second.is_a<bool>() && subtract->second.get_unchecked<bool>();`
`25`	`22`	`}`
`26`	`23`
`27`	`24`	`void MySummation::reconfigure(const Dependencies& deps, const ResourceConfig& cfg) {`