Remove mavlink enum dependencies on non related library
#23 320 ouverte le 24 mars 2023
Métriques du dépôt
- Stars
- (9 336 étoiles)
- Métriques de merge PR
- (Merge moyen 26j 16h) (166 PRs mergées en 30 j)
Description
Feature request
Is your feature request related to a problem? Please describe. In numerous place, we were using mavlink enum. Those are well defined and conveniant but they bring a hard link to mavlink that can be harmfull codewise.
Therefore, we got a recent effort to reduce this dependency : see https://github.com/ArduPilot/ardupilot/pull/23229 . In this PR, we decouple the GPS status from the mavlink enum. That lower our inclusion mess.
Describe the solution you'd like We should now update our codebase to remove the mavlink GPS enum in favor of the GPS_Status enum.
Some other work could be done to make the same for EKF and other mavlink enums !
Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.
Platform [ X ] All [ ] AntennaTracker [ ] Copter [ ] Plane [ ] Rover [ ] Submarine
Additional context Add any other context or screenshots about the feature request here.