<p style="white-space: pre-wrap; word-wrap: break-word;">Thanks for the review, I'll update the .adoc file suggestions and re-submit.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19303">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19303/3/include/osmocom/bsc/acc.h">File include/osmocom/bsc/acc.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19303/3/include/osmocom/bsc/acc.h@38">Patch Set #3, Line 38:</a> <code style="font-family:monospace,monospace">len_allowed_adm</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">qty / quantity / count would be more obvious than 'len', IMHO. But only change if you agree. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">TBH I thought a lot about using "size" or "length" terms. "size" because we talk about creating sets/subsets, so one can speak about "set sizes". Otherwise as "length" since one can usually think or imagine of it as a list of ACC numbers (again, as a usual representation of a list or set).<br>So all in all, I have no big preference and I just picked one, but I think any of them terms can be used interchangeably.<br>For that reason I think keeping it as it is to avoid spending time on this, but if someone really thinks another term is better I can change all references.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In this struct I use "length" here when I use the param as a parameter used to generate the sets, while I use "count" where the variable is really holding a count, that is, counting whatever from an already generated set.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/19303">change 19303</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/+/19303"/><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: I952c9eeae02809c7184078c655574ec817902e06 </div>
<div style="display:none"> Gerrit-Change-Number: 19303 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 21 Jul 2020 16:27:46 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>