<p><a href="https://gerrit.osmocom.org/13191">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13191/1/msc/BSC_ConnectionHandler.ttcn">File msc/BSC_ConnectionHandler.ttcn:</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/13191/1/msc/BSC_ConnectionHandler.ttcn@523">Patch Set #1, Line 523:</a> <code style="font-family:monospace,monospace">         </code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Cosmetic: why two tabs? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">oh indeed, cosmetic. Unfortunately gerrit coloring doesn't seem to catch that.</p><p style="white-space: pre-wrap; word-wrap: break-word;">two tabs because it makes more sense to me. Otherwise you can't optically distinguish the closing brace of the received message from the closing brace for the entire interlave (or alt) statement.</p><p style="white-space: pre-wrap; word-wrap: break-word;">AFAICT the indenting in these cases is inconsistent in our ttcn files, and I didn't want to</p><ul><li>add two  } } on the same indenting level or</li><li>have a closing } on the same level as its scope.</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">i.e. be closer to the indenting we use in C.</p><p style="white-space: pre-wrap; word-wrap: break-word;">[EDIT]</p><p style="white-space: pre-wrap; word-wrap: break-word;">it seems a lot of places use</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  alt {<br>  [] foo {<br>      content<br>      }<br>  }</pre><p style="white-space: pre-wrap; word-wrap: break-word;">I find it a bit ugly but if you insist I can change to that...</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13191">change 13191</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/13191"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia06af1e347601949f4ddb19f963daa400766d9e7 </div>
<div style="display:none"> Gerrit-Change-Number: 13191 </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-CC: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 10 Mar 2019 22:13:27 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>