Attention is currently required from: laforge, pespin.
View Change
1 comment:
Patchset:
Patch Set #1:
I'm not sure about this one is the proper way to check for 3digit. […]
There can be 3-digit MNCs starting with 0. MNC=099 is *not* equal to MNC=99, so I think using strlen() here is better, because "cgi_ps.rai.lac.plmn.mnc > 99" does not distinguish between MNC=099 and MNC=99.
To view, visit change 28485. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I819bdb0cc126113542547e9c397673a2dee0f168
Gerrit-Change-Number: 28485
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-CC: Jenkins Builder
Gerrit-Attention: laforge <laforge@osmocom.org>
Gerrit-Attention: pespin <pespin@sysmocom.de>
Gerrit-Comment-Date: Wed, 29 Jun 2022 17:07:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin@sysmocom.de>
Gerrit-MessageType: comment