<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 3:</p><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 3:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think those fields are mostly aimed at internal use by stats reporter, and I don't think anyone should be using those directly out of libosmocore.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">well, "think" and "should" is the problem: it is public API and this patch breaks it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This is a case of deciding to break an axiomatic principle because we're pretty sure to get away with it.<br>It really seems ok to do it, but it's objectively wrong from the underlying libosmo* release principles.<br>I'd like to have Harald's opinion on that.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;"><br>I fully agree with neels' analysis - but indeed for pragmatic reasons I think we can get away with it.  We are not aware of any other substantial libosmocore users out there, and even if there are, it is hightly unlikely that they are using stuff that even we did never really use the way it was intended (the fifo history for stat_items).</p><p style="white-space: pre-wrap; word-wrap: break-word;">So yes, it's against our general API comptibility rules, but without even knowing a single example of code that would break it's hard to justify the extra complexity that goes along with maintaining both and old and a new API.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So I don't have a general problem with the approach the patch takes, obviously just as long as old osmo-* still builds against libosmocore master.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25464">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/25464">change 25464</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/libosmocore/+/25464"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I137992a5479fc39bbceb6c6c2af9c227bd33b39b </div>
<div style="display:none"> Gerrit-Change-Number: 25464 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 23 Sep 2021 13:02:55 +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>