<p><a href="https://gerrit.osmocom.org/13398">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13398/1/include/osmocom/core/linuxlist.h">File include/osmocom/core/linuxlist.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/13398/1/include/osmocom/core/linuxlist.h@50">Patch Set #1, Line 50:</a> <code style="font-family:monospace,monospace">/*! Define a new llist_head pointing to another llist_head.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">another is confusing here, specially because immediatelly afterwards it' used to point to itself in LLIST_HEAD. So better use "a given" instead of "other".</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13398/1/include/osmocom/core/linuxlist.h@69">Patch Set #1, Line 69:</a> <code style="font-family:monospace,monospace"> * Insert a new entry between two known consecutive entries.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">No need for extra line. You can start it with "/*! Insert", see functions below.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13398/1/include/osmocom/core/linuxlist.h@85">Patch Set #1, Line 85:</a> <code style="font-family:monospace,monospace"> *  \param _new a new entry to be added.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"the entry to be added".</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13398/1/include/osmocom/core/linuxlist.h@86">Patch Set #1, Line 86:</a> <code style="font-family:monospace,monospace"> *  \param head head of linked list to add it after.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"add it after" is quite misleading here. Would be a lot clearer saying " linked list head to prepend the element to". I see nowhere the word "prepend" being used, and imho it's the one really explaining in an easy way what this function does.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13398/1/include/osmocom/core/linuxlist.h@344">Patch Set #1, Line 344:</a> <code style="font-family:monospace,monospace">            </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">whitespace, we can remove it in this commit or add an extra one.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13398">change 13398</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/13398"/><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: Iac67bdb9d5fbf7c222d04858967337f2428d6a94 </div>
<div style="display:none"> Gerrit-Change-Number: 13398 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </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-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 25 Mar 2019 14:50:46 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>