laforge has submitted this change. (
https://gerrit.osmocom.org/c/pysim/+/35824?usp=email )
Change subject: pylint: transport/__init__.py
......................................................................
pylint: transport/__init__.py
pySim/transport/__init__.py:139:0: C0325: Unnecessary parens after 'if' keyword
(superfluous-parens)
Change-Id: Ibeeb7d6fde40bb37774bbd09ad185203ac7bcb48
---
M pySim/transport/__init__.py
1 file changed, 12 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/pySim/transport/__init__.py b/pySim/transport/__init__.py
index dcecfdd..dd06af8 100644
--- a/pySim/transport/__init__.py
+++ b/pySim/transport/__init__.py
@@ -136,7 +136,7 @@
# available. There are two SWs commonly used for this 9fxx (sim) and
61xx (usim), where
# xx is the number of response bytes available.
# See also:
- if (sw is not None):
+ if sw is not None:
while ((sw[0:2] == '9f') or (sw[0:2] == '61')):
# SW1=9F: 3GPP TS 51.011 9.4.1, Responses to commands which
are correctly executed
# SW1=61: ISO/IEC 7816-4, Table 5 — General meaning of the
interindustry values of SW1-SW2
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/35824?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: Ibeeb7d6fde40bb37774bbd09ad185203ac7bcb48
Gerrit-Change-Number: 35824
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-MessageType: merged
