-
Notifications
You must be signed in to change notification settings - Fork 230
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
lib/run_part: Minor cleanups #1077
Conversation
stoeckmann
commented
Sep 13, 2024
- Use correct data type
- Unify logging (at least the fprintf calls)
- Reduce visibility of locally used function
Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
11b5abc
to
742b1b0
Compare
Use shadow_logfd for logging instead of fixed stderr to use shadow's own logging infrastructure. Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
The run_part function is only used in run_part.c itself, so no need to expose it to other files. Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
2ff1b6a
to
029ebe5
Compare
At least if they can be assigned directly to a function call. Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Remove some of these whitespaces. Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
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.
Reviewed-by: Alejandro Colomar <alx@kernel.org>
I'll let someone else merge. Thanks!
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.
Two minor comments. Once those are fixed I'll proceed to merge.
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.
LGTM! Thank you for the patches