Attention is currently required from: laforge, pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30371 )
Change subject: pcu: TC_cs_max_dl: Fix race condition waiting for first DL data after X2002 ......................................................................
Patch Set 2: Code-Review-1
(1 comment)
File pcu/PCU_Tests.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/30371/comment/3f5c556e_d7f43... PS2, Line 1094: for (i := 0; i < 10; i := i + 1) { : f_rx_rlcmac_dl_block(dl_block, fn); : : if (not match(dl_block, tr_RLCMAC_DL_DUMMY_CTRL)) { : /* Received first data, starting processing: */ : break; : } : }
I thought about it, but decided to do it this way to avoid encapsulating more stuff, since in the en […]
Agreeing with Harald. Let's better avoid code duplication. The function can simply accept an out parameter, which will be holding the first non-dummy block.