Pau Espin Pedrol has posted comments on this change. ( https://gerrit.osmocom.org/13232 )
Change subject: vty: Ensure to properly save route pointcode and mask when writing ...................................................................... Patch Set 2: Code-Review+1 (1 comment) https://gerrit.osmocom.org/#/c/13232/2/src/osmo_ss7_vty.c File src/osmo_ss7_vty.c: https://gerrit.osmocom.org/#/c/13232/2/src/osmo_ss7_vty.c@365 PS2, Line 365: osmo_ss7_pointcode_print2(rtable->inst, rt->cfg.mask), I really don't like having this "2" API with a different static buffer instead of simply copying it it a local buffer in the caller, but not directly related to this commit I since it seems the API is already there. -- To view, visit https://gerrit.osmocom.org/13232 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-sccp Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ica32e83cbe8af2317cb07f8d8422a399fa537012 Gerrit-Change-Number: 13232 Gerrit-PatchSet: 2 Gerrit-Owner: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de> Gerrit-Comment-Date: Wed, 13 Mar 2019 11:55:31 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: Yes