[MERGED] osmo-pcu[master]: Fix warnings

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/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Thu Dec 21 11:40:33 UTC 2017


Harald Welte has submitted this change and it was merged.

Change subject: Fix warnings
......................................................................


Fix warnings

Fix warnings detected by compiler and coverity scan.

Change-Id: If463c7f8769e18d3df74837f0cb0f545cca9b23e
Fixes: CID181479
---
M src/bts.cpp
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/bts.cpp b/src/bts.cpp
index 867658e..3655567 100644
--- a/src/bts.cpp
+++ b/src/bts.cpp
@@ -219,7 +219,7 @@
 	}
 
 	/* The static allocator might have already registered the counter group.
-	   If this happens and we still called explicitly (in tests/* for example)
+	   If this happens and we still called explicitly (in tests/ for example)
 	   than just allocate the group with different index.
 	   This shall be removed once weget rid of BTS singleton */
 	if (rate_ctr_get_group_by_name_idx(bts_ctrg_desc.group_name_prefix, 0))
@@ -1439,7 +1439,7 @@
 			return;
 		}
 		LOGPTBFUL(ul_tbf, LOGL_ERROR,
-			"RX: [PCU <- BTS] %s FIXME: Packet resource request\n");
+			"RX: [PCU <- BTS] FIXME: Packet resource request\n");
 
 		/* Reset N3101 counter: */
 		ul_tbf->m_n3101 = 0;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If463c7f8769e18d3df74837f0cb0f545cca9b23e
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list