|WARNING| pw163546 [PATCH v2 1/2] test: use inline helpers in buffer comparison macros

[checkpatch at dpdk.org]

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/163546

_coding style issues_


WARNING:BAD_SIGN_OFF: Duplicate signature
#126: 
Acked-by: Marat Khalili <marat.khalili at huawei.com>

WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'?
#142: FILE: app/test/test.h:12:
+#include <stdio.h>
           ^^^^^

WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#226: FILE: app/test/test.h:101:
+#define TEST_ASSERT_BUFFERS_ARE_EQUAL(a, b, len, msg, ...) do {		\
+	if (memcmp((a), (b), (len)) != 0) {				\
+		printf("TestCase %s() line %d failed: " msg "
",	\
+			__func__, __LINE__, ##__VA_ARGS__);		\
+		TEST_TRACE_FAILURE(__FILE__, __LINE__, __func__);	\
+		return TEST_FAILED;					\
+	}								\
 } while (0)

WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#301: FILE: app/test/test.h:131:
+#define TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT_OFFSET(a, b, len, off, msg, ...) \
+do {									\
+	if (!test_buffers_equal_bit_offset((a), (b), (len), (off))) {	\
+		printf("TestCase %s() line %d failed: " msg "
",	\
+			__func__, __LINE__, ##__VA_ARGS__);		\
+		TEST_TRACE_FAILURE(__FILE__, __LINE__, __func__);	\
+		return TEST_FAILED;					\
+	}								\
 } while (0)

total: 0 errors, 4 warnings, 175 lines checked