<p><a href="https://gerrit.osmocom.org/c/pysim/+/25548">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/+/25548/1/pySim/card_handler.py">File pySim/card_handler.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/+/25548/1/pySim/card_handler.py@38">Patch Set #1, Line 38:</a> <code style="font-family:monospace,monospace">                print("Ready for Programming: Insert card now (or CTRL-C to cancel)")</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Does it make sense to print this message, when sl is not set? It looks like it makes more sense to m […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">the CardHandler class suffers a bit from design problems. When we use the normal CardHandler, ther is indeed no reason why not giving it an sl object. I just thought i could make it optional here as well, just for completeness but I think we should drop that.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For CardHandlerAuto pySim-shell is not using the sl object and pySim-prog is using it. There we need this feature. In any case, there at least the message printing makes sense since it executes the get command (self.__exec_cmd(self.cmds['get'])) so I think everything is fine here.</p><p style="white-space: pre-wrap; word-wrap: break-word;">One big problem is that the mechanically automated reader we are using has technical problems with its built in reader. If the built in reader would behave like any other reader things would be much more straight forward.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/25548">change 25548</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/+/25548"/><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: I0f793aec51751b7c7b87d55b66326cce9970274e </div>
<div style="display:none"> Gerrit-Change-Number: 25548 </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-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 29 Sep 2021 14:01:31 +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: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>