Patch Set 2:

> What is the minimal dependency to get this in? Which patch removes the <= 32 
> assumption?

It's part of this patch. I can split it too if necessary.

> tests/alloc/AllocTest.cpp requires updates to use/test the new classes as 
> well.

I'll check and update in next revision.

> Any idea why we padded the structs before?

No idea. It's added in one commit and I do not see anything in the code relying 
on it.

-- 
To view, visit https://gerrit.osmocom.org/4072
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1ef2eb99c517f25e7d1e71b985a3e0eb3879eb2c
Gerrit-PatchSet: 2
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: Max <[email protected]>
Gerrit-Reviewer: Holger Freyther <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <[email protected]>
Gerrit-HasComments: No

Reply via email to