dexter has uploaded this change for review. (
https://gerrit.osmocom.org/c/pysim/+/34689?usp=email )
Change subject: pySim-shell: do not use global variable app
......................................................................
pySim-shell: do not use global variable app
The constructor of Cmd2ApduTracer assigns self.cmd2 from the global
variable app. However the constructor has a parameter cmd2_app, which is
also a reference to app. Lets use cmd2_app here instead.
Related: OS#6210
Change-Id: Ibc415ee84ee2d1e419fd641a8dcb9bc592644fdb
---
M pySim-shell.py
1 file changed, 15 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/89/34689/1
diff --git a/pySim-shell.py b/pySim-shell.py
index c7b2607..a949509 100755
--- a/pySim-shell.py
+++ b/pySim-shell.py
@@ -276,7 +276,7 @@
class Cmd2ApduTracer(ApduTracer):
def __init__(self, cmd2_app):
- self.cmd2 = app
+ self.cmd2 = cmd2_app
def trace_response(self, cmd, sw, resp):
self.cmd2.poutput("-> %s %s" % (cmd[:10], cmd[10:]))
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/34689?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Ibc415ee84ee2d1e419fd641a8dcb9bc592644fdb
Gerrit-Change-Number: 34689
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-MessageType: newchange
