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 <laforge(a)osmocom.org> Gerrit-MessageType: newchange