Attention is currently required from: fixeria.

pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/37212?usp=email )

Change subject: mobile: cosmetic: fix -Wswitch in tch_voice_state_init() ......................................................................

Patch Set 1:

(1 comment)

File src/host/layer23/src/mobile/tch_voice.c:

https://gerrit.osmocom.org/c/osmocom-bb/+/37212/comment/60bf1749_980b7758 PS1, Line 135: #endif

It's fine because we cannot end up having `state->handler == TCH_VOICE_IOH_GAPK` if built without `W […]

as you see, but I'm sure the same question I had will probably be triggered by other readers when they look at this code ("Was this correct or should it be before the break?"). It's much much clear here adding the #else with the comment or a explicit "rr->audio_mode = 0x00;" imho.