Messages in this thread Patch in this message | | | From | Jacob Keller <> | Subject | [PATCH] checkpatch: add NL_SET_ERR_MSG to 80 column exceptions | Date | Fri, 1 May 2020 16:11:31 -0700 |
| |
NL_SET_ERR_MSG and NL_SET_ERR_MSG_MOD are used to report extended error responses about failure of a netlink command. These strings often end up going over the 80-column limit. Just like logging messages, it is preferred to leave the message all on a single line.
Add these to the exception list so that checkpatch.pl will no longer complain about the long lines due to use of these macros.
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com> Cc: Jakub Kicinski <kuba@kernel.org> Cc: Andy Whitcroft <apw@canonical.com> Cc: Joe Perches <joe@perches.com> --- scripts/checkpatch.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index eac40f0abd56..5da3b06fbeaa 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -471,7 +471,8 @@ our $logFunctions = qr{(?x: WARN(?:_RATELIMIT|_ONCE|)| panic| MODULE_[A-Z_]+| - seq_vprintf|seq_printf|seq_puts + seq_vprintf|seq_printf|seq_puts| + NL_SET_ERR_MSG(?:_MOD)? )}; our $allocFunctions = qr{(?x: -- 2.25.2
| |