Attention is currently required from: dexter, fixeria, laforge, merlinchlosta.
Hello Jenkins Builder, dexter, fixeria, merlinchlosta,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/pysim/+/35427?usp=email
to look at the new patch set (#6).
The following approvals got outdated and were removed:
Code-Review+1 by fixeria, Verified-1 by Jenkins Builder
Change subject: ts_31_102: Start to use construct for EF.SUCI_Calc_Info
......................................................................
ts_31_102: Start to use construct for EF.SUCI_Calc_Info
We cannot fully switch to construct for all of it easily due to
the priority value and the ordering/sorting by priority implemented
in the hand-coded version. But we can at least migrate the
encode/decode of the hnet_pubkey_list via construct.
Change-Id: I4ad5ea57bab37c2dc218e7752d538aa4cdc36ee3
---
M pySim/ts_31_102.py
M tests/test_utils.py
2 files changed, 44 insertions(+), 62 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/27/35427/6
--
To view, visit
https://gerrit.osmocom.org/c/pysim/+/35427?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I4ad5ea57bab37c2dc218e7752d538aa4cdc36ee3
Gerrit-Change-Number: 35427
Gerrit-PatchSet: 6
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: merlinchlosta <merlin.chlosta(a)rub.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: merlinchlosta <merlin.chlosta(a)rub.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: newpatchset