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 15:14:36 UTC 2021


Hello osmith, Jenkins Builder, dexter, 

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/c/osmo-pcu/+/25055

to look at the new patch set (#3).

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(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/55/25055/3
-- 
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-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210823/79d03cb3/attachment.htm>


More information about the gerrit-log mailing list