-
Notifications
You must be signed in to change notification settings - Fork 17.2k
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
Libraries: Match output type and variable type #10151
Libraries: Match output type and variable type #10151
Conversation
the AP_HAL formatting changes asnd RC_Channel changes look good to me. These change are pretty much only affecting debug messages so I think they are safe to merge as is. |
Looks like this should have been merged and wasn't. @muramura could you rebase, please? |
@@ -435,7 +435,7 @@ static void PrintDebug(const Proto* f) | |||
printf("upvalues (%d) for %p:\n",n,VOID(f)); | |||
for (i=0; i<n; i++) | |||
{ | |||
printf("\t%d\t%s\t%d\t%d\n", | |||
printf("\t%d\t%s\t%u\t%u\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the change here, this is external code dependencies, and I'd rather not add conflicts.
@muramura this hasn't been rebased. |
the RC decoders in AP_HAL/utility are going to be removed and replaced with AP_RCProtocol |
e68a200
to
7dec784
Compare
rebase, drop tabs change, address comment |
I was detected with the cppcheck command that there was no output type and variable type.
I think that matching the type will increase the authenticity of the message.