Harald Welte has submitted this change and it was merged.
Change subject: Remove comment warning
......................................................................
Remove comment warning
It's unclear why the warning was placed next to commented log statement
to begin with, so let's just follow that warning's advice and drop it.
Change-Id: I3ef7a45d015a28fdadf75f97294bc5d4f825b8ae
---
M src/pcu_l1_if.cpp
1 file changed, 2 insertions(+), 3 deletions(-)
Approvals:
Harald Welte: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/pcu_l1_if.cpp b/src/pcu_l1_if.cpp
index c16ac0c..c2c6cc6 100644
--- a/src/pcu_l1_if.cpp
+++ b/src/pcu_l1_if.cpp
@@ -549,9 +549,8 @@
/* omit frame numbers not starting at a MAC block */
if (fn13 != 0 && fn13 != 4 && fn13 != 8)
return 0;
-#warning uncomment
-// LOGP(DL1IF, LOGL_DEBUG, "Time indication received: %d\n",
-// time_ind->fn % 52);
+
+ LOGP(DL1IF, LOGL_DEBUG, "Time indication received: %d\n", time_ind->fn
% 52);
BTS::main_bts()->set_current_frame_number(time_ind->fn);
return 0;
--
To view, visit https://gerrit.osmocom.org/3049
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3ef7a45d015a28fdadf75f97294bc5d4f825b8ae
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: Max <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>