Change in osmo-pcu[master]: tbf: Set m_created_ts in constructor

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

pespin gerrit-no-reply at lists.osmocom.org
Mon Nov 8 15:26:19 UTC 2021


pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/26155 )

Change subject: tbf: Set m_created_ts in constructor
......................................................................

tbf: Set m_created_ts in constructor

This way the timestamp is also set for dummy reject TBFs. For other
TBFs, setup() is called immediatelly after calling the constructor, so
we are fine too.

Change-Id: I2966ec7f3f9161d528a173d94797b72d1398c747
---
M src/tbf.cpp
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  fixeria: Looks good to me, but someone else must approve
  neels: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/tbf.cpp b/src/tbf.cpp
index 2d6139b..4bd87dd 100644
--- a/src/tbf.cpp
+++ b/src/tbf.cpp
@@ -137,6 +137,8 @@
 	m_llc.init();
 
 	m_name_buf[0] = '\0';
+
+	m_created_ts = time(NULL);
 }
 
 
@@ -657,7 +659,6 @@
 	if (ms_mode(m_ms) != GPRS)
 		enable_egprs();
 
-	m_created_ts = time(NULL);
 	/* select algorithm */
 	rc = the_pcu->alloc_algorithm(bts, this, single_slot, use_trx);
 	/* if no resource */

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/26155
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I2966ec7f3f9161d528a173d94797b72d1398c747
Gerrit-Change-Number: 26155
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211108/0fdd765e/attachment.htm>


More information about the gerrit-log mailing list