Attention is currently required from: fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/30216 )
Change subject: tbf_fsm: Move osmo_fsm_inst fi out of struct tbf_fsm_ctx ......................................................................
Patch Set 2:
(1 comment)
File src/tbf.h:
https://gerrit.osmocom.org/c/osmo-pcu/+/30216/comment/fefd7001_00819862 PS1, Line 260: state_fi
Maybe move those internals to the private/protected part and only expose the *fi pointers?
The priv/protected stuff will disappear eventually when we move code to C. In any case, the mainly used field is state_fsm.state_flags which I really need to know into cleaning because it's from old code and I may end up rewriting that.
Anyways, not gonna block your patchset.
Yes, it's really bikeshed, whether we want to call it "main", "general" or "state" or whatever. There's more interesting stuff to clean up in there.
Also all this needs to be done iteratively with lots of steps, and I tend to spread burst of changes over time. It's quite usual that in osmo-pcu I started doing 1 change and I end up having 10 patches cleaning up /refactoring stuff before being able to actually touch the thing I want to touch in first place.