osmo-pcu[master]: Add multislot classes from latest spec

Max gerrit-no-reply at lists.osmocom.org
Tue Oct 10 18:02:55 UTC 2017


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 <msuraev at sysmocom.de>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-HasComments: No


More information about the gerrit-log mailing list