laforge has uploaded this change for review. ( 
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(-)



  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/24/35824/1

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: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-MessageType: newchange

Reply via email to