Patch Set 2: (2 comments)
https://gerrit.osmocom.org/#/c/4311/2/src/ctrl.c File src/ctrl.c: Line 119: do { \ > would indeed; shall we move it now? If we won't do it now it might be easily forgotten. Line 156: /* not used yet */ > I first wrote this and then figured that no-one is using these values anywa Erm, could you clarify? Who's using the other part of the same function? -- To view, visit https://gerrit.osmocom.org/4311 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I98ee6a06b3aa6a67adb868e0b63b0e04eb42eb50 Gerrit-PatchSet: 2 Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Max <[email protected]> Gerrit-Reviewer: Neels Hofmeyr <[email protected]> Gerrit-HasComments: Yes
