Attention is currently required from: osmith, fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc-nat/+/28249 )

Change subject: subscr_conn_fsm: refactor timeouts ......................................................................

Patch Set 3:

(1 comment)

File src/osmo-bsc-nat/bsc_nat.c:

https://gerrit.osmocom.org/c/osmo-bsc-nat/+/28249/comment/1f14ef6f_79bc4717 PS2, Line 31: struct osmo_tdef g_bsc_nat_tdefs[] = {

Done

@fixeria it cannot be static const since it is initialized. In the case where there are several instances of this tdef it indeed makes sense to have one const and then duplicate it in memory for each instance (we do that in several places), but in here there's no use in doing that since anyway there's only 1 instance, so it's fine resetting/changing values in it directly.