Attention is currently required from: fixeria.

laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31505 )

Change subject: gsm/{bsslap,bssmap_le}: zero-initialize structs using memset() ......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS1:

I just find memset(0) looks ugly, and regression tests are allowed to look ugly, API implementations not so much.

That's a matter of taste. I never saw anything wrong with it, and we've had it in our codebase as a standard way of initialization for probably a decade or so. In fact, I still intuitively write code that way. It's explicit, everyone knows without doubt what it does.