mavlink callback manager comparison

gistfile1.txt

void MavlinkMessageHandler::process_message(const mavlink_message_t& message)
{
    std::lock_guard<std::mutex> lock(_mutex);

#if MESSAGE_DEBUGGING == 1
    bool forwarded = false;
#endif
    for (auto& entry : _table) {
        if (entry.msg_id == message.msgid &&
            (!entry.cmp_id.has_value() || entry.cmp_id == message.compid)) {
#if MESSAGE_DEBUGGING == 1
            LogDebug() << "Forwarding msg " << int(message.msgid) << " to "
                       << size_t(entry->cookie);
            forwarded = true;
#endif
            entry.callback(message);
        }
    }

#if MESSAGE_DEBUGGING == 1
    if (!forwarded) {
        LogDebug() << "Ignoring msg " << int(message.msgid);
    }
#endif
}

vs


    // Execute all callbacks for a specific message ID
    void notify(MessageID msgid, Args... args)
    {
        CallbackList<Args...>* callbackListPtr = nullptr;

        {
            std::lock_guard<std::mutex> lock(_mutex);
            auto it = _callbacks.find(msgid);
            if (it != _callbacks.end()) {
                callbackListPtr = &(it->second);
            }
        } // Unlock the map to avoid holding the mutex while executing callbacks

        // If we have a valid CallbackList pointer, execute the callbacks
        if (callbackListPtr) {
            (*callbackListPtr)(args...);
        }
    }