<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/pysim/+/24024">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">[pylint] fix non-existing 'res' in EF_CNL._encode_record_hex()<br><br>pySim/ts_51_011.py:648:45: E0602: Undefined variable 'res' (undefined-variable)<br>pySim/ts_51_011.py:648:68: E0602: Undefined variable 'res' (undefined-variable)<br>pySim/ts_51_011.py:649:24: E0602: Undefined variable 'res' (undefined-variable)<br><br>Change-Id: I039e2b271dd3ab44f437108c5e8def43e97098a3<br>---<br>M pySim/ts_51_011.py<br>1 file changed, 5 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/pySim/ts_51_011.py b/pySim/ts_51_011.py</span><br><span>index 15a7907..7e8e627 100644</span><br><span>--- a/pySim/ts_51_011.py</span><br><span>+++ b/pySim/ts_51_011.py</span><br><span>@@ -649,8 +649,11 @@</span><br><span>         return res</span><br><span>     def _encode_record_hex(self, in_json):</span><br><span>         plmn = enc_plmn(in_json['mcc'], in_json['mnc'])</span><br><span style="color: hsl(0, 100%, 40%);">-        return b2h(pack('!3sBBB', h2b(plmn), res['network_subset'], res['service_provider_id'],</span><br><span style="color: hsl(0, 100%, 40%);">-                        res['corporate_id']))</span><br><span style="color: hsl(120, 100%, 40%);">+        return b2h(pack('!3sBBB',</span><br><span style="color: hsl(120, 100%, 40%);">+                        h2b(plmn),</span><br><span style="color: hsl(120, 100%, 40%);">+                        in_json['network_subset'],</span><br><span style="color: hsl(120, 100%, 40%);">+                        in_json['service_provider_id'],</span><br><span style="color: hsl(120, 100%, 40%);">+                        in_json['corporate_id']))</span><br><span> </span><br><span> # TS 51.011 Section 10.3.31</span><br><span> class EF_NIA(LinFixedEF):</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/24024">change 24024</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/pysim/+/24024"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: pysim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I039e2b271dd3ab44f437108c5e8def43e97098a3 </div>
<div style="display:none"> Gerrit-Change-Number: 24024 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>