diff options
author | Oliver Smith <osmith@sysmocom.de> | 2021-07-07 10:57:08 +0200 |
---|---|---|
committer | Oliver Smith <osmith@sysmocom.de> | 2021-07-07 11:28:38 +0200 |
commit | 2cceff4535fb31aa2f8da4b029e74c8d53824074 (patch) | |
tree | 97ef5c65a68bc256cc7b1d938258e3a468672e75 /lint/checkpatch | |
parent | 497ce6743a9c06c78b26cfa57298fff2275f0b2b (diff) |
lint: ignore COMPLEX_MACRO
Complains about, for example:
#define MDCX4 \
"MDCX 18983217 1@mgw MGCP 1.0\r\n" \
"M: sendrecv\r" \
"C: 2\r\n" \
"I: %s\r\n" \
"L: p:20, a:AMR, nt:IN\r\n" \
"\n" \
"v=0\r\n" \
"o=- %s 23 IN IP4 5.6.7.8\r\n" \
"c=IN IP4 5.6.7.8\r\n" \
"t=0 0\r\n" \
"m=audio 4441 RTP/AVP 99\r\n" \
"a=rtpmap:99 AMR/8000\r\n" \
"a=ptime:40\r\n"
Related: OS#5087
Change-Id: Ic9d752ca841161a62e3631c84b4237a0d8594363
Diffstat (limited to 'lint/checkpatch')
-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 b98e98b..75fef8e 100755 --- a/lint/checkpatch/checkpatch_osmo.sh +++ b/lint/checkpatch/checkpatch_osmo.sh @@ -9,6 +9,7 @@ SCRIPT_DIR="$(dirname "$(realpath "$0")")" # * ASSIGN_IN_IF: not followed (e.g. 'if ((u8 = gsup_msg->cause))') # * AVOID_EXTERNS: we do use externs # * BLOCK_COMMENT_STYLE: we don't use a trailing */ on a separate line +# * COMPLEX_MACRO: we don't use parentheses when building macros of strings across multiple lines # * CONSTANT_COMPARISON: not followed: "Comparisons should place the constant on the right side" # * EMBEDDED_FUNCTION_NAME: often __func__ isn't used, arguably not much benefit in changing this when touching code # * EXECUTE_PERMISSIONS: not followed, files need to be executable: git-version-gen, some in debian/ @@ -29,6 +30,7 @@ $SCRIPT_DIR/checkpatch.pl \ --ignore ASSIGN_IN_IF \ --ignore AVOID_EXTERNS \ --ignore BLOCK_COMMENT_STYLE \ + --ignore COMPLEX_MACRO \ --ignore CONSTANT_COMPARISON \ --ignore EMBEDDED_FUNCTION_NAME \ --ignore EXECUTE_PERMISSIONS \ |