<p>Max has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/11975">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Improve code style<br><br>* reorder imports to make pylint3 happy<br>* drop unused imports<br>* use proper spacing for list constants<br>* don't use reserved names for internal variables<br><br>The check was run as follows:<br>pylint3 -d C0103,C0301,C0410,C0326,R0913,R0901 ...<br>to disable useless warnings.<br><br>Change-Id: I5b90ee790f73dc509081401776911f25e43f1801<br>---<br>M osmopy/trap_helper.py<br>M scripts/ctrl2cgi.py<br>M scripts/soap.py<br>3 files changed, 21 insertions(+), 23 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/python/osmo-python-tests refs/changes/75/11975/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/osmopy/trap_helper.py b/osmopy/trap_helper.py</span><br><span>index d07ebd3..d4a3b75 100644</span><br><span>--- a/osmopy/trap_helper.py</span><br><span>+++ b/osmopy/trap_helper.py</span><br><span>@@ -25,8 +25,6 @@</span><br><span> import sys, os, datetime, signal, logging, logging.handlers</span><br><span> from functools import partial</span><br><span> from osmopy.osmo_ipa import Ctrl</span><br><span style="color: hsl(0, 100%, 40%);">-from osmopy.twisted_ipa import CTRL</span><br><span style="color: hsl(0, 100%, 40%);">-from twisted.internet import defer</span><br><span> </span><br><span> # keys from OpenBSC openbsc/src/libbsc/bsc_rf_ctrl.c, values SOAP-specific</span><br><span> oper = { 'inoperational' : 0, 'operational' : 1 }</span><br><span>diff --git a/scripts/ctrl2cgi.py b/scripts/ctrl2cgi.py</span><br><span>index 6818b2a..1d6813d 100755</span><br><span>--- a/scripts/ctrl2cgi.py</span><br><span>+++ b/scripts/ctrl2cgi.py</span><br><span>@@ -24,17 +24,17 @@</span><br><span> </span><br><span> __version__ = "0.0.4" # bump this on every non-trivial change</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-from twisted.internet import defer, reactor</span><br><span style="color: hsl(0, 100%, 40%);">-from osmopy.twisted_ipa import CTRL, IPAFactory, __version__ as twisted_ipa_version</span><br><span style="color: hsl(0, 100%, 40%);">-from osmopy.osmo_ipa import Ctrl</span><br><span style="color: hsl(0, 100%, 40%);">-from treq import post, collect</span><br><span style="color: hsl(0, 100%, 40%);">-from functools import partial</span><br><span style="color: hsl(0, 100%, 40%);">-from osmopy.trap_helper import reloader, debug_init, get_type, get_r, p_h, make_params, comm_proc</span><br><span style="color: hsl(0, 100%, 40%);">-from distutils.version import StrictVersion as V # FIXME: use NormalizedVersion from PEP-386 when available</span><br><span style="color: hsl(0, 100%, 40%);">-import argparse, datetime, signal, sys, os, logging, logging.handlers</span><br><span style="color: hsl(120, 100%, 40%);">+import argparse, os, logging, logging.handlers</span><br><span> import hashlib</span><br><span> import json</span><br><span> import configparser</span><br><span style="color: hsl(120, 100%, 40%);">+from functools import partial</span><br><span style="color: hsl(120, 100%, 40%);">+from distutils.version import StrictVersion as V # FIXME: use NormalizedVersion from PEP-386 when available</span><br><span style="color: hsl(120, 100%, 40%);">+from twisted.internet import defer, reactor</span><br><span style="color: hsl(120, 100%, 40%);">+from treq import post, collect</span><br><span style="color: hsl(120, 100%, 40%);">+from osmopy.trap_helper import debug_init, get_type, get_r, p_h, make_params, comm_proc</span><br><span style="color: hsl(120, 100%, 40%);">+from osmopy.twisted_ipa import CTRL, IPAFactory, __version__ as twisted_ipa_version</span><br><span style="color: hsl(120, 100%, 40%);">+from osmopy.osmo_ipa import Ctrl</span><br><span> </span><br><span> # we don't support older versions of TwistedIPA module</span><br><span> assert V(twisted_ipa_version) > V('0.4')</span><br><span>@@ -48,14 +48,14 @@</span><br><span>     comm_proc(decoded.get('commands'), f, log)</span><br><span> </span><br><span> def gen_hash(params, skey):</span><br><span style="color: hsl(0, 100%, 40%);">-    input = ''</span><br><span style="color: hsl(0, 100%, 40%);">-    for key in ['time_stamp','position_validity','admin_status','policy_status']:</span><br><span style="color: hsl(0, 100%, 40%);">-        input += str(params.get(key))</span><br><span style="color: hsl(0, 100%, 40%);">-    input += skey</span><br><span style="color: hsl(0, 100%, 40%);">-    for key in ['bsc_id','lat','lon','position_validity']:</span><br><span style="color: hsl(0, 100%, 40%);">-        input += str(params.get(key))</span><br><span style="color: hsl(120, 100%, 40%);">+    inp = ''</span><br><span style="color: hsl(120, 100%, 40%);">+    for key in ['time_stamp', 'position_validity', 'admin_status', 'policy_status']:</span><br><span style="color: hsl(120, 100%, 40%);">+        inp += str(params.get(key))</span><br><span style="color: hsl(120, 100%, 40%);">+    inp += skey</span><br><span style="color: hsl(120, 100%, 40%);">+    for key in ['bsc_id', 'lat', 'lon', 'position_validity']:</span><br><span style="color: hsl(120, 100%, 40%);">+        inp += str(params.get(key))</span><br><span>     m = hashlib.md5()</span><br><span style="color: hsl(0, 100%, 40%);">-    m.update(input.encode('utf-8'))</span><br><span style="color: hsl(120, 100%, 40%);">+    m.update(inp.encode('utf-8'))</span><br><span>     res = m.hexdigest()</span><br><span>     #print('HASH: \nparams="%r"\ninput="%s" \nres="%s"' %(params, input, res))</span><br><span>     return res</span><br><span>diff --git a/scripts/soap.py b/scripts/soap.py</span><br><span>index 6bf786c..267b4d8 100755</span><br><span>--- a/scripts/soap.py</span><br><span>+++ b/scripts/soap.py</span><br><span>@@ -24,15 +24,15 @@</span><br><span> </span><br><span> __version__ = "0.7.2" # bump this on every non-trivial change</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+import argparse, os, logging</span><br><span style="color: hsl(120, 100%, 40%);">+from functools import partial</span><br><span style="color: hsl(120, 100%, 40%);">+from distutils.version import StrictVersion as V # FIXME: use NormalizedVersion from PEP-386 when available</span><br><span> from twisted.internet import defer, reactor</span><br><span style="color: hsl(120, 100%, 40%);">+from suds.client import Client</span><br><span style="color: hsl(120, 100%, 40%);">+from treq import post, collect</span><br><span style="color: hsl(120, 100%, 40%);">+from osmopy.trap_helper import debug_init, get_type, get_r, p_h, make_params, comm_proc</span><br><span> from osmopy.twisted_ipa import CTRL, IPAFactory, __version__ as twisted_ipa_version</span><br><span> from osmopy.osmo_ipa import Ctrl</span><br><span style="color: hsl(0, 100%, 40%);">-from treq import post, collect</span><br><span style="color: hsl(0, 100%, 40%);">-from suds.client import Client</span><br><span style="color: hsl(0, 100%, 40%);">-from functools import partial</span><br><span style="color: hsl(0, 100%, 40%);">-from osmopy.trap_helper import reloader, debug_init, get_type, get_r, p_h, make_params, comm_proc</span><br><span style="color: hsl(0, 100%, 40%);">-from distutils.version import StrictVersion as V # FIXME: use NormalizedVersion from PEP-386 when available</span><br><span style="color: hsl(0, 100%, 40%);">-import argparse, datetime, signal, sys, os, logging, logging.handlers</span><br><span> </span><br><span> # we don't support older versions of TwistedIPA module</span><br><span> assert V(twisted_ipa_version) > V('0.4')</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/11975">change 11975</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/11975"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: python/osmo-python-tests </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I5b90ee790f73dc509081401776911f25e43f1801 </div>
<div style="display:none"> Gerrit-Change-Number: 11975 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Max <msuraev@sysmocom.de> </div>