<p>Vadim Yanitskiy has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/13405">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">core/msgb.h: drop meaningless parameter of msgb_eq_* helpers<br><br>Thanks to the following Doxygen warning:<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>it was discovered that parameter 'len' is not required at all.<br>It basically doesn't make any sense to pass any length value,<br>because it can be calculated using msgb_length().<br><br>Let's drop this parameter. Given that this part of the API was<br>broken so far (see I1079d629abdb8770eef6be7341e586a933cd9cca),<br>it should be more or less safe to do this.<br><br>Change-Id: Icd9b72eb6bfa9628ff1ed2f948b57058551a4328<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/05/13405/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 5de391e..273fc16 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, msg2, 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) 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, msg2, 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) 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, msg2, 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) 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, msg2, 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) 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, msg2, 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) 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/13405">change 13405</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/13405"/><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: Icd9b72eb6bfa9628ff1ed2f948b57058551a4328 </div>
<div style="display:none"> Gerrit-Change-Number: 13405 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>