summaryrefslogtreecommitdiffstats
path: root/include/osmocom/core
diff options
context:
space:
mode:
authorVadim Yanitskiy <axilirator@gmail.com>2019-03-25 22:18:10 +0700
committerHarald Welte <laforge@gnumonks.org>2019-03-27 08:56:21 +0100
commit115a2ccae5b02f10e377bd0f339aac875913f18d (patch)
tree16d1e51b5c26b5f4740d0e77bea63c2cf8a72047 /include/osmocom/core
parent73e66b3eef96bd1e5a48c6ad03fc03c1fe47c1af (diff)
core/msgb.h: fix dead msgb2 reference in msgb_eq_* helpers
Neither Doxygen documentation of the msgb data comparison helpers, nor their actual definitions does refer msgb2. Instead, 'msg2' is referenced in both cases. This was discovered while investigating the following Doxygen warnings: msgb.h:XXX: warning: argument 'msg2' of command @param is not found in the argument list of msgb_eq(msg1, msgb2, len) msgb.h:XXX: warning: The following parameters of msgb_eq_l2(msg1, msgb2, len) are not documented: parameter 'msgb2' parameter 'len' Due to this bug it was impossible to use the affected macros, because 'msg2' was not listed in their parameters. Having the unit test coverage would spot this bug at the beginning! Change-Id: I1079d629abdb8770eef6be7341e586a933cd9cca
Diffstat (limited to 'include/osmocom/core')
-rw-r--r--include/osmocom/core/msgb.h10
1 files changed, 5 insertions, 5 deletions
diff --git a/include/osmocom/core/msgb.h b/include/osmocom/core/msgb.h
index f006b345..5de391e1 100644
--- a/include/osmocom/core/msgb.h
+++ b/include/osmocom/core/msgb.h
@@ -668,35 +668,35 @@ bool _msgb_eq(const char *file, size_t line, const char *func, uint8_t level,
* \param[in] msg2 reference message buffer
* \returns boolean indicating whether msgb content is equal
*/
-#define msgb_eq(msg1, msgb2, len) msgb_eq_data(msg1, msgb_data(msg2), msgb_length(msg2))
+#define msgb_eq(msg1, msg2, len) msgb_eq_data(msg1, msgb_data(msg2), msgb_length(msg2))
/*! Compare msgbs L1 content
* \param[in] msg1 message buffer
* \param[in] msg2 reference message buffer
* \returns boolean indicating whether msgb L1 content is equal
*/
-#define msgb_eq_l1(msg1, msgb2, len) msgb_eq_l1_data(msg1, msgb_l1(msg2), msgb_l1len(msg2))
+#define msgb_eq_l1(msg1, msg2, len) msgb_eq_l1_data(msg1, msgb_l1(msg2), msgb_l1len(msg2))
/*! Compare msgbs L2 content
* \param[in] msg1 message buffer
* \param[in] msg2 reference message buffer
* \returns boolean indicating whether msgb L2 content is equal
*/
-#define msgb_eq_l2(msg1, msgb2, len) msgb_eq_l2_data(msg1, msgb_l2(msg2), msgb_l2len(msg2))
+#define msgb_eq_l2(msg1, msg2, len) msgb_eq_l2_data(msg1, msgb_l2(msg2), msgb_l2len(msg2))
/*! Compare msgbs L3 content
* \param[in] msg1 message buffer
* \param[in] msg2 reference message buffer
* \returns boolean indicating whether msgb L3 content is equal
*/
-#define msgb_eq_l3(msg1, msgb2, len) msgb_eq_l3_data(msg1, msgb_l3(msg2), msgb_l3len(msg2))
+#define msgb_eq_l3(msg1, msg2, len) msgb_eq_l3_data(msg1, msgb_l3(msg2), msgb_l3len(msg2))
/*! Compare msgbs L4 content
* \param[in] msg1 message buffer
* \param[in] msg2 reference message buffer
* \returns boolean indicating whether msgb L4 content is equal
*/
-#define msgb_eq_l4(msg1, msgb2, len) msgb_eq_l4_data(msg1, msgb_l4(msg2), msgb_l4len(msg2))
+#define msgb_eq_l4(msg1, msg2, len) msgb_eq_l4_data(msg1, msgb_l4(msg2), msgb_l4len(msg2))
/* non inline functions to ease binding */