Change in osmo-ci[master]: lint: checkpatch_osmo.sh: ignore UNSPECIFIED_INT

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

neels gerrit-no-reply at lists.osmocom.org
Wed Sep 15 17:11:21 UTC 2021


neels has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/25441 )

Change subject: lint: checkpatch_osmo.sh: ignore UNSPECIFIED_INT
......................................................................

lint: checkpatch_osmo.sh: ignore UNSPECIFIED_INT

As suggested by Pau, let's remove this check because it isn't useful for us.

Related: OS#5087
Change-Id: I15345a154f490362093e46fc0df75e75bbb237fe
---
M lint/checkpatch/checkpatch_osmo.sh
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve
  neels: Looks good to me, approved



diff --git a/lint/checkpatch/checkpatch_osmo.sh b/lint/checkpatch/checkpatch_osmo.sh
index 7ef8ca4..b9860b4 100755
--- a/lint/checkpatch/checkpatch_osmo.sh
+++ b/lint/checkpatch/checkpatch_osmo.sh
@@ -32,6 +32,7 @@
 # * SPLIT_STRING: we do split long messages over multiple lines
 # * STRING_FRAGMENTS: sometimes used intentionally to improve readability
 # * UNNECESSARY_INT: not followed (see https://gerrit.osmocom.org/c/libosmocore/+/25345)
+# * UNSPECIFIED_INT: not followed (doesn't seem useful for us)
 
 $SCRIPT_DIR/checkpatch.pl \
 	--exclude '\.(ok|err)$' \
@@ -64,6 +65,7 @@
 	--ignore SPLIT_STRING \
 	--ignore STRING_FRAGMENTS \
 	--ignore UNNECESSARY_INT \
+	--ignore UNSPECIFIED_INT \
 	--max-line-length 120 \
 	--no-signoff \
 	--no-tree \

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/25441
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I15345a154f490362093e46fc0df75e75bbb237fe
Gerrit-Change-Number: 25441
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210915/a11e44b8/attachment.htm>


More information about the gerrit-log mailing list