BehaviorTree / BehaviorTree.CPP / 21622459334

80%

push

github

Fix Groot2Publisher destructor infinite loop (#1057) (#1100)

The destructor could hang indefinitely when the ZMQ server thread
was waiting on recv() while active_server_ remained true.

Changes:
- Set active_server_=false before joining threads
- Call zmq_context.shutdown() to interrupt blocking recv()
- Add try-catch around ZMQ operations to handle context termination
- Reorder destructor to remove hooks after threads are joined

Includes tests for:
- Destructor completion after exception
- Destructor with multiple tree nodes attached
- Rapid create/destroy cycles

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

5 of 10 new or added lines in 1 file covered. (50.0%)

3760 of 5402 relevant lines covered (69.6%)

24453.45 hits per line