[MERGED] libosmocore[master]: cosmetic: log: fix typo, clarify msg for rate_ctr name mangling
Harald Welte has submitted this change and it was merged. Change subject: cosmetic: log: fix typo, clarify msg for rate_ctr name mangling .. cosmetic: log: fix typo, clarify msg for rate_ctr name mangling Change-Id: I9a04d501698f8a3360ef9dcbf04b57c5ac10e63b --- M src/rate_ctr.c 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/rate_ctr.c b/src/rate_ctr.c index ec054c5..777a42a 100644 --- a/src/rate_ctr.c +++ b/src/rate_ctr.c @@ -169,8 +169,9 @@ goto err_free; } - LOGP(DLGLOBAL, LOGL_INFO, "Needed to mangle ounter group '%s' names still using '.' as " - "separator, please consider updating the application\n", desc->group_name_prefix); + LOGP(DLGLOBAL, LOGL_INFO, "Needed to mangle counter group '%s' names: it is still using '.' as " + "separator, which is not allowed. please consider updating the application\n", + desc->group_name_prefix); return desc_new; err_free: -- To view, visit https://gerrit.osmocom.org/7817 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9a04d501698f8a3360ef9dcbf04b57c5ac10e63b Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder
libosmocore[master]: cosmetic: log: fix typo, clarify msg for rate_ctr name mangling
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/7817 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9a04d501698f8a3360ef9dcbf04b57c5ac10e63b Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr Gerrit-Reviewer: Harald Welte Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: No
[PATCH] libosmocore[master]: cosmetic: log: fix typo, clarify msg for rate_ctr name mangling
Review at https://gerrit.osmocom.org/7817 cosmetic: log: fix typo, clarify msg for rate_ctr name mangling Change-Id: I9a04d501698f8a3360ef9dcbf04b57c5ac10e63b --- M src/rate_ctr.c 1 file changed, 3 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/17/7817/1 diff --git a/src/rate_ctr.c b/src/rate_ctr.c index ec054c5..777a42a 100644 --- a/src/rate_ctr.c +++ b/src/rate_ctr.c @@ -169,8 +169,9 @@ goto err_free; } - LOGP(DLGLOBAL, LOGL_INFO, "Needed to mangle ounter group '%s' names still using '.' as " - "separator, please consider updating the application\n", desc->group_name_prefix); + LOGP(DLGLOBAL, LOGL_INFO, "Needed to mangle counter group '%s' names: it is still using '.' as " + "separator, which is not allowed. please consider updating the application\n", + desc->group_name_prefix); return desc_new; err_free: -- To view, visit https://gerrit.osmocom.org/7817 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I9a04d501698f8a3360ef9dcbf04b57c5ac10e63b Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Neels Hofmeyr