[S] Change in pysim[master]: pySim/tlv.py: Don't create an exception from within raise

laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/33661 ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: pySim/tlv.py: Don't create an exception from within raise ...................................................................... pySim/tlv.py: Don't create an exception from within raise An invalid variable used in a raise ValueError() would cause a further exception, depriving the user of a meaningful error message. Change-Id: I6eb31b91bd69c311f07ff259a424edc58b57529a --- M pySim/tlv.py 1 file changed, 13 insertions(+), 1 deletion(-) Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified diff --git a/pySim/tlv.py b/pySim/tlv.py index 22a14cd..3e245b5 100644 --- a/pySim/tlv.py +++ b/pySim/tlv.py @@ -395,7 +395,7 @@ res.append(inst) else: raise ValueError('%s: Unknown TLV Class %s in %s; expected %s' % - (self, i[0], decoded, self.members_by_name.keys())) + (self, k, decoded, self.members_by_name.keys())) self.children = res return res -- To view, visit https://gerrit.osmocom.org/c/pysim/+/33661 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I6eb31b91bd69c311f07ff259a424edc58b57529a Gerrit-Change-Number: 33661 Gerrit-PatchSet: 2 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu> Gerrit-MessageType: merged