[PATCH] openbsc[master]: log: abis_rsl: don't log 'error' when there is no error

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Mon Aug 22 23:24:40 UTC 2016


Review at  https://gerrit.osmocom.org/749

log: abis_rsl: don't log 'error' when there is no error

The message 'RF Channel Release due error 0' keeps catching my eye because
it says 'error' even though the error code is zero, i.e. no error.
This shall end now.

Change-Id: Ie0b9d62e8ce85a096c963931e0ae5527b8dc490a
---
M openbsc/src/libbsc/abis_rsl.c
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/49/749/1

diff --git a/openbsc/src/libbsc/abis_rsl.c b/openbsc/src/libbsc/abis_rsl.c
index b84a0b5..096ea29 100644
--- a/openbsc/src/libbsc/abis_rsl.c
+++ b/openbsc/src/libbsc/abis_rsl.c
@@ -769,7 +769,11 @@
 	msg->lchan = lchan;
 	msg->dst = lchan->ts->trx->rsl_link;
 
-	DEBUGP(DRSL, "%s RF Channel Release CMD due error %d\n", gsm_lchan_name(lchan), error);
+	if (error)
+		DEBUGP(DRSL, "%s RF Channel Release due to error: %d\n",
+		       gsm_lchan_name(lchan), error);
+	else
+		DEBUGP(DRSL, "%s RF Channel Release\n", gsm_lchan_name(lchan));
 
 	if (error) {
 		/*

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie0b9d62e8ce85a096c963931e0ae5527b8dc490a
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>



More information about the gerrit-log mailing list