<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/9263">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/9263/1/src/input/ipaccess.c">File src/input/ipaccess.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/9263/1/src/input/ipaccess.c@183">Patch Set #1, Line 183:</a> <code style="font-family:monospace,monospace">                               goto err;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't really understand the semantics of the old code, and why we have it like that.  Hoewever, the original behavior appears to have been to simply print an error message but continue normal processing (return 0) while you're propagating this up to the caller.  This change also doesn't really seem to relate to the commit message?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9263/1/src/input/ipaccess.c@392">Patch Set #1, Line 392:</a> <code style="font-family:monospace,monospace">    if (rc!=-EBADF && (what & BSC_FD_WRITE))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">coding style (spaces).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9263">change 9263</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/9263"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-abis </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I52faa9e6717137a7dab9c4e006eaa50b7367fc3e </div>
<div style="display:none"> Gerrit-Change-Number: 9263 </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-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 23 May 2018 15:44:41 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>