Repository navigation
Implement checkpatch.pl check for pr_*! without \n #1140
Description
Implement a checkpatch.pl check for pr_*! without \n (a warning, not an error, since one may be continuing the line), so that we reduce the chance of issues such as #1139.
Consider other printing facilities too that could be checked.
Consider if C could use such a check (if there is none yet).
This requires submitting a proper patch to the LKML and the Rust for Linux mailing list. Please recall to test your changes (including generating the documentation if changed, running the Rust doctests if changed, etc.), to use a proper title for the commit, to sign your commit under the Developer's Certificate of Origin and to add a Suggested-by: tag and a Link: tag to this issue. Please see https://docs.kernel.org/process/submitting-patches.html and https://rust-for-linux.com/contributing for details.
Please note that the checkpatch.pl maintainers will need to agree to the change and that the patch needs to be submitted to them.
Activity
albankurti commentedon Feb 3, 2025
Emailed potential implementation
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
3 remaining items
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …
checkpatch: add warning for pr_* and dev_* macros without a trailing …