Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/35096?usp=email )
Change subject: gprs_rlcmac_sched: fix condition for generating dummy blocks on idle ......................................................................
Patch Set 2:
(1 comment)
File src/gprs_rlcmac_sched.cpp:
https://gerrit.osmocom.org/c/osmo-pcu/+/35096/comment/bc33e489_dccdf6cf PS2, Line 492: skip_idle = skip_idle && trx != 0; I think now that we don't have the #ifdef, we can really simplify this to:
bool skip_dummy; if (bts->gen_dummy_blk) { skip_dummy = false; } else { const unsigned num_tbfs = pdch->num_tbfs(GPRS_RLCMAC_DL_TBF) + pdch->num_tbfs(GPRS_RLCMAC_UL_TBF); skip_dummy = (num_tbfs == 0); }
This can again be simplified to:
bool skip_dummy = bts->gen_dummy_blk; if (skip_dummy) { /* Only skip if no one is listening on this PDCH: */ const unsigned num_tbfs = pdch->num_tbfs(GPRS_RLCMAC_DL_TBF) + pdch->num_tbfs(GPRS_RLCMAC_UL_TBF); skip_dummy = (num_tbfs == 0); }