Attention is currently required from: fixeria, phcoder.
dexter has posted comments on this change by phcoder. ( https://gerrit.osmocom.org/c/pysim/+/41786?usp=email )
Change subject: Print SMSC in pySim-read.py ......................................................................
Patch Set 15: Code-Review+1 Verified+1
(1 comment)
Patchset:
PS15: I have tried this out and it works fine. I have also set the EF.SMSP record 1 to all FF to see what happens. No problem, it just shows "SMSC: "
I would suggest to indent the SMSC: output to make more clear that it is related to SMSP. (similar to SIM Service Table, there you have the hex string and below are the parsed results with an indentation. Maybe also print "SMSC: (none)" in case it is not set.
In any case, those are just cosmetic suggestions. I think its ok the way it is and I don't want to block you. Also the output of pySim-read.py has some consistency issues here and there anyway.