Patch Set 3:

(1 comment)

https://gerrit.osmocom.org/#/c/4273/2/src/hlr_vty_subscr.c
File src/hlr_vty_subscr.c:

Line 34: #define hexdump_buf(buf) osmo_hexdump_nospc((void*)buf, sizeof(buf))
> what's tricky here is that it can only be used on a static buf[].  Having t
I have instead renamed to hexdump_buf() in the most recent patch set.


-- 
To view, visit https://gerrit.osmocom.org/4273
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I42b3b70a0439a8f2e4964d7cc31e593c1f0d7537
Gerrit-PatchSet: 3
Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofm...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofm...@sysmocom.de>
Gerrit-HasComments: Yes

Reply via email to