<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">The idea of a commit log is that the reviewer can skip all that<br>because you have explained it :)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I think the commit description provides enough context to understand the issue and how it was added (originating commit), how to reproduce the issue (TTCN3 test, and how it behaves in commit description), and why exactly the added code behaves (there's a comment explaining the case/scenario on top of the code).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Feel free to provide an explicit change in order to merge the change, or we can keep with osmo-bts misbehaving and a TTCN3 failing for a few more months for apparently no good reason. It's a 2 lines change with a TTCN3 test behind it showcasing the issue, not a complete refactor of 1000 code lines.</p><p><a href="https://gerrit.osmocom.org/12180">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12180">change 12180</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/12180"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I6d6d12aec10c801fe55012ca6e58d0bc8755b15d </div>
<div style="display:none"> Gerrit-Change-Number: 12180 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: 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-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 20 Dec 2018 00:58:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>