<p style="white-space: pre-wrap; word-wrap: break-word;">Letting Hoernchen and laforge review this one.<br>I'd be great perhaps having some unit test showing the difference in behavior after the fix.<br>Remember to fix stuff pointed out by the linter if applying.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/26657">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/26657/1//COMMIT_MSG">Commit Message:</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/libosmocore/+/26657/1//COMMIT_MSG@16">Patch Set #1, Line 16:</a> <code style="font-family:monospace,monospace">Fixes #4508</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">OS#4508</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/26657/1/src/conv.c">File src/conv.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/libosmocore/+/26657/1/src/conv.c@551">Patch Set #1, Line 551:</a> <code style="font-family:monospace,monospace">              for (s=0; s<decoder->n_states; s++)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">spaces between s=0 and others.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/26657/1/src/conv.c@552">Patch Set #1, Line 552:</a> <code style="font-family:monospace,monospace">             {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">{ in the same line of for.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/26657/1/src/conv.c@558">Patch Set #1, Line 558:</a> <code style="font-family:monospace,monospace">                 for (i=n-1; i>=0; i--)       {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">spaces between s=0 and others.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/26657/1/src/conv.c@579">Patch Set #1, Line 579:</a> <code style="font-family:monospace,monospace">     {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">{ in the same line of for.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/26657">change 26657</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/libosmocore/+/26657"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I87e51d3880c0fe7bf3d6cd08fd46517a424a230c </div>
<div style="display:none"> Gerrit-Change-Number: 26657 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: tnt <tnt@246tNt.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 22 Dec 2021 09:50:41 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>