Attention is currently required from: neels, pespin.

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

Change subject: gsm0502: add burst length definitions from chapter 5.2 ......................................................................

Patch Set 1:

(1 comment)

File include/osmocom/gsm/gsm0502.h:

https://gerrit.osmocom.org/c/libosmocore/+/31669/comment/a279ecb4_60230daa PS1, Line 36: #define GSM_NBITS_NB_GMSK_PAYLOAD 2 * 58

ok, still if you use this after a division you'll have problems: […]

Thanks for explanation. Now I see what could go wrong, even though it's unlikely that anyone will ever need to use these constants as a divisor. I wanted to understand potential pitfalls of not having braces rather than applying code review suggestions blindly. Will add the braces.