<p style="white-space: pre-wrap; word-wrap: break-word;">please also note there' salso an IST (ISIM Service Table) in the ADF_ISIM.  Even if you don't have the time / energy to fully imlpement all of it, it would probably make sense to think a bit how to abstract the handling of SST+UST in a way that the same parser / code can be used for IST or even other future service tables structured in the same way.  There could e.g. be  a base class which then the SST/UST/IST specific classed derive from - probably mostly adding a list of string names for the services and a list of integers indicating those services to be printed?</p><p><a href="https://gerrit.osmocom.org/c/pysim/+/16991">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/pysim/+/16991/11/pySim-read.py">File pySim-read.py:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/pysim/+/16991/11/pySim-read.py@209">Patch Set #11, Line 209:</a> <code style="font-family:monospace,monospace">     try:</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this should be factored out in a separate function which knows all the AIDs (not just USIM and ISIM but at least also HPSIM?) and which simply returns the sim_type dict.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As there may be other [unknown] applications on the card, a different data format fo the return value might be best, where the [possibly unknown] application name is not the key? Just thiking aloud here.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/16991">change 16991</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/pysim/+/16991"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: pysim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I6680e8c7f86326f72b98a33338e0dd5b58e55013 </div>
<div style="display:none"> Gerrit-Change-Number: 16991 </div>
<div style="display:none"> Gerrit-PatchSet: 11 </div>
<div style="display:none"> Gerrit-Owner: herlesupreeth <herlesupreeth@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: herlesupreeth <herlesupreeth@gmail.com> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 05 Mar 2020 15:48:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>