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

Reply via email to