laforge submitted this change.

View Change


Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
pySim-prog.py: Resolve possible variable use before assignment

pySim-prog.py:741:7: E0606: Possibly using variable 'cp' before assignment (possibly-used-before-assignment)

Change-Id: I6ab307db378d2ca76dfeae53dc3befa7c103974d
---
M pySim-prog.py
1 file changed, 12 insertions(+), 0 deletions(-)

diff --git a/pySim-prog.py b/pySim-prog.py
index 3f506de..db9c327 100755
--- a/pySim-prog.py
+++ b/pySim-prog.py
@@ -723,6 +723,7 @@
card.erase()
card.reset()

+ cp = None
# Generate parameters
if opts.source == 'cmdline':
cp = gen_parameters(opts)

To view, visit change 36903. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I6ab307db378d2ca76dfeae53dc3befa7c103974d
Gerrit-Change-Number: 36903
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged