-
Notifications
You must be signed in to change notification settings - Fork 763
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
fixed warnings [8886] #1290
fixed warnings [8886] #1290
Conversation
Signed-off-by: ahcorde <[email protected]>
The CI failure seems to be unrelated to the patch. @richiware Please advise how to proceed. |
@MiguelCompany or @EduPonz . Would you help on this? |
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.
Some cosmetic suggestions
Signed-off-by: ahcorde <[email protected]>
@MiguelCompany thank you for the review 👍 |
@MiguelCompany should we wait to run CI ? |
It seems CI is having problems getting your code (see here), but I have just finished running it manually, and everything seemed fine, so merging this in. |
* fixed warnings Signed-off-by: ahcorde <[email protected]> * Added cosmetic suggestions Signed-off-by: ahcorde <[email protected]>
* fixed warnings Signed-off-by: ahcorde <[email protected]> * Added cosmetic suggestions Signed-off-by: ahcorde <[email protected]>
* fixed warnings Signed-off-by: ahcorde <[email protected]> * Added cosmetic suggestions Signed-off-by: ahcorde <[email protected]>
* fixed warnings Signed-off-by: ahcorde <[email protected]> * Added cosmetic suggestions Signed-off-by: ahcorde <[email protected]>
There are a number of warning when I compile performance_test
Most of them are related to
uint32_t
which this type corresponds with the literalu
and notul
Signed-off-by: ahcorde [email protected]