<p>Vadim Yanitskiy has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/13404">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">core/msgb.h: fix dead msgb2 reference in msgb_eq_* helpers<br><br>Neither Doxygen documentation of the msgb data comparison helpers,<br>nor their actual definitions does refer msgb2. Instead, 'msg2' is<br>referenced in both cases. This was discovered while investigating<br>the following Doxygen warnings:<br><br>  msgb.h:XXX: warning: argument 'msg2' of command @param is not<br>              found in the argument list of<br>              msgb_eq(msg1, msgb2, len)<br><br>  msgb.h:XXX: warning: The following parameters of<br>              msgb_eq_l2(msg1, msgb2, len) are not documented:<br>                parameter 'msgb2'<br>                parameter 'len'<br><br>Due to this bug it was impossible to use the affected macros,<br>because 'msg2' was not listed in their parameters. Having the<br>unit test coverage would spot this bug at the beginning!<br><br>Change-Id: I1079d629abdb8770eef6be7341e586a933cd9cca<br>---<br>M include/osmocom/core/msgb.h<br>1 file changed, 5 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/04/13404/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/core/msgb.h b/include/osmocom/core/msgb.h</span><br><span>index f006b34..5de391e 100644</span><br><span>--- a/include/osmocom/core/msgb.h</span><br><span>+++ b/include/osmocom/core/msgb.h</span><br><span>@@ -668,35 +668,35 @@</span><br><span>  *  \param[in] msg2 reference message buffer</span><br><span>  *  \returns boolean indicating whether msgb content is equal</span><br><span>  */</span><br><span style="color: hsl(0, 100%, 40%);">-#define msgb_eq(msg1, msgb2, len) msgb_eq_data(msg1, msgb_data(msg2), msgb_length(msg2))</span><br><span style="color: hsl(120, 100%, 40%);">+#define msgb_eq(msg1, msg2, len) msgb_eq_data(msg1, msgb_data(msg2), msgb_length(msg2))</span><br><span> </span><br><span> /*! Compare msgbs L1 content</span><br><span>  *  \param[in] msg1 message buffer</span><br><span>  *  \param[in] msg2 reference message buffer</span><br><span>  *  \returns boolean indicating whether msgb L1 content is equal</span><br><span>  */</span><br><span style="color: hsl(0, 100%, 40%);">-#define msgb_eq_l1(msg1, msgb2, len) msgb_eq_l1_data(msg1, msgb_l1(msg2), msgb_l1len(msg2))</span><br><span style="color: hsl(120, 100%, 40%);">+#define msgb_eq_l1(msg1, msg2, len) msgb_eq_l1_data(msg1, msgb_l1(msg2), msgb_l1len(msg2))</span><br><span> </span><br><span> /*! Compare msgbs L2 content</span><br><span>  *  \param[in] msg1 message buffer</span><br><span>  *  \param[in] msg2 reference message buffer</span><br><span>  *  \returns boolean indicating whether msgb L2 content is equal</span><br><span>  */</span><br><span style="color: hsl(0, 100%, 40%);">-#define msgb_eq_l2(msg1, msgb2, len) msgb_eq_l2_data(msg1, msgb_l2(msg2), msgb_l2len(msg2))</span><br><span style="color: hsl(120, 100%, 40%);">+#define msgb_eq_l2(msg1, msg2, len) msgb_eq_l2_data(msg1, msgb_l2(msg2), msgb_l2len(msg2))</span><br><span> </span><br><span> /*! Compare msgbs L3 content</span><br><span>  *  \param[in] msg1 message buffer</span><br><span>  *  \param[in] msg2 reference message buffer</span><br><span>  *  \returns boolean indicating whether msgb L3 content is equal</span><br><span>  */</span><br><span style="color: hsl(0, 100%, 40%);">-#define msgb_eq_l3(msg1, msgb2, len) msgb_eq_l3_data(msg1, msgb_l3(msg2), msgb_l3len(msg2))</span><br><span style="color: hsl(120, 100%, 40%);">+#define msgb_eq_l3(msg1, msg2, len) msgb_eq_l3_data(msg1, msgb_l3(msg2), msgb_l3len(msg2))</span><br><span> </span><br><span> /*! Compare msgbs L4 content</span><br><span>  *  \param[in] msg1 message buffer</span><br><span>  *  \param[in] msg2 reference message buffer</span><br><span>  *  \returns boolean indicating whether msgb L4 content is equal</span><br><span>  */</span><br><span style="color: hsl(0, 100%, 40%);">-#define msgb_eq_l4(msg1, msgb2, len) msgb_eq_l4_data(msg1, msgb_l4(msg2), msgb_l4len(msg2))</span><br><span style="color: hsl(120, 100%, 40%);">+#define msgb_eq_l4(msg1, msg2, len) msgb_eq_l4_data(msg1, msgb_l4(msg2), msgb_l4len(msg2))</span><br><span> </span><br><span> </span><br><span> /* non inline functions to ease binding */</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/13404">change 13404</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/13404"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I1079d629abdb8770eef6be7341e586a933cd9cca </div>
<div style="display:none"> Gerrit-Change-Number: 13404 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>