Attention is currently required from: pespin.
Jenkins Builder has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmo-netif/+/30757 )

Change subject: Introduce utils/osmo-amr-inspect program
......................................................................


Patch Set 4:

(2 comments)

File utils/osmo-amr-inspect.c:

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-2200):
https://gerrit.osmocom.org/c/libosmo-netif/+/30757/comment/e611cf0f_92bb0ce5
PS4, Line 51:   do { \
suspect code indent for conditional statements (8, 8)


Robot Comment from checkpatch (run ID jenkins-gerrit-lint-2200):
https://gerrit.osmocom.org/c/libosmo-netif/+/30757/comment/a2caea53_d807922b
PS4, Line 56:   } while(0)
space required before the open parenthesis '('



--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/30757
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: Iffa6cc2e5391b77e3097d4c3b8d3f5211427dbe2
Gerrit-Change-Number: 30757
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Wed, 21 Dec 2022 20:12:36 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to