<p><a href="https://gerrit.osmocom.org/9671">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/9671/28/src/osmo-bsc/handover_fsm.c">File src/osmo-bsc/handover_fsm.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/9671/28/src/osmo-bsc/handover_fsm.c@352">Patch Set #28, Line 352:</a> <code style="font-family:monospace,monospace">  {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">@laforge, your opinion is against these inner scopes, but I don't really see a need to remove them. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">it's just something I've never seen in the kernel, and we use kernel coding style in Osmocom.  So my argument is "consistency with existing code base and documented coding style". Yes, "if" there was an if, I wouldn't flag it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">we also add another indent level without any need for it, reducing the usable part of the line.</p><p style="white-space: pre-wrap; word-wrap: break-word;">A quick look at the function without reading it from top to bottom makes me think this is some kind of condition/branch/loop due to that extra indent, only to then discover no, this is normal, un-conditional "main body" of the function.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9671">change 9671</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/9671"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I82e3f918295daa83274a4cf803f046979f284366 </div>
<div style="display:none"> Gerrit-Change-Number: 9671 </div>
<div style="display:none"> Gerrit-PatchSet: 28 </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 </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 24 Jul 2018 05:34:30 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>