Attention is currently required from: pespin, fixeria, dexter.
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/31145 )
Change subject: bts: add IMMEDIATE ASSIGNMENT via PCH transmission ......................................................................
Patch Set 16:
(2 comments)
File src/bts.h:
https://gerrit.osmocom.org/c/osmo-pcu/+/31145/comment/5734f1c4_a9443a55 PS16, Line 284: bool use_dt_confirm; what does "dt" mean?
File src/pcu_l1_if.cpp:
https://gerrit.osmocom.org/c/osmo-pcu/+/31145/comment/1578dbb2_536aaee9 PS5, Line 320: data[sizeof(tlli) + (PAGING_GROUP_LEN - 1 - i)]
I think its still ok to keep it like it is. If we add items like this to pcuif_proto. […]
i also dislike the pattern of unreadable encoding magic, especially in the middle of flow control and other decisions happening; that invites code dup and copy paste errors. Maybe adding packed structs for the other functions would not be a bad idea. (I'm not familiar with this code; if all of this is written in this way then I'd be fine to be outvoted, letting it be known that it objectively is ugly but we choose to live with it...)
If it should be unreadable encoding magic, then can we have a separate function doing the encoding?