-
Notifications
You must be signed in to change notification settings - Fork 3k
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
features: nanostack: fix warning in thread_extension.h #9696
Conversation
@naveenkaje, thank you for your changes. |
@naveenkaje, thank you for your changes. |
CI started |
@naveenkaje The string of PRs that you created today (https://github.com/ARMmbed/mbed-os/pulls/naveenkaje), they all referenced warning present when building with GCC. Do the warnings also go away with the ARM and IAR compilers? |
Test run: SUCCESSSummary: 12 of 12 test jobs passed |
@cmonr is another review required here? |
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.
100644 → 100755
File permission needs revert
Thanks for pointing it.
The fix looks fine, a question is about the line 90 - not triggering this warning also?
No, line 90 is not flagged.
Fix the following build warning seen with GCC Compile [ 51.2%]: thread_bootstrap.c [Warning] thread_extension.h@88,44: statement with no effect [-Wunused-value]
6852e24
to
1f015aa
Compare
@NirSonnenschein Not really since the change seems so minor.
|
CI started |
Test run: SUCCESSSummary: 12 of 12 test jobs passed |
Description
Fix the following build warning seen with GCC
Pull request type