-
Notifications
You must be signed in to change notification settings - Fork 2.1k
psa_crypto: fold some long lines #21303
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
Conversation
CI finds four more lines, would you mind folding them too? |
mguetschow ***@***.***> wrote:
CI finds four more lines, would you mind folding them too?
Yes, I was waiting for CI to complain to me, but that's why I wanted to run
the CI check manually :-)
|
I am uncertain if @brief doxygen lines can be wrapped. I could not find any clear detail yet. |
bf2e3f3
to
f08c570
Compare
Congrats, now you hit the commit message length restriction :P |
I think that I got them all. |
In the description the compiler switch -D is missing for setting the corresponding macro.
f08c570
to
44cb7f1
Compare
rewrote it. |
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.
👍 🫰
bit annoyed by the malloc_monitor commit not fitting in...but I guess I can just look away and we all get stuff done :> |
Contribution description
This should be a do-nothing, change nothing PR that just shortens long lines so that the static tests do not spuriously complain.
Testing procedure
Compiled and ran the tests/sys/psa_crypto_hashes test case.
Issues/PRs references
The complaints showed up in PR #21297.