Attention is currently required from: dexter, fixeria, laforge.

laforge uploaded patch set #4 to this change.

View Change

The following approvals got outdated and were removed: Verified-1 by Jenkins Builder

Get rid of [now] superfluous HexAdapter

With the introduction of using osmocom.construct.{Bytes,GreedyBytes}
in Change-Id I1c8df6350c68aa408ec96ff6cd1e405ceb1a4fbb we don't have a
need for wrapping each instance of Bytes or GreedyBytes into a
HexAdapter anymore. The osmocom.construct.{Bytes,GreedyBytes} will
automatically perform the related hex-string-to-bytes conversion if
needed - and during printing we have osmocom.utils.JsonEncoder that
makes sure to convert any bytes type to a hex-string.

Change-Id: I9c77e420c314f5e74458628dc4e767eab6d97123
---
M pySim/cat.py
M pySim/cdma_ruim.py
M pySim/euicc.py
M pySim/global_platform/__init__.py
M pySim/global_platform/install_param.py
M pySim/gsm_r.py
M pySim/sms.py
M pySim/sysmocom_sja2.py
M pySim/ts_102_221.py
M pySim/ts_102_310.py
M pySim/ts_31_102.py
M pySim/ts_31_103.py
M pySim/ts_31_103_shared.py
M pySim/ts_51_011.py
M tests/unittests/test_utils.py
15 files changed, 202 insertions(+), 197 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/37/40237/4

To view, visit change 40237. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newpatchset
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I9c77e420c314f5e74458628dc4e767eab6d97123
Gerrit-Change-Number: 40237
Gerrit-PatchSet: 4
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Attention: laforge <laforge@osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Attention: dexter <pmaier@sysmocom.de>