osmo-hlr[master]: ctrl: completely replace all CTRL commands

Max gerrit-no-reply at lists.osmocom.org
Tue Oct 17 15:34:01 UTC 2017


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 <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-HasComments: Yes


More information about the gerrit-log mailing list