<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 11:</p><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p><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></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Creating a class for Service Table sounds good. Do you think having Service Table object as member variable for each card type is a good idea? And BTW, in pySim-read.py any reason why cla byte and sel_ctrl are not set based on card type (its done properly in pySim-prog.py)?</p><p><a href="https://gerrit.osmocom.org/c/pysim/+/16991">View Change</a></p><ul style="list-style: none; padding: 0;"></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: Sat, 07 Mar 2020 19:35:28 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>