laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/27881 ) Change subject: SimCard.reset(): fix SyntaxWarning: 'is' with a literal ...................................................................... SimCard.reset(): fix SyntaxWarning: 'is' with a literal Change-Id: I5860179acd1cb330e91dbe5b57cd60cd520f2d9d --- M pySim/cards.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/pySim/cards.py b/pySim/cards.py index 1ed1634..b186044 100644 --- a/pySim/cards.py +++ b/pySim/cards.py @@ -63,7 +63,7 @@ def reset(self): rc = self._scc.reset_card() - if rc is 1: + if rc == 1: return self._scc.get_atr() else: return None -- To view, visit https://gerrit.osmocom.org/c/pysim/+/27881 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I5860179acd1cb330e91dbe5b57cd60cd520f2d9d Gerrit-Change-Number: 27881 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged