Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

core: use separate mavlink_status_t again #1740

Merged
merged 1 commit into from
Apr 18, 2022
Merged

Commits on Apr 18, 2022

  1. core: use separate mavlink_status_t again

    Previously, we made an attempt to fix the outgoing sequence numbers.
    Therefore, we would ignore the chan number for the mavlink_status_t,
    however, this lead to serious problems when doing system tests with
    multiple mavsdk instances sending messages to each other as they would
    sometimes both use the mavlink_status_t data and mess up each others
    parsing.
    
    Therefore, we remove the previous hack, and go back to separate
    mavlink_status_t. This will probably mess up outgoing sequence numbers
    but given they can't reliably be used to assess drop rate anyway it's
    an ok pill to swallow, for now. And this enables more consistent system
    tests, and in general applications with more than one Mavsdk instance.
    julianoes committed Apr 18, 2022
    Configuration menu
    Copy the full SHA
    f9309ae View commit details
    Browse the repository at this point in the history