Attention is currently required from: daniel, dexter.
laforge has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-iuh/+/27136 )
Change subject: ranap_common: add decoder for RAB-FailedList
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File src/ranap_common.c:
https://gerrit.osmocom.org/c/osmo-iuh/+/27136/comment/a372e934_9840a3a1
PS1, Line 598: RANAP_DEBUG
a more general review issue that seems to affect all of the surrounding code: A decoder
error should be more than a _DEBUG_ message. Not important to fix it in this patch, but
in general that's very unusual and should probably be addressed.
--
To view, visit
https://gerrit.osmocom.org/c/osmo-iuh/+/27136
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I0f42487968f2a39f2425068c001950509a3bd28f
Gerrit-Change-Number: 27136
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 11 Feb 2022 13:58:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment