diff options
author | Oliver Smith <osmith@sysmocom.de> | 2021-07-07 11:04:47 +0200 |
---|---|---|
committer | Oliver Smith <osmith@sysmocom.de> | 2021-07-07 11:28:38 +0200 |
commit | 5409188d554edea8c0fa5b749572ce3d3777e86b (patch) | |
tree | d1056bd2b8c5eeb7c92f0b069dd11dacff33d207 | |
parent | 2cceff4535fb31aa2f8da4b029e74c8d53824074 (diff) |
lint: ignore LINE_CONTINUATIONS
Causes false positives, e.g. in tests/mgcp/mgcp_test.c in osmo-mgw:
#define MDCX4_PT2 \
"MDCX 18983218 1@mgw MGCP 1.0\r\n" \
"M: sendrecv\r" \
"C: 2\r\n" \
"I: %s\r\n" \
"L: p:20-20, a:AMR, nt:IN\r\n" \
...
tests/mgcp/mgcp_test.c:189: WARNING:LINE_CONTINUATIONS: Avoid unnecessary line continuations
Related: OS#5087
Change-Id: I8d8004f2a8ef926588487187af7cdef8254c7248
-rwxr-xr-x | lint/checkpatch/checkpatch_osmo.sh | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/lint/checkpatch/checkpatch_osmo.sh b/lint/checkpatch/checkpatch_osmo.sh index 75fef8e..1932c03 100755 --- a/lint/checkpatch/checkpatch_osmo.sh +++ b/lint/checkpatch/checkpatch_osmo.sh @@ -18,6 +18,7 @@ SCRIPT_DIR="$(dirname "$(realpath "$0")")" # * GLOBAL_INITIALISERS: we initialise globals to NULL for talloc ctx (e.g. *tall_lapd_ctx = NULL) # * IF_0: used intentionally # * 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 # * REPEATED_WORD: false positives in doxygen descriptions (e.g. '\param[in] data Data passed through...') # * SPDX_LICENSE_TAG: we don't place it on line 1 @@ -39,6 +40,7 @@ $SCRIPT_DIR/checkpatch.pl \ --ignore GLOBAL_INITIALISERS \ --ignore IF_0 \ --ignore INITIALISED_STATIC \ + --ignore LINE_CONTINUATIONS \ --ignore LINE_SPACING \ --ignore REPEATED_WORD \ --ignore SPDX_LICENSE_TAG \ |