pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmocore/+/15660 )
Change subject: add osmo_fsm_inst_watch()
......................................................................
Patch Set 2:
TBH I don't like this API, I think it's too specific.
I'd rather add a free_cb, which is more common for readers and more generic
(can be reused for other purposes):
void mycb(fi, pdata)
{
bool *freed = (bool*) pdata;
freed = true;
}
void event_action()
{
bool freed = false;
osmo_fsm_free_cb_set(fi, mycb, freed);
osmo_fsm_inst(....)
if (!freed)
return;
osmo_fsm_inst(....)
}
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/15660
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I4d8306488506c60b4c2fc1c4cb3ac04654db9c43
Gerrit-Change-Number: 15660
Gerrit-PatchSet: 2
Gerrit-Owner: neels <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <[email protected]>
Gerrit-Comment-Date: Wed, 02 Oct 2019 22:21:21 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment