<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/15660">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15660/2//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15660/2//COMMIT_MSG@20">Patch Set #2, Line 20:</a> <code style="font-family:monospace,monospace">immediate error handling directly terminated and deallocated bar</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Looking at the problem from the other side: can we delay deallocation of the FSM instances and move it to osmo_select_main()? This way calling osmo_fsm_inst_term() would not deallocate a given FSM immediately, but just mark it as 'to be deallocated' instead, and you will be sure that bar still exists (we may want to discard all events in such case).</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15660/2//COMMIT_MSG@32">Patch Set #2, Line 32:</a> <code style="font-family:monospace,monospace">osmo_fsm_inst_watch</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">More I learn Erlang, more I realize that we're basically reimplementing basic primitives that it already has. At least this could be done using 'erlang:monitor/2'.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/15660">change 15660</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmocore/+/15660"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4d8306488506c60b4c2fc1c4cb3ac04654db9c43 </div>
<div style="display:none"> Gerrit-Change-Number: 15660 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 03 Oct 2019 07:33:44 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>