Attention is currently required from: Hoernchen, dexter.

laforge has posted comments on this change by laforge. ( 
https://gerrit.osmocom.org/c/pysim/+/43254?usp=email )

Change subject: ts_102_221: support EF.ARR record decoding for DF, not just EF
......................................................................


Patch Set 1: Code-Review+2

(2 comments)

This change is ready for review.

File pySim/ts_102_221.py:

https://gerrit.osmocom.org/c/pysim/+/43254/comment/e68a4f0c_52a838ee?usp=email :
PS1, Line 760:         read_arr_argparser = 
copy.copy(LinFixedEF.ShellCommands.read_rec_dec_parser)
> When I try this out and do a "read_record_decoded --help", I also get the 
> --decode-for-df parameter  […]
Done


https://gerrit.osmocom.org/c/pysim/+/43254/comment/f3893a73_37079b84?usp=email :
PS1, Line 761:         read_arr_argparser.add_argument('--decode-for-df', 
action='store_true')
> Maybe adding a help text like: help='decode EF. […]
Done



--
To view, visit https://gerrit.osmocom.org/c/pysim/+/43254?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I05e629e8b7dc705730d7039d2d0170ee24f8f844
Gerrit-Change-Number: 43254
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Hoernchen <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Attention: Hoernchen <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Sat, 22 Aug 2026 08:22:40 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: dexter <[email protected]>

Reply via email to