<p style="white-space: pre-wrap; word-wrap: break-word;">If this is moved to a generic libosmocore sub-system, I'm starting to think how this would work in detail.  It seems that there's no concept of a "timer group" or something that is a sub-node under the vty "tdef" node.  The timer numbers in GSM are not globally unique, but they are locally scoped to each protocol or network element.  Even more, we deal with other protocols like SCCP.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So let's assume that both SCCP and BSSAP define a timer called "3", and both sub-sytems wanted to make use of this sub-system, how would this work?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm sorry if I missed something. Putting a -1 as a precaution.  I like the idea, let's just make sure if we move it to libosmocore we get it right the first time it is merged.</p><p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/12717">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/12717/3/src/vty/tdef_vty.c">File src/vty/tdef_vty.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/12717/3/src/vty/tdef_vty.c@119">Patch Set #3, Line 119:</a> <code style="font-family:monospace,monospace">   </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">whitespace</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12717/3/tests/tdef/tdef_vty_test.vty">File tests/tdef/tdef_vty_test.vty:</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/12717/3/tests/tdef/tdef_vty_test.vty@4">Patch Set #3, Line 4:</a> <code style="font-family:monospace,monospace">tdef</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I would appreciate a somewhat more human-readable name like "timer-definitons" instead of just "tdef".  I believe we could use "timer-definitions" as the primary name and add "tdef" as an alias for compatibility, as desired/required?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12717">change 12717</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/12717"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ibd6b1ed7f1bd6e1f2e0fde53352055a4468f23e5 </div>
<div style="display:none"> Gerrit-Change-Number: 12717 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 29 Jan 2019 10:41:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>