Change in pysim[master]: [pylint] fix non-existing 'res' in EF_CNL._encode_record_hex()

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

fixeria gerrit-no-reply at lists.osmocom.org
Sun May 2 00:56:48 UTC 2021


fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/24024 )


Change subject: [pylint] fix non-existing 'res' in EF_CNL._encode_record_hex()
......................................................................

[pylint] fix non-existing 'res' in EF_CNL._encode_record_hex()

pySim/ts_51_011.py:648:45: E0602: Undefined variable 'res' (undefined-variable)
pySim/ts_51_011.py:648:68: E0602: Undefined variable 'res' (undefined-variable)
pySim/ts_51_011.py:649:24: E0602: Undefined variable 'res' (undefined-variable)

Change-Id: I039e2b271dd3ab44f437108c5e8def43e97098a3
---
M pySim/ts_51_011.py
1 file changed, 5 insertions(+), 2 deletions(-)



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

diff --git a/pySim/ts_51_011.py b/pySim/ts_51_011.py
index 15a7907..c3f5650 100644
--- a/pySim/ts_51_011.py
+++ b/pySim/ts_51_011.py
@@ -649,8 +649,11 @@
         return res
     def _encode_record_hex(self, in_json):
         plmn = enc_plmn(in_json['mcc'], in_json['mnc'])
-        return b2h(pack('!3sBBB', h2b(plmn), res['network_subset'], res['service_provider_id'],
-                        res['corporate_id']))
+        return b2h(pack('!3sBBB',
+			h2b(plmn),
+			in_json['network_subset'],
+			in_json['service_provider_id'],
+			in_json['corporate_id']))
 
 # TS 51.011 Section 10.3.31
 class EF_NIA(LinFixedEF):

-- 
To view, visit https://gerrit.osmocom.org/c/pysim/+/24024
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I039e2b271dd3ab44f437108c5e8def43e97098a3
Gerrit-Change-Number: 24024
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210502/7dd37408/attachment.htm>


More information about the gerrit-log mailing list