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. -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/30619 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: If18cb4a48237751e0dddede6793191b36dfe386d Gerrit-Change-Number: 30619 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Comment-Date: Fri, 16 Dec 2022 11:48:32 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <[email protected]> Gerrit-MessageType: comment
