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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: daniel <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Fri, 11 Feb 2022 13:58:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Reply via email to