Change in osmo-iuh[master]: ranap_common: add decoder for RAB-FailedList - gerrit-log

11 Feb 2022


      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@sysmocom.de
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel dwillmann@sysmocom.de
Gerrit-Reviewer: laforge laforge@osmocom.org
Gerrit-Reviewer: pespin pespin@sysmocom.de
Gerrit-Attention: daniel dwillmann@sysmocom.de
Gerrit-Attention: dexter pmaier@sysmocom.de
Gerrit-Comment-Date: Fri, 11 Feb 2022 13:58:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment