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

AP_Proximity: avoid use of int16_t-read call #27496

Merged

Conversation

peterbarker
Copy link
Contributor

Replaces #27059 and is careful not to change any logic.

@peterbarker peterbarker force-pushed the pr/int16-t-lifghtwaresf45b branch from 4fb5ce8 to 4fe8b00 Compare July 9, 2024 06:47
@tridge
Copy link
Contributor

tridge commented Jul 10, 2024

@peterbarker looks good.
btw, after the last discussion I realised that uart->available() is actually constrained to the size of the readbuf on all HALs. That means <= 4k unless someone has done an unusual begin()

@peterbarker peterbarker merged commit 31f3a79 into ArduPilot:master Jul 10, 2024
92 checks passed
@peterbarker peterbarker deleted the pr/int16-t-lifghtwaresf45b branch July 10, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants