Attention is currently required from: osmith.
neels has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-ci/+/29359
)
Change subject: lint: ignore MACRO_WITH_FLOW_CONTROL
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
i have made one too, OSMO_NAME_C_IMPL(), but still i think it's ok to let the linter
complain about it. because in general we should avoid flow control macros, and for the
very rare cases where we allow it, it should be good enough to find someone to remove the
linter vote before merging
--
To view, visit
https://gerrit.osmocom.org/c/osmo-ci/+/29359
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I79da5a426db59031e3b16aecedeaa1498c91e847
Gerrit-Change-Number: 29359
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 15 Sep 2022 12:06:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment