<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: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">Ack! Let's avoid code duplication please, we already have a lot of copy-paste in osmo-bts-{sysmo,oct2g,lc12}...</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">See my comment in osmo-bsc patches. They may look rather similar right now, but they have slight differences in behavior. Also that's because we have really poor implementations right now. These patches aim at adding initial infrastructure to improve and properly handle each object properly in the future (like moving them to Disabled Dependency when we Adminsitratively lock them, to cascade status on the object hierarchy, etc.). So in the end they will end up differing more and more.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So I think trying to put stuff together for different objects is counterproductive mesure in the long run, because we'll end up going the opposite path afterwards. Let's rather keep differentiated FSMs in the common/ dir and try to squeeze as much as possible repeated code in per-device code (and that can be done further in patches after these ones once further clean up is done).</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20407">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20407">change 20407</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-bts/+/20407"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Id9e61f8d773e6e6170c68b5b836d276c747d8d69 </div>
<div style="display:none"> Gerrit-Change-Number: 20407 </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-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 05 Oct 2020 09:11:08 +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>