<p><a href="https://gerrit.osmocom.org/c/pysim/+/23473">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/pysim/+/23473/1/pySim-shell.py">File pySim-shell.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/+/23473/1/pySim-shell.py@a259">Patch Set #1, Line 259:</a> <code style="font-family:monospace,monospace"> #</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">the fixme probably still applies to DF_TELECOM and DF_GSM, where it's currently unclear how to do that better.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/pysim/+/23473/1/pySim-shell.py@266">Patch Set #1, Line 266:</a> <code style="font-family:monospace,monospace">                   if "a0000000871002" in a:</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I would move this entire new block into a function / method, rather than only having it in 'main'.  This way it can be used in the future from possibly other parts of the code.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also, please avoid hard-coding some specific AIDs here.  The CardApplication classes should themselves "know" their AID and register somewhere with common infrastructure (either by explicit code or by some kind of syntactic sugar).</p><p style="white-space: pre-wrap; word-wrap: break-word;">This loop then should iterate over the AIDs in the EF_DIR and try to find an application registered for the AID.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This way we can always easily add support for new applications by simply creating another [instance of] a derived class of CardApplication, and not have to change some kind of hard-coded duplicated aid list in this iterator function.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/23473">change 23473</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/+/23473"/><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: I42ee23375f98e6322708c1c4db6d65e1425feecd </div>
<div style="display:none"> Gerrit-Change-Number: 23473 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 23 Mar 2021 17:54:28 +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>