<p style="white-space: pre-wrap; word-wrap: break-word;">I appreciate the review, but in this case...</p><p><a href="https://gerrit.osmocom.org/13576">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13576/3/include/osmocom/gsm/gsup.h">File include/osmocom/gsm/gsup.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/13576/3/include/osmocom/gsm/gsup.h@71">Patch Set #3, Line 71:</a> <code style="font-family:monospace,monospace">OSMO_GSUP_CLASS_IE</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Looking at this IE, the reader may ask: class of what? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">It is actually rather absurd, in the face of HUGE GIGANTIC patches waiting for review, we are seriously nitpicking about "Kind" vs "Class" vs "Message Class". This is Bikeshed at its best, if I may say so myself ;)</p><p style="white-space: pre-wrap; word-wrap: break-word;">"Message Class" is saying exactly the same thing but with more words; what part of GSUP is not a Message?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I already once edited all those patches from "KIND" to "CLASS", here, in osmo-hlr and osmo-msc, across entire branch histories; with git format-patch, sed and git am, taking great care to not accidentally modify other sentences containing the word "kind" in unrelated contexts. IMHO that's no sane way to spend my time.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13576/3/include/osmocom/gsm/gsup.h@241">Patch Set #3, Line 241:</a> <code style="font-family:monospace,monospace">OSMO_GSUP_CLASS_ARRAYSIZE</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Which array? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Please see the comment just above it. It explicitly serves as size for an array, hence it is called ARRAYSIZE.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The other may be an End Marker, which it actually isn't: it never is actually used as a marker anywhere. The only place it is used is gsup_test.c for a validity check.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Apologies, but I'd rather not adopt a bad name choice if you don't force me.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13576">change 13576</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/13576"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ic397a9f2c4a7224e47cab944c72e75ca5592efef </div>
<div style="display:none"> Gerrit-Change-Number: 13576 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 11 Apr 2019 23:24:10 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>