laforge has submitted this change. (
https://gerrit.osmocom.org/c/python/pyosmocom/+/42555?usp=email )
Change subject: tlv: Remove unused branch
......................................................................
tlv: Remove unused branch
0xff is checked in the first if statement and already raises a ValueError
Change-Id: Ia6a0656721a1fcaf5f16526fefe62c30b0ddb664
---
M src/osmocom/tlv.py
1 file changed, 0 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
diff --git a/src/osmocom/tlv.py b/src/osmocom/tlv.py
index 14c22f0..ada7380 100644
--- a/src/osmocom/tlv.py
+++ b/src/osmocom/tlv.py
@@ -36,8 +36,6 @@
# three-byte tag
tag = binary[0] << 16 | binary[1] << 8 | binary[2]
return (tag, binary[3:])
- elif binary[0] == 0xff:
- return None, binary
else:
# single byte tag
tag = binary[0]
--
To view, visit https://gerrit.osmocom.org/c/python/pyosmocom/+/42555?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: python/pyosmocom
Gerrit-Branch: master
Gerrit-Change-Id: Ia6a0656721a1fcaf5f16526fefe62c30b0ddb664
Gerrit-Change-Number: 42555
Gerrit-PatchSet: 1
Gerrit-Owner: daniel <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
