Change in osmo-pcu[master]: tbf_free: Get rid of uneeded tbf_state transition

pespin gerrit-no-reply at lists.osmocom.org
Mon Aug 23 16:28:14 UTC 2021


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

Change subject: tbf_free: Get rid of uneeded tbf_state transition
......................................................................

tbf_free: Get rid of uneeded tbf_state transition

We are freeing the object immediately afterwards anyway, so no need to
pretend it went through the normal state release.
Leaving current state as it is actually provides more information on
what was the status/state at the time the TBF had to be freed.

Change-Id: I3016caaccc2c43e1e300f3c6042d69f8adcd9d69
---
M src/tbf.cpp
M tests/alloc/AllocTest.err
M tests/app_info/AppInfoTest.err
M tests/tbf/TbfTest.err
4 files changed, 49,178 insertions(+), 98,246 deletions(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved




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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I3016caaccc2c43e1e300f3c6042d69f8adcd9d69
Gerrit-Change-Number: 25055
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: osmith <osmith 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/20210823/55a17ea4/attachment.htm>


More information about the gerrit-log mailing list