[MERGED] libosmocore[master]: cosmetic: log: fix typo, clarify msg for rate_ctr name mangling

2018-04-16 Thread Harald Welte
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

2018-04-16 Thread Harald Welte

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

2018-04-15 Thread Neels Hofmeyr

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