[RFC PATCH 0/2] practical auto-QA de-noiser or -E_TOO_UGLY ?

Jim Cromie posted 2 patches 2 months ago
include/linux/compiler.h | 12 +++++++++
scripts/checkpatch.pl    | 54 ++++++++++++++++++++++++++++++++++------
2 files changed, 59 insertions(+), 7 deletions(-)
[RFC PATCH 0/2] practical auto-QA de-noiser or -E_TOO_UGLY ?
Posted by Jim Cromie 2 months ago
Can I haz (in compiler.h)

  #define __no_side_effects(...)    /* checkpatch hint only */

module_param_named() if submitted now, would draw several warnings
from checkpatch --strict, on name, value (but not type).

  CHECK: Macro argument reuse '_var' - possible side-effects?

This macro allows an author to suppress that complaint, clearly and
declaratively, and strongly suggests that they used --strict, saw the
complaint, and silenced it.  So they know not to pass i++.

Do the possible QA/CI benefits suffice for the mild "hackish" flavor ?


Jim Cromie (2):
  checkpatch: add __no_side_effects() hint/assertion macro
  checkpatch: cosmetic-style tweak

 include/linux/compiler.h | 12 +++++++++
 scripts/checkpatch.pl    | 54 ++++++++++++++++++++++++++++++++++------
 2 files changed, 59 insertions(+), 7 deletions(-)

-- 
2.51.0