Change in libosmo-sccp[master]: Allow ASP role to be configured

[pespin]

pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/15974 )

Change subject: Allow ASP role to be configured
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.osmocom.org/c/libosmo-sccp/+/15974/3/src/osmo_ss7_vty.c 
File src/osmo_ss7_vty.c:

https://gerrit.osmocom.org/c/libosmo-sccp/+/15974/3/src/osmo_ss7_vty.c@734 
PS3, Line 734: 	if (asp->cfg.role_set_by_vty) {
If this flag is only used this way, I don't see the point in having it. If default is OSMO_SS7_ASP_ROLE_SG, then do if(asp->cfg.role != OSMO_SS7_ASP_ROLE_SG)

Or even let it be writen to config always.



-- 
To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/15974
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: I2df9cd9747ad5c9a05d567d9a71bab6184c53674
Gerrit-Change-Number: 15974
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin at sysmocom.de>
Gerrit-Comment-Date: Tue, 05 Nov 2019 19:55:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191105/356b435b/attachment.htm>