Change in osmo-ci[master]: lint: fix && complaints

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

laforge gerrit-no-reply at
Sun Jul 11 17:44:48 UTC 2021

laforge has submitted this change. ( )

Change subject: lint: fix && complaints

lint: fix && complaints

Don't complain with:
	ERROR:SPACING: space prohibited after that '&&' (ctx:ExW)
in code similar to:
	if (conn->conn->mode != MGCP_CONN_LOOPBACK
	    && conn->conn->mode != MGCP_CONN_RECV_ONLY
	    && !mgcp_rtp_end_remote_addr_available(&conn->end)) {

The check was supposed to complain about spaces if the && is used as
unary operator (to get the address of a goto label). But it's clearly
producing false positives in the Osmocom context with use as non-unary
operator, so remove this check.

Related: OS#5087
Change-Id: I7ce79e6b291b3a3dab6587a589eeef0a0bc53de9
M lint/checkpatch/
1 file changed, 1 insertion(+), 1 deletion(-)

  pespin: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved; Verified

diff --git a/lint/checkpatch/ b/lint/checkpatch/
index dc56812..ad99e0c 100755
--- a/lint/checkpatch/
+++ b/lint/checkpatch/
@@ -5072,7 +5072,7 @@
 				# unary operator, or a cast
 				} elsif ($op eq '!' || $op eq '~' ||
 					 $opv eq '*U' || $opv eq '-U' ||
-					 $opv eq '&U' || $opv eq '&&U') {
+					 $opv eq '&U') {  # Osmocom specific: &&U removed
 					if ($ctx !~ /[WEBC]x./ && $ca !~ /(?:\)|!|~|\*|-|\&|\||\+\+|\-\-|\{)$/) {
 						if (ERROR("SPACING",
 							  "space required before that '$op' $at\n" . $hereptr)) {

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I7ce79e6b291b3a3dab6587a589eeef0a0bc53de9
Gerrit-Change-Number: 24869
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith at>
Gerrit-Reviewer: laforge <laforge at>
Gerrit-Reviewer: pespin <pespin at>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list