diff options
author | Oliver Smith <osmith@sysmocom.de> | 2021-09-01 11:23:55 +0200 |
---|---|---|
committer | osmith <osmith@sysmocom.de> | 2021-09-03 08:27:28 +0000 |
commit | 4f75177e8c7978afde432de4a79d6ea6e49b1825 (patch) | |
tree | 24a109947b9a636a2e401f25d650d9c6ed2f0253 /lint/checkpatch | |
parent | d314ae8280a40881b6b6a7457a7c94802ed73c77 (diff) |
lint: checkpatch_osmo.sh: ignore LONG_LINE_*
According to a comment in checkpatch.pl, these should automatically be
ignored when LONG_LINE is ignored. However I found that it's still
complaining about LONG_LINE_COMMENT, so explicitly disable it.
Related: OS#5087
Change-Id: I7aed3bfbfcb0b9e2f1d743b111e8418846f031d2
Diffstat (limited to 'lint/checkpatch')
-rwxr-xr-x | lint/checkpatch/checkpatch_osmo.sh | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lint/checkpatch/checkpatch_osmo.sh b/lint/checkpatch/checkpatch_osmo.sh index b465b5a..849ccda 100755 --- a/lint/checkpatch/checkpatch_osmo.sh +++ b/lint/checkpatch/checkpatch_osmo.sh @@ -24,7 +24,7 @@ SCRIPT_DIR="$(dirname "$(realpath "$0")")" # * INITIALISED_STATIC: we use this, see also http://lkml.iu.edu/hypermail/linux/kernel/0808.1/2235.html # * LINE_CONTINUATIONS: false positives # * LINE_SPACING: we don't always put a blank line after declarations -# * LONG_LINE: should be 120 chars, but exceptions are done often so don't fail here +# * LONG_LINE*: should be 120 chars, but exceptions are done often so don't fail here # * PREFER_DEFINED_ATTRIBUTE_MACRO: macros like __packed not defined in libosmocore # * PREFER_FALLTHROUGH: pseudo keyword macro "fallthrough" is not defined in libosmocore # * REPEATED_WORD: false positives in doxygen descriptions (e.g. '\param[in] data Data passed through...') @@ -54,6 +54,8 @@ $SCRIPT_DIR/checkpatch.pl \ --ignore LINE_CONTINUATIONS \ --ignore LINE_SPACING \ --ignore LONG_LINE \ + --ignore LONG_LINE_COMMENT \ + --ignore LONG_LINE_STRING \ --ignore PREFER_DEFINED_ATTRIBUTE_MACRO \ --ignore PREFER_FALLTHROUGH \ --ignore REPEATED_WORD \ |