[dpdk-test-report] |WARNING| pw70948 [PATCH v7 08/11] eal/windows: replace sys/queue.h with a complete one from FreeBSD

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Jun 10 09:56:53 CEST 2020


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

_coding style issues_


WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#236: FILE: lib/librte_eal/windows/include/sys/queue.h:103:
+#define	TRACEBUF	struct qm_trace trace;

ERROR:SPACING: space prohibited before that ',' (ctx:WxE)
#237: FILE: lib/librte_eal/windows/include/sys/queue.h:104:
+#define	TRACEBUF_INITIALIZER	{ __LINE__, 0, __FILE__, NULL } ,
        	                    	                                ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#237: FILE: lib/librte_eal/windows/include/sys/queue.h:104:
+#define	TRACEBUF_INITIALIZER	{ __LINE__, 0, __FILE__, NULL } ,

ERROR:SPACING: space required after that ';' (ctx:VxV)
#262: FILE: lib/librte_eal/windows/include/sys/queue.h:129:
+#define	TRASHIT(x)		do {(x) = (void *)-1;} while (0)
        	          		                    ^

WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop
#262: FILE: lib/librte_eal/windows/include/sys/queue.h:129:
+#define	TRASHIT(x)		do {(x) = (void *)-1;} while (0)

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#318: FILE: lib/librte_eal/windows/include/sys/queue.h:186:
+	QUEUE_TYPEOF(type) *curelm = SLIST_FIRST(head1);		\
 	                   ^

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#320: FILE: lib/librte_eal/windows/include/sys/queue.h:188:
+		if ((SLIST_FIRST(head1) = SLIST_FIRST(head2)) != NULL)	\

WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop
#359: FILE: lib/librte_eal/windows/include/sys/queue.h:227:
+#define	SLIST_INIT(head) do {						\
+	SLIST_FIRST((head)) = NULL;					\
+} while (0)

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#381: FILE: lib/librte_eal/windows/include/sys/queue.h:249:
+		QUEUE_TYPEOF(type) *curelm = SLIST_FIRST(head);		\
 		                   ^

WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop
#389: FILE: lib/librte_eal/windows/include/sys/queue.h:257:
+#define SLIST_REMOVE_AFTER(elm, field) do {				\
+	SLIST_NEXT(elm, field) =					\
+	    SLIST_NEXT(SLIST_NEXT(elm, field), field);			\
+} while (0)

WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop
#394: FILE: lib/librte_eal/windows/include/sys/queue.h:262:
+#define	SLIST_REMOVE_HEAD(head, field) do {				\
+	SLIST_FIRST((head)) = SLIST_NEXT(SLIST_FIRST((head)), field);	\
+} while (0)

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#405: FILE: lib/librte_eal/windows/include/sys/queue.h:273:
+	QUEUE_TYPEOF(type) *swap_first = SLIST_FIRST(head1);		\
 	                   ^

ERROR:SPACING: space required before the open parenthesis '('
#454: FILE: lib/librte_eal/windows/include/sys/queue.h:322:
+	for((var) = STAILQ_FIRST((head));				\

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#479: FILE: lib/librte_eal/windows/include/sys/queue.h:347:
+	if ((STAILQ_NEXT((elm), field) = STAILQ_NEXT((tqelm), field)) == NULL)\

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#485: FILE: lib/librte_eal/windows/include/sys/queue.h:353:
+	if ((STAILQ_NEXT((elm), field) = STAILQ_FIRST((head))) == NULL)	\

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#509: FILE: lib/librte_eal/windows/include/sys/queue.h:377:
+		QUEUE_TYPEOF(type) *curelm = STAILQ_FIRST(head);	\
 		                   ^

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#518: FILE: lib/librte_eal/windows/include/sys/queue.h:386:
+	if ((STAILQ_NEXT(elm, field) =					\

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#524: FILE: lib/librte_eal/windows/include/sys/queue.h:392:
+	if ((STAILQ_FIRST((head)) =					\

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#530: FILE: lib/librte_eal/windows/include/sys/queue.h:398:
+	QUEUE_TYPEOF(type) *swap_first = STAILQ_FIRST(head1);		\
 	                   ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#531: FILE: lib/librte_eal/windows/include/sys/queue.h:399:
+	QUEUE_TYPEOF(type) **swap_last = (head1)->stqh_last;		\
 	                   ^

ERROR:CODE_INDENT: code indent should use tabs where possible
#599: FILE: lib/librte_eal/windows/include/sys/queue.h:467:
+^I     ^Ipanic("Bad link elm %p next->prev != elm", (elm));^I\$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#599: FILE: lib/librte_eal/windows/include/sys/queue.h:467:
+^I     ^Ipanic("Bad link elm %p next->prev != elm", (elm));^I\$

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#618: FILE: lib/librte_eal/windows/include/sys/queue.h:486:
+	QUEUE_TYPEOF(type) *curelm = LIST_FIRST(head1);			      \
 	                   ^

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#620: FILE: lib/librte_eal/windows/include/sys/queue.h:488:
+		if ((LIST_FIRST(head1) = LIST_FIRST(head2)) != NULL) {	      \

WARNING:SINGLE_STATEMENT_DO_WHILE_MACRO: Single statement macros should not use a do {} while (0) loop
#658: FILE: lib/librte_eal/windows/include/sys/queue.h:526:
+#define	LIST_INIT(head) do {						\
+	LIST_FIRST((head)) = NULL;					\
+} while (0)

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#664: FILE: lib/librte_eal/windows/include/sys/queue.h:532:
+	if ((LIST_NEXT((elm), field) = LIST_NEXT((listelm), field)) != NULL)\

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#681: FILE: lib/librte_eal/windows/include/sys/queue.h:549:
+	if ((LIST_NEXT((elm), field) = LIST_FIRST((head))) != NULL)	\

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#700: FILE: lib/librte_eal/windows/include/sys/queue.h:568:
+^I^ILIST_NEXT((elm), field)->field.le_prev = ^I^I\$

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#708: FILE: lib/librte_eal/windows/include/sys/queue.h:576:
+	QUEUE_TYPEOF(type) *swap_tmp = LIST_FIRST(head1);		\
 	                   ^

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#711: FILE: lib/librte_eal/windows/include/sys/queue.h:579:
+	if ((swap_tmp = LIST_FIRST((head1))) != NULL)			\

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#713: FILE: lib/librte_eal/windows/include/sys/queue.h:581:
+	if ((swap_tmp = LIST_FIRST((head2))) != NULL)			\

ERROR:CODE_INDENT: code indent should use tabs where possible
#745: FILE: lib/librte_eal/windows/include/sys/queue.h:643:
+^I    ^Ipanic("Bad tailq NEXT(%p->tqh_last) != NULL", (head)); ^I\$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#745: FILE: lib/librte_eal/windows/include/sys/queue.h:643:
+^I    ^Ipanic("Bad tailq NEXT(%p->tqh_last) != NULL", (head)); ^I\$

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#786: FILE: lib/librte_eal/windows/include/sys/queue.h:738:
+	if ((TAILQ_NEXT((elm), field) = TAILQ_NEXT((listelm), field)) != NULL)\

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#787: FILE: lib/librte_eal/windows/include/sys/queue.h:739:
+^I^ITAILQ_NEXT((elm), field)->field.tqe_prev = ^I^I\$

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#797: FILE: lib/librte_eal/windows/include/sys/queue.h:763:
+	if ((TAILQ_NEXT((elm), field) = TAILQ_FIRST((head))) != NULL)	\

WARNING:LONG_LINE: line over 90 characters
#807: FILE: lib/librte_eal/windows/include/sys/queue.h:795:
+    (TAILQ_EMPTY(head) ? NULL : __containerof((head)->tqh_last, QUEUE_TYPEOF(type), field.tqe_next))

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#807: FILE: lib/librte_eal/windows/include/sys/queue.h:795:
+    (TAILQ_EMPTY(head) ? NULL : __containerof((head)->tqh_last, QUEUE_TYPEOF(type), field.tqe_next))$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#815: FILE: lib/librte_eal/windows/include/sys/queue.h:803:
+    ((elm)->field.tqe_prev == &(head)->tqh_first ? NULL :^I^I\$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#816: FILE: lib/librte_eal/windows/include/sys/queue.h:804:
+     __containerof((elm)->field.tqe_prev, QUEUE_TYPEOF(type), field.tqe_next))$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#825: FILE: lib/librte_eal/windows/include/sys/queue.h:812:
+^I^ITAILQ_NEXT((elm), field)->field.tqe_prev = ^I^I\$

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#835: FILE: lib/librte_eal/windows/include/sys/queue.h:825:
+	QUEUE_TYPEOF(type) *swap_first = (head1)->tqh_first;		\
 	                   ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#836: FILE: lib/librte_eal/windows/include/sys/queue.h:826:
+	QUEUE_TYPEOF(type) **swap_last = (head1)->tqh_last;		\
 	                   ^

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#843: FILE: lib/librte_eal/windows/include/sys/queue.h:831:
+	if ((swap_first = (head1)->tqh_first) != NULL)			\

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#849: FILE: lib/librte_eal/windows/include/sys/queue.h:835:
+	if ((swap_first = (head2)->tqh_first) != NULL)			\

total: 30 errors, 15 warnings, 753 lines checked


More information about the test-report mailing list