fixes

Author: facontidavide

include/behaviortree_cpp/utils/convert_impl.hpp

@@ -154,18 +154,11 @@ void convertNumber(const SRC& source, DST& target)
       throw std::runtime_error("Value is negative and can't be converted to unsigned");
     }
   }
-  // these conversions are always safe:
-  // - same type
-  // - float -> double
-  if constexpr(is_same<SRC, DST>() || (is_same<SRC, float>() && is_same<DST, double>()))
+  // these conversions are always safe: // No check needed
+  if constexpr(is_same<SRC, DST>() ||                                // same type
+               (is_same<SRC, float>() && is_same<DST, double>()) ||  // float -> double
+               (is_convertible_to_bool<SRC>() && is_same<DST, bool>()))
   {
-    // No check needed
-    target = static_cast<DST>(source);
-  }
-  // conversion to bool requires validation (only 0 and 1 allowed)
-  else if constexpr(is_convertible_to_bool<SRC>() && is_same<DST, bool>())
-  {
-    checkLowerLimit<SRC, DST>(source);
     target = static_cast<DST>(source);
   }
   else if constexpr(both_integers)

include/behaviortree_cpp/utils/simple_string.hpp

@@ -20,6 +20,7 @@ class SimpleString
 public:
   SimpleString()
   {
+    _storage.soo.capacity_left = CAPACITY;
     _storage.soo.data[0] = '\0';
   }