<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 7:</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 7:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Just a general question (not a refactoring request): do we really need to use linux_list for storing the OML addresses in the memory? I don't think on practice anybody would need more than 2 or let's say 8 (in the most paranoid case) entries. So a simple array of pointers might be a better fit here.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I don't see what's wrong with using linuxlist here. IMHO it fits better than a static array. Let the user put as many addresses as it likes, and we don't care about quick access here.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The problem with a static array is always that you never know how much is enough. Yes one could put a pointer array with a size of 10 into the BTS struct, but then you are limited to 10 (arbitrary limit) and when the user edits the list from the VTY one would need logic to allocate/free and possibly re-order the items in that array. I am really happy with the linuxlist implementation we have, it works well for many things.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/24513">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/+/24513">change 24513</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/+/24513"/><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: I205f68a3a7f35fee4c38a7cfba2b014237df2727 </div>
<div style="display:none"> Gerrit-Change-Number: 24513 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@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-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 01 Jul 2021 09:05:29 +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>