-
Notifications
You must be signed in to change notification settings - Fork 11
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
misc: adapt to gcc 14.2 warnings #97
base: master
Are you sure you want to change the base?
Conversation
c68de47
to
a53b047
Compare
Unit Test Results7 329 tests - 396 6 645 ✅ - 365 41m 22s ⏱️ + 4m 13s For more details on these failures, see this check. Results for commit 56af6c1. ± Comparison against base commit b0964a1. This pull request removes 422 and adds 26 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
bfeac7a
to
4d365cc
Compare
* lib-lwip ce151524...aba8a8ea (1): > newtif: add message to assert JIRA: RTOS-927
JIRA: RTOS-927
4d365cc
to
56af6c1
Compare
phy->bus, phy->addr, linkup ? "UP " : "DOWN", speed, (full_duplex != 0) ? "Full" : "Half", | ||
bctl, bstat, adv, lpa, pc1, pc2); |
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.
[clang-format-pr] reported by reviewdog 🐶
suggested fix
phy->bus, phy->addr, linkup ? "UP " : "DOWN", speed, (full_duplex != 0) ? "Full" : "Half", | |
bctl, bstat, adv, lpa, pc1, pc2); | |
phy->bus, phy->addr, linkup ? "UP " : "DOWN", speed, (full_duplex != 0) ? "Full" : "Half", | |
bctl, bstat, adv, lpa, pc1, pc2); |
Description
Warnings fixed:
Motivation and Context
Types of changes
How Has This Been Tested?
Checklist:
Special treatment
misc: match declaration and definition signatures lwip#13