-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Document all constants for both user and developer use #44
Comments
@LorenzMeier he has a good point here |
Indeed. Let’s get this closed. |
UPDATED @LorenzMeier Confirmed offline that the following need to be documented (these allow you to configure the library for your needs and e.g. make it more efficient and less standalone).
The following do not need to be documented - they are internal/not intended to be tuned:
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
Sync with upstream master | Fri Aug 6 17:06:46 UTC 2021
Some of the constants that need documentation are below. Those that are available for the user to edit should also be added to the wiki and possibly the README.
For users:
MAVLINK_USE_CONVENIENCE_FUNCTIONS
MAVLINK_EXTERNAL_RX_BUFFER
MAVLINK_CRC_EXTRA
MAVLINK_ENDIAN
Various constants in protocol.h, mavlink.h
For developers:
MAVLINK_NEED_BYTE_SWAP
MAVLINK_ALIGNED_FIELDS
Various constants in protocol.h, mavlink.h
MAVLINK_MAX_EXTENDED_PAYLOAD_LEN - HW Note, this no longer exists
The text was updated successfully, but these errors were encountered: