osmith has uploaded this change for review.

View Change

lint: ignore MACRO_WITH_FLOW_CONTROL

It seems that we don't care about this one, e.g. here:
https://gerrit.osmocom.org/c/osmo-msc/+/28848

Change-Id: I79da5a426db59031e3b16aecedeaa1498c91e847
---
M lint/checkpatch/checkpatch_osmo.sh
1 file changed, 2 insertions(+), 0 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/59/29359/1

diff --git a/lint/checkpatch/checkpatch_osmo.sh b/lint/checkpatch/checkpatch_osmo.sh
index b0c3985..7a60ad6 100755
--- a/lint/checkpatch/checkpatch_osmo.sh
+++ b/lint/checkpatch/checkpatch_osmo.sh
@@ -68,6 +68,7 @@
 # * 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
+# * MACRO_WITH_FLOW_CONTROL: not followed
 # * MISSING_SPACE: warns about breaking strings at space characters, not useful for long strings of hex chars
 # * PREFER_DEFINED_ATTRIBUTE_MACRO: macros like __packed not defined in libosmocore
 # * PREFER_FALLTHROUGH: pseudo keyword macro "fallthrough" is not defined in libosmocore
@@ -108,6 +109,7 @@
 	--ignore LONG_LINE \
 	--ignore LONG_LINE_COMMENT \
 	--ignore LONG_LINE_STRING \
+	--ignore MACRO_WITH_FLOW_CONTROL \
 	--ignore MISSING_SPACE \
 	--ignore PREFER_DEFINED_ATTRIBUTE_MACRO \
 	--ignore PREFER_FALLTHROUGH \

To view, visit change 29359. To unsubscribe, or for help writing mail filters, visit settings.