laforge has uploaded this change for review.

View Change

pySim.tlv: Fix ComprTlvMeta() not passing kwargs to parent __new__

This fixes commit cdf661b24cfebb63b57405c2b55b3c42a8f277c8
"pySim.tlv.COMPR_TLV_IE: Patch comprehension bit if derived class misses it"
where we introduce a comprehension-TLV specific derived metaclass, which forgets
to pass the kwargs through to the parent metaclass.

Change-Id: If65a8169bcf91bb2f943d0316f1140e07f0b8b8e
---
M pySim/tlv.py
1 file changed, 15 insertions(+), 1 deletion(-)

git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/36/37436/1
diff --git a/pySim/tlv.py b/pySim/tlv.py
index 3d7a420..d7c6876 100644
--- a/pySim/tlv.py
+++ b/pySim/tlv.py
@@ -270,7 +270,7 @@

class ComprTlvMeta(TlvMeta):
def __new__(mcs, name, bases, namespace, **kwargs):
- x = super().__new__(mcs, name, bases, namespace)
+ x = super().__new__(mcs, name, bases, namespace, **kwargs)
if x.tag:
# we currently assume that the tag values always have the comprehension bit set;
# let's fix it up if a derived class has forgotten about that

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

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: If65a8169bcf91bb2f943d0316f1140e07f0b8b8e
Gerrit-Change-Number: 37436
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-MessageType: newchange