Change in ...libosmocore[master]: add osmo_fsm_inst_watch()

[pespin]

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 <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin at sysmocom.de>
Gerrit-Comment-Date: Wed, 02 Oct 2019 22:21:21 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191002/04bdafde/attachment.htm>