<p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/25813">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-pcu/+/25813/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/osmo-pcu/+/25813/1//COMMIT_MSG@9">Patch Set #1, Line 9:</a> <code style="font-family:monospace,monospace">a lot easier to follow</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I would not say it's hard to follow the current variant of this function. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm happy you find it's not hard to follow the function, but it is to me. It's not the first time I have to spend time looking at that function for a while to make sure everything is fine and not faulty, due to final path being shared with some parts of the code jumping (goto) into it with some varaibles not set, etc. Even gcc had problems tracking variable dependencies there.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think it's "generally" understood a good practice to avoid goto jumping to one place or another while it's a usual good practice to use them for error handling + return paths.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/25813/1/src/bts.cpp">File src/bts.cpp:</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-pcu/+/25813/1/src/bts.cpp@870">Patch Set #1, Line 870:</a> <code style="font-family:monospace,monospace"> struct gprs_rlcmac_pdch *pdch;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Unrelated change.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">It's not unrelated: Due to untangling the final path, I expect gcc being able to follow variable initialization properly now.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/25813">change 25813</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-pcu/+/25813"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcu </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I80e554315d36a515a7edc9ae51057ce31eb9110d </div>
<div style="display:none"> Gerrit-Change-Number: 25813 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 18 Oct 2021 13:35:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>