laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved
Fix lint errors: remove unnecessary semicolons

Change-Id: I8c8586476d12461d555a2278ef936256944c9cde
---
M src/osmocom/construct.py
M tests/test_tlv.py
2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/osmocom/construct.py b/src/osmocom/construct.py
index 31e2a77..693f365 100644
--- a/src/osmocom/construct.py
+++ b/src/osmocom/construct.py
@@ -464,7 +464,7 @@
if self.steps == []:
return obj
else:
- return obj_step_aligned;
+ return obj_step_aligned

class StripHeaderAdapter(Adapter):
"""
@@ -507,7 +507,7 @@
if self.steps == []:
return obj
else:
- return obj_step_aligned;
+ return obj_step_aligned

def filter_dict(d, exclude_prefix='_'):
"""filter the input dict to ensure no keys starting with 'exclude_prefix' remain."""
diff --git a/tests/test_tlv.py b/tests/test_tlv.py
index 894ef63..3af1e07 100755
--- a/tests/test_tlv.py
+++ b/tests/test_tlv.py
@@ -45,7 +45,7 @@
self.assertEqual(bertlv_parse_len(b'\x83\x12\x34\x56\x78'), (0x123456, b'\x78'))

def test_BerTlvParseOne(self):
- res = bertlv_parse_one(b'\x81\x01\x01');
+ res = bertlv_parse_one(b'\x81\x01\x01')
self.assertEqual(res, ({'tag':1, 'constructed':False, 'class':2}, 1, b'\x01', b''))

class TestComprTlv(unittest.TestCase):
@@ -127,7 +127,7 @@
class XC_constr_class(Transcodable):
_construct = Int8ub
def __init__(self):
- super().__init__();
+ super().__init__()

def test_XC_constr_class(self):
"""Transcodable derived class with _construct class variable"""
@@ -137,7 +137,7 @@

class XC_constr_instance(Transcodable):
def __init__(self):
- super().__init__();
+ super().__init__()
self._construct = Int8ub

def test_XC_constr_instance(self):
@@ -148,7 +148,7 @@

class XC_method_instance(Transcodable):
def __init__(self):
- super().__init__();
+ super().__init__()
def _from_bytes(self, do):
return ('decoded', do)
def _to_bytes(self):

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

Gerrit-MessageType: merged
Gerrit-Project: python/pyosmocom
Gerrit-Branch: master
Gerrit-Change-Id: I8c8586476d12461d555a2278ef936256944c9cde
Gerrit-Change-Number: 41382
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>