Remove mavlink enum dependencies on non related library
#23.320 aperta il 24 mar 2023
Metriche repository
- Star
- (9336 stelle)
- Metriche merge PR
- (Merge medio 26g 16h) (166 PR mergiate in 30 g)
Descrizione
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.