<p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/16933">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/osmo-msc/+/16933/1/src/libvlr/vlr.c">File src/libvlr/vlr.c:</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/osmo-msc/+/16933/1/src/libvlr/vlr.c@71">Patch Set #1, Line 71:</a> <code style="font-family:monospace,monospace">     { /* terminator */ }</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">"I'll be back"</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Absolutely!</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-msc/+/16933/1/src/libvlr/vlr.c@1268">Patch Set #1, Line 1268:</a> <code style="font-family:monospace,monospace">    osmo_tdefs_reset(vlr->cfg.tdefs);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">That means afaiu each time a vlc is allocated, your VTY-set values will be erased. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">That's how it's done in the other places of code. We never allocate more than one VLR anyway, so we can keep it as is for now. If somebody needs (for any not-obvious reason) more than one VLR, it can be changed to:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  vlr->cfg.tdefs = *vlr_instance_tdefs;<br>  osmo_tdefs_reset(&vlr->cfg.tdefs);</pre><p style="white-space: pre-wrap; word-wrap: break-word;">This is kind of a problem of the current osmo_tdef API - it mixes timer definition and current value in the same structure. I like the approach of osmo_fsm and osmo_fsm_inst.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/16933">change 16933</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/osmo-msc/+/16933"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ia8cf98da0aea0e626c5ff088a833d7359c43847f </div>
<div style="display:none"> Gerrit-Change-Number: 16933 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 20 Jan 2020 11:12:17 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>