Bug in Executor::get_next_ready_executable ?

[jmachowinski]

Hey

rclcpp/rclcpp/src/rclcpp/executor.cpp

Line 692 in f9c4894

if (callback_group && !callback_group->can_be_taken_from()) {

I am highly confused by this code block:

        auto callback_group = entity_iter->second.callback_group.lock();
        if (callback_group && !callback_group->can_be_taken_from()) {
          continue;
        }

Shouldn't it be ?

        auto callback_group = entity_iter->second.callback_group.lock();
        if (!callback_group || !callback_group->can_be_taken_from()) {
          continue;
        }

[jmachowinski]

@wjwwood @mjcarroll this was changed in #2142

[mjcarroll]

You are correct, and the ones that follow are also incorrect, I will patch.

Good catch!

[jmachowinski]

fixed