Attention is currently required from: fixeria.
dexter has posted comments on this change. (
https://gerrit.osmocom.org/c/pysim/+/33954 )
Change subject: utils: tolerate uninitialized fields in dec_addr_tlv
......................................................................
Patch Set 2:
(1 comment)
File pySim/utils.py:
https://gerrit.osmocom.org/c/pysim/+/33954/comment/c6a3fb52_0e55bc8c
PS1, Line 775: if tlv[2] == [0xff] * len(tlv[2]):
Not critical: I would recommend doing `if all([v ==
0xff for v in tlv[2]]):`, as it's more self-expl […]
Self explanatory is in the
eye of the beholder.
--
To view, visit
https://gerrit.osmocom.org/c/pysim/+/33954
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Idc0a92ea88756266381c8da2ad62de061a8ea7a1
Gerrit-Change-Number: 33954
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 28 Jul 2023 08:37:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment