<p style="white-space: pre-wrap; word-wrap: break-word;">I understand; what if we choose a different representation that would allow me to use the negative T space without confusion?</p><p style="white-space: pre-wrap; word-wrap: break-word;">some ideas:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> "Timeout of -T1"<br> "Timeout of T-OSMO-1"<br> "Timeout of X1"</pre><p style="white-space: pre-wrap; word-wrap: break-word;">In the code, there can hardly be confusion, since "if (T == -1)" is obviously not "if (T == 1)", so this is completely about the user experience; (log, vty)</p><p style="white-space: pre-wrap; word-wrap: break-word;">The point is that there are a lot of places where I am not sure which timer is the correct one, or where there simply is none defined. Instead of having to search all specs to make sure I don't accidentally use an already designated number, and instead of inventing stupid numbers like 993210 or 1000001, I find it very tempting to use the negative T space.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Related: I am writing all of my FSM instances with struct osmo_tdef_state_timeout arrays now, and use distinct T numbers for all states that need a timeout, so I need to pick a lot of T numbers. Just picking -1, -2, -3,... requires the least imagination and research at this time, and gives a warm feeling of never interfering with 3GPP numbers.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'd then change this logging and the osmo_tdef_vty to the alternate representation, like '# timer X3 default' vs. '# timer T3 default';  prohibit '# timer T-3'.</p><p><a href="https://gerrit.osmocom.org/12988">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12988">change 12988</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/12988"/><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: I3a59457623da9309fbbda235fe18fadd1636bff6 </div>
<div style="display:none"> Gerrit-Change-Number: 12988 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 25 Feb 2019 02:19:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>