Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34811?usp=email ) Change subject: mobile: vty: rework support enable/disable commands ...................................................................... Patch Set 1: (2 comments) File src/host/layer23/src/mobile/vty_interface.c: Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11921): https://gerrit.osmocom.org/c/osmocom-bb/+/34811/comment/87025cb1_995e3c2e PS1, Line 2054: #define SUP_EN_DI(item, cmd, desc, restart) \ Macros with multiple statements should be enclosed in a do - while loop Robot Comment from checkpatch (run ID jenkins-gerrit-lint-11921): https://gerrit.osmocom.org/c/osmocom-bb/+/34811/comment/6835087e_44feb8f2 PS1, Line 2085: #define SET_EN_DI(item, cmd, desc, restart) \ Macros with multiple statements should be enclosed in a do - while loop -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34811?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: Ibfef31f6dd7694dad535ad07fdf47bd32571b7b2 Gerrit-Change-Number: 34811 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-CC: Jenkins Builder Gerrit-Comment-Date: Wed, 18 Oct 2023 13:47:51 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment