<p><a href="https://gerrit.osmocom.org/c/pysim/+/17661">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/+/17661/1/pySim/cards.py">File pySim/cards.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/+/17661/1/pySim/cards.py@101">Patch Set #1, Line 101:</a> <code style="font-family:monospace,monospace">           if sw == '9000':</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">for PLMNwAcT there is further processing required (the same for OPLMNwAcT, HPLMNwAcT), so is it stil […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">oh, sorry, I didin't see the format_xplmn_w_act() function. nevermind for this one then.  If you like the idea, you mihgt want to implement a dictionary of 'transform' functions.  So that you hav a generic methid like read('PLMNwAcT') which then does the _scc.read_binary(EF['PLMNwAcT]), contains the if-clause and in case of success (90000) checks if the dict has an entry for 'PLMNwAcT'.  If yes, it could call the function reference in that dict, and return.  That way we save all this copy+paste boilerplate code.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But not required, this is just an idea...</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/17661">change 17661</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/+/17661"/><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: I14d7c2dc51fac6d5cf4a708a77ad23d252ba6094 </div>
<div style="display:none"> Gerrit-Change-Number: 17661 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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: Wed, 01 Apr 2020 08:52:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: herlesupreeth <herlesupreeth@gmail.com> </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>