<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Requiring a ta argument can be a good way to make sure callers never forget to set the ta.<br>For sched_ul_ass_or_rej(), can there ever be a situation where ta was forgotten to set in the ms? IIUC this isn't a functional change. API design wise I tend to prefer the "safer" API that ensures ta is never omitted...?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure why you spent time to write this patch, but to not drag out a bikeshed: if you insist that this must happen, and if you are certain that ta is never forgotten, consider this a +1 from me. If you're unsure then why not just leave this API as it is...</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The data structure architecture in osmo-pcu is totally fucked up, with classes totally entangled one with another, and these patches clearly show what I'm saying. I'm trying to split stuff consistently so we can finally make some sense out of this code.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It makes no sense to pass a TA value (even invalid one) to a TBF function which then sets that MS attribute to the MS object.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If the TA is forgotten to be set at any time, it contains the same INVALID default value, so there's no change.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So just to be really clear here: I don't think I'm losing my time, and I think all these patches are needed to be able to finally have some code which we can extend and understand properly. Be sure that a lot more of these patches will come over time.</p><p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/18224">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/18224">change 18224</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/+/18224"/><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: Iebb9c57f458690e045ddc45c800209ad8cf621e0 </div>
<div style="display:none"> Gerrit-Change-Number: 18224 </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 13 May 2020 13:21:36 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>