<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><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><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? </p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not 100% sure.  After all, the Service Table is an attribute of the *instance* of a given card class.  Its contents can differer from one card to another of that product/class.  I'm not an expert on pySim code.  In theory, one would probably have something that separates the card specific code (and constant data related to a given card model) from the dynamic data that we are just reading from the filesystem of one specific card?</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">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></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think so.  In general, I would assume lots of things are just for historical reasons and the lack of any real architecture.  Any clean-up/refactoring is welcome.  With the tester in place we should be able to ensure we don't break functionality for one card type or the other.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I will try to re-factor the code for only pySim-read.py and address the detecting of applications on SIM card and based on that parse the service table. I may need some of your advice in terms of architecture since i have less experience in that aspect :). </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: Fri, 13 Mar 2020 07:37:12 +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>