Change in osmo-pcu[master]: Simplify tbf::set_polling()

pespin gerrit-no-reply at lists.osmocom.org
Fri Jul 30 15:05:59 UTC 2021


Hello osmith, Jenkins Builder, 

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

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

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

Change subject: Simplify tbf::set_polling()
......................................................................

Simplify tbf::set_polling()

When setting a POLL, it will always happen on PACCH, so all the CCCH
part makes no sense there. Let's drop it and move the logging of each
case to the caller, where logging file+line is more useful.

Change-Id: I242f97fd6f927131ac64c1a7c9c3812b6389de04
---
M src/nacc_fsm.c
M src/tbf.cpp
M src/tbf_dl.cpp
M src/tbf_dl.h
M src/tbf_dl_ass_fsm.c
M src/tbf_ul_ack_fsm.c
M src/tbf_ul_ass_fsm.c
7 files changed, 20 insertions(+), 46 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/11/25111/3
-- 
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/25111
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I242f97fd6f927131ac64c1a7c9c3812b6389de04
Gerrit-Change-Number: 25111
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
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/20210730/c5d0d9a6/attachment.htm>


More information about the gerrit-log mailing list