<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><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I'd rather have a bit longer names than really having to waste hours playing tetris with header files and source files. Having these patches solves the build problems in osmo-hnodeb and whoever wishes to use this library. BTW, none of our code outside osmo-iuh really uses those structures being used, so nothing breaks, and we don't really care if they are longer because we don't use a lot of them in any place.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I would like to see one real-world example where we actually run into such clashes.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Take osmo-iuh.git master, take osmo-hnodeb.git with all the current patches.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;"><br>IMHO, code that implements HNBAP doesn't have anything do with RANAP nor RUA.  And code implementing RUA should have a clean layer separation to RANAP, too.  Those are different layers, stacked on top of each other.  like BSSMAP in SCCP. RUA contains an octetstring, and that octetstring contains RANAP.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">So being different layers means we mandate the user to use different C files and make him play tetris to make sure no headers are imported incorrectly? Why if someone wants to write a quick test/tool in one file? I see no point in mandating the use of different C files, and trying to do so just to avoid having a bit longer names goes the opposite way to what we do with other libraries, where we explicitly want to have an osmo_ prefix in order to avoid clashes with other libs.</p><p><a href="https://gerrit.osmocom.org/c/osmo-iuh/+/25991">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-iuh/+/25991">change 25991</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-iuh/+/25991"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-iuh </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I1bcbe047d233684647d6ffe9febb14a12c579696 </div>
<div style="display:none"> Gerrit-Change-Number: 25991 </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: 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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 29 Oct 2021 12:48:30 +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>