<p style="white-space: pre-wrap; word-wrap: break-word;">So I was starting to add new ACC barring/ramping code and I already run into hell when moving some bits from another file into gsm_bts_alloc() which is inside gsm_data.c and then I had to add gsm_data.o in Makefile to some files which in turn wanted to pull basically the whole project in... so this time (unlike previously with osmo-bts) I did this movement upfront to already help with the dependency hell created by having all in gsm_data making it difficult to put stuff there since all the model is a big fat block.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm explaining this before anyone wants to start arguing whether this is really necessary or it helps... It Does. We do it for other objects, so more reason to do it for such an important one. I'm adding a lot of new includes: Yes, we already have plenty of them for other objects.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Let's please go ahead and merge this, get code in a more maintainable fashion making it easier to add stuff into the data model by properly isolating objects instead of adding more churn to the big fat block.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19287">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19287">change 19287</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/+/19287"/><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: I00c15f5285b5c1a0109279b7ab192d5467a04ece </div>
<div style="display:none"> Gerrit-Change-Number: 19287 </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: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 15 Jul 2020 19:08:23 +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>