Attention is currently required from: laforge.
lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-msc/+/38410?usp=email )
Change subject: vlr: add a logging layer ......................................................................
Patch Set 6:
(2 comments)
Patchset:
PS5:
whoy doesn't LOGVLR take a vlr as first pointer and then print the context? Like we usually do in o […]
I only converted the code, even there is also LOGVSUBP
File src/libvlr/vlr.c:
https://gerrit.osmocom.org/c/osmo-msc/+/38410/comment/b46b52dd_15f6ea22?usp=... : PS5, Line 457: LOGVLR(LOGL_DEBUG, "freeing VLR subscr %s (max total use count was %d)\n", vlr_subscr_name(vsub),
So I'd suggest something like `LOGVSUB(vsub, LOGL_DEBUG, "freeing VLR subscr (max total use count wa […]
Done
-
lynxis lazus