Change in pysim[master]: ts_102_221: put select response decoder into separate file

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

dexter gerrit-no-reply at lists.osmocom.org
Mon Nov 8 16:20:44 UTC 2021


Hello Jenkins Builder, 

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/c/pysim/+/26160

to look at the new patch set (#3).

Change subject: ts_102_221: put select response decoder into separate file
......................................................................

ts_102_221: put select response decoder into separate file

The decoder function that decodes the select response has become more
and more complex over time, so it makes sense to put the code into a
separate file, lets also mark the functions we only call from inside the
decoder as private. This makes the code more readable and also helps to
prevent circular dependencies.

Change-Id: If5fae46bb40d0f285236a6ab287070125c69ed70
---
M pySim/sysmocom_sja2.py
M pySim/ts_102_221.py
A pySim/ts_102_221_select.py
M pySim/ts_31_102.py
M pySim/ts_31_103.py
M pySim/ts_51_011.py
6 files changed, 183 insertions(+), 155 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/60/26160/3
-- 
To view, visit https://gerrit.osmocom.org/c/pysim/+/26160
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: If5fae46bb40d0f285236a6ab287070125c69ed70
Gerrit-Change-Number: 26160
Gerrit-PatchSet: 3
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211108/30b4cf6a/attachment.htm>


More information about the gerrit-log mailing list