osmith has posted comments on this change. ( https://gerrit.osmocom.org/13588 )
Change subject: GSUP routing: use Message Class IE ...................................................................... Patch Set 2: Change looks good, but as Vadim commented in [1], it would be nice to unify all the GSUP forwarding log messages. I have done it in patchset 12 and upwards of [2]. Would you like me to rebase this patch and make it use that unified log macro? [1]: https://gerrit.osmocom.org/#/c/osmo-hlr/+/13006/8/src/hlr.c@478 [2]: https://gerrit.osmocom.org/#/c/osmo-hlr/+/13006/ -- To view, visit https://gerrit.osmocom.org/13588 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8dc3967d9372d63e9d57ca2608dd3316edb234a4 Gerrit-Change-Number: 13588 Gerrit-PatchSet: 2 Gerrit-Owner: Neels Hofmeyr <[email protected]> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-CC: Vadim Yanitskiy <[email protected]> Gerrit-CC: osmith <[email protected]> Gerrit-Comment-Date: Mon, 15 Apr 2019 08:20:20 +0000 Gerrit-HasComments: No Gerrit-HasLabels: No
