-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
tests: add missing cstdint include for gcc13 #26792
Conversation
See also 853691d Signed-off-by: Karl Palsson <[email protected]>
PR #26792: Size comparison from e022057 to 71a48e0 Increases (12 builds for bl602, bl702, efr32, psoc6, qpg, telink)
Decreases (4 builds for bl602, cc32xx, efr32, telink)
Full report (57 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
The C++ variant See also: https://stackoverflow.com/a/13643019/184207 Note there are already both cases in our codebase, but maybe having a common style makes sense? |
I have no comment on that, that's a style question for you guys, I'm far too new to go making any sort of sweeping unification of that, I just copied the last fix and got it working again :) |
Yeah any fix is fine (and helpful) from my POV. I actually hesitated creating more such PRs because I couldn't decide on what to do 😰 Since there is both style in the codebase anyways, I don't think its a big deal to introduce more of one or the other. Just wanted to point out, in case someone has a strong opinion which way to go. |
See also 853691d Signed-off-by: Karl Palsson <[email protected]>
See also 853691d
Another gcc13 breakage while doing initial install.sh on a fresh fedora 38 system. Just add the missing cstdint include as in prior fixes.