* [dpdk-test-report] |WARNING| pw69328 [PATCH 1/2] eal/windows: replace sys/queue.h with a complete one from FreeBSD
[not found] <20200426032245.2437733-2-dmitry.kozliuk@gmail.com>
@ 2020-04-26 3:23 ` checkpatch
0 siblings, 0 replies; only message in thread
From: checkpatch @ 2020-04-26 3:23 UTC (permalink / raw)
To: test-report; +Cc: Dmitry Kozlyuk
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/69328
_coding style issues_
WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon
#237: FILE: lib/librte_eal/windows/include/sys/queue.h:103:
+#define TRACEBUF struct qm_trace trace;
ERROR:SPACING: space prohibited before that ',' (ctx:WxE)
#238: 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
#238: 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)
#263: 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
#263: 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)
#319: 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
#321: 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
#360: 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)
#382: 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
#390: 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
#395: 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)
#406: 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 '('
#455: 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
#480: 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
#486: 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)
#510: 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
#519: 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
#525: FILE: lib/librte_eal/windows/include/sys/queue.h:392:
+ if ((STAILQ_FIRST((head)) = \
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#531: 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)
#532: 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
#600: 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
#600: 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)
#619: 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
#621: 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
#659: 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
#665: 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
#682: 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
#701: 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)
#709: 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
#712: 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
#714: 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
#746: 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
#746: 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
#787: 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
#788: 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
#798: 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
#808: 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
#808: 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
#816: 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
#817: 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
#826: 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)
#836: 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)
#837: 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
#844: 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
#850: 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
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-04-26 3:23 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20200426032245.2437733-2-dmitry.kozliuk@gmail.com>
2020-04-26 3:23 ` [dpdk-test-report] |WARNING| pw69328 [PATCH 1/2] eal/windows: replace sys/queue.h with a complete one from FreeBSD checkpatch
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).