Attention is currently required from: fixeria.
pespin has posted comments on this change by pespin. (
https://gerrit.osmocom.org/c/osmo-hnbgw/+/40263?usp=email )
Change subject: cnlink: Fix several funcs missing prefix
......................................................................
Patch Set 2:
(1 comment)
File include/osmocom/hnbgw/cnlink.h:
https://gerrit.osmocom.org/c/osmo-hnbgw/+/40263/comment/1cfb267f_e2de404e?u… :
PS2, Line 96: hnbgw_cnlink_term_and_free
COMMIT_MSG says adding prefix, but you're also
declaring a function here?
Yes because it was missing, and since I'm cleaning up
the name of the function I'm also taking the chance to add it here. It will be needed
in a follow-up patch anyway so it's fine having it here.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-hnbgw/+/40263?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: If20890bb99885e8c18732588376f8fdaa326d639
Gerrit-Change-Number: 40263
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Sun, 11 May 2025 21:44:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>