Attention is currently required from: fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/30619 )
Change subject: pdch: Introduce APIs to print PDCH name ......................................................................
Patch Set 2:
(1 comment)
File src/pdch.cpp:
https://gerrit.osmocom.org/c/osmo-pcu/+/30619/comment/aa14fd19_3d63aa0b PS2, Line 1346: pdch_name_buf
I still think it makes sense to do NULL checking here. […]
I am happy you agreed with Neels on doing that in another project/context; I really don't want to do this here. I really want to track if at some point in code the TBF is expected and it's NULL. This is also useful for new readers which wish to understand the requirements in different parts of the code.