Attention is currently required from: iedemam, neels, pespin, fixeria, dexter. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/28276 )
Change subject: Expand VTY option which controls use of TCH for signalling ......................................................................
Patch Set 4:
(2 comments)
File include/osmocom/bsc/bts.h:
https://gerrit.osmocom.org/c/osmo-bsc/+/28276/comment/9dd0757c_0859f3fe PS4, Line 524: chan_alloc_allow_tch_for_signalling cosmetic: I'd rename it to "tch_for_signalling_policy" or something like that which makes it more clear that it's no longer some boolean value. Changing from bool to an enum but keeping the name identical might also hide bugs, as existing code using "if (chan_alloc_allow_tch_for_signalling)" will still compile but probably no longer do what we expect it to do.
File src/osmo-bsc/bts_vty.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/28276/comment/a036c560_c466fe52 PS4, Line 568: DEFUN_ATTR it might be nice to add a backwards-compatibility alias for the (0|1) so old config files still parse.
-
laforge