Fix missing read_parameter_from_ports_ initialization in DelayNode (#1103)

facontidavide · claude · facontidavide · commit 417b3e4cd20e · 2026-02-04T12:20:40.000+01:00
Add explicit initialization of read_parameter_from_ports_(false) in the
first constructor to match the pattern used by TimeoutNode, RetryNode,
RepeatNode, and ParallelNode.

Co-Authored-By: Claude Opus 4.5 &lt;noreply@anthropic.com&gt;
diff --git a/src/decorators/delay_node.cpp b/src/decorators/delay_node.cpp
@@ -6,7 +6,10 @@
 namespace BT
 {
 DelayNode::DelayNode(const std::string& name, unsigned milliseconds)
-  : DecoratorNode(name, {}), timer_id_(0), msec_(milliseconds)
+  : DecoratorNode(name, {})
+  , timer_id_(0)
+  , msec_(milliseconds)
+  , read_parameter_from_ports_(false)
 {
   setRegistrationID("Delay");
 }

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,10 @@`
`6`	`6`	`namespace BT`
`7`	`7`	`{`
`8`	`8`	`DelayNode::DelayNode(const std::string& name, unsigned milliseconds)`
`9`		`- : DecoratorNode(name, {}), timer_id_(0), msec_(milliseconds)`
	`9`	`+ : DecoratorNode(name, {})`
	`10`	`+ , timer_id_(0)`
	`11`	`+ , msec_(milliseconds)`
	`12`	`+ , read_parameter_from_ports_(false)`
`10`	`13`	`{`
`11`	`14`	`setRegistrationID("Delay");`
`12`	`15`	`}`