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

Max gerrit-no-reply at lists.osmocom.org
Tue Oct 24 14:38:38 UTC 2017


Patch Set 5: Code-Review-1

(3 comments)

https://gerrit.osmocom.org/#/c/4311/5/src/ctrl.c
File src/ctrl.c:

Line 98: 		cmd->reply = NULL;
Maybe it's better to return smth like "unknown/unexpected error"? It would be more descriptive than empty message.


Line 389: 					  _LAST_CTRL_NODE_HLR);
So we not only replace the commands, we also change the way we setup and use ctrl interface? I think it would be better to split "change interface" part into separate patch.


https://gerrit.osmocom.org/#/c/4311/5/src/ctrl.h
File src/ctrl.h:

Line 30: 	_LAST_CTRL_NODE_HLR
Same as ctrl.c - shouldn't this belong to a separate patch? This have nothing to do with actual ctrl commands change - would be the same with old commands too?


-- 
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: 5
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