laforge submitted this change.

View Change

Approvals: dexter: Looks good to me, approved; Verified Jenkins Builder: Verified
osmocom.construct: Don't forget context when calling sizeof in Rpad

When calling the self.sizeof() method, we must pass in the context
in order to make expressions like 'this._.total_len' work.

Change-Id: I97e4b8fffcccba0b9d0ecb18c76a2a30fd990fd2
---
M src/osmocom/construct.py
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/osmocom/construct.py b/src/osmocom/construct.py
index 753b5de..8f01f3c 100644
--- a/src/osmocom/construct.py
+++ b/src/osmocom/construct.py
@@ -341,7 +341,7 @@
return obj.rstrip(self.pattern)

def _encode(self, obj, context, path):
- target_size = self.sizeof() * self.num_per_byte
+ target_size = self.sizeof(**context) * self.num_per_byte
if len(obj) > target_size:
raise SizeofError("Input ({}) exceeds target size ({})".format(
len(obj), target_size))

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

Gerrit-MessageType: merged
Gerrit-Project: python/pyosmocom
Gerrit-Branch: master
Gerrit-Change-Id: I97e4b8fffcccba0b9d0ecb18c76a2a30fd990fd2
Gerrit-Change-Number: 38186
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>