Attention is currently required from: neels.
fixeria has posted comments on this change by neels. ( https://gerrit.osmocom.org/c/osmo-bts/+/39376?usp=email )
Change subject: coverity: ensure lchan->si->* bitmask sizes ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File include/osmo-bts/lchan.h:
https://gerrit.osmocom.org/c/osmo-bts/+/39376/comment/dfa049d9_46c0656e?usp=... : PS1, Line 388: osmo_static_assert Not sure if it's a good idea to put static asserts to a header file, because this way we end up checking this assert in every `*.c` file that includes this header. Not a big deal given that it's checked at compile time, but still does not feel right. Maybe put it to `src/common/lchan.c` instead?