<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20630">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/osmo-bsc/+/20630/1/src/osmo-bsc/lb.c">File src/osmo-bsc/lb.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/osmo-bsc/+/20630/1/src/osmo-bsc/lb.c@378">Patch Set #1, Line 378:</a> <code style="font-family:monospace,monospace">              return -EAGAIN;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">EAGAIN? it's already started, this error code looks more like suggesting the opposite.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20630/1/src/osmo-bsc/lb.c@616">Patch Set #1, Line 616:</a> <code style="font-family:monospace,monospace">       /* Nothing to write? */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">As a side note, I see bsc_gsmnet->smlc being used tons of times in these functions, since basically slmc is the only object touched. So it'd be clearer storing it into an slmc local variable to shorten and make clearer the code.</p><p style="white-space: pre-wrap; word-wrap: break-word;">And btw, I find it a bit strange that this stuff is not placed into an lb_vty.c file as we usually do.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/20630">change 20630</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/c/osmo-bsc/+/20630"/><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-Change-Id: I01314a29a2cad6f325d9f4687a9dedca6b90a3ce </div>
<div style="display:none"> Gerrit-Change-Number: 20630 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 13 Oct 2020 09:59:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>