-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
doc: Correct docstring format to silence warnings #927
Conversation
Several unikraft files have inconsistent docstrings which generate many warnings when building with clang. This change corrects all inconsistencies, silencing the warnings. Signed-off-by: Andrei Tatar <andrei@unikraft.io>
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.
Thanks for fixing all the documentation comments 🎉
Reviewed-by: Marco Schlumpp marco@unikraft.io
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.
Looks good, thanks a lot @andreittr 🚀
Reviewed-by: Stefan Jumarea stefanjumarea02@gmail.com
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.
Approved-by: Razvan Deaconescu razvand@unikraft.io
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.
All good also from my side.
Reviewed-by: Radu Nichita radunichita99@gmail.com
Several unikraft files have inconsistent docstrings which generate many warnings when building with clang. This change corrects all inconsistencies, silencing the warnings. Signed-off-by: Andrei Tatar <andrei@unikraft.io> Reviewed-by: Marco Schlumpp <marco@unikraft.io> Reviewed-by: Stefan Jumarea <stefanjumarea02@gmail.com> Approved-by: Razvan Deaconescu <razvand@unikraft.io> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: unikraft#927
Several unikraft files have inconsistent docstrings which generate many warnings when building with clang. This change corrects all inconsistencies, silencing the warnings. Signed-off-by: Andrei Tatar <andrei@unikraft.io> Reviewed-by: Marco Schlumpp <marco@unikraft.io> Reviewed-by: Stefan Jumarea <stefanjumarea02@gmail.com> Approved-by: Razvan Deaconescu <razvand@unikraft.io> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: unikraft#927
Several unikraft files have inconsistent docstrings which generate many warnings when building with clang. This change corrects all inconsistencies, silencing the warnings. Signed-off-by: Andrei Tatar <andrei@unikraft.io> Reviewed-by: Marco Schlumpp <marco@unikraft.io> Reviewed-by: Stefan Jumarea <stefanjumarea02@gmail.com> Approved-by: Razvan Deaconescu <razvand@unikraft.io> Tested-by: Unikraft CI <monkey@unikraft.io> GitHub-Closes: unikraft#927
Description of changes
Several unikraft files have inconsistent docstrings which generate many warnings when building with clang.
This change corrects all inconsistencies, silencing the warnings.
This PR contains only changes in comments. No actual compilable code was changed.
Tested with a mostly "everything + the kitchen sink" Kconfig of base unikraft (no external libs); no doc warnings remain.
Prerequisite checklist
checkpatch.uk
on your commit series before opening this PR;Base target
Additional configuration
You may need to clear your compiler cache (
ccache -C
) on systems where by default it interposes on your compiler (e.g. Fedora) in order to see the effect. Apparently ccache compares sources after preprocessing.