From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1F8BFA0562 for ; Mon, 30 Mar 2020 06:12:13 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 4EE1E1C0C7; Mon, 30 Mar 2020 06:12:11 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id A31561C07C; Mon, 30 Mar 2020 06:12:08 +0200 (CEST) In-Reply-To: <20200330041026.784624-9-dmitry.kozliuk@gmail.com> References: <20200330041026.784624-9-dmitry.kozliuk@gmail.com> To: test-report@dpdk.org Cc: Dmitry Kozlyuk Message-Id: <20200330041208.A31561C07C@dpdk.org> Date: Mon, 30 Mar 2020 06:12:08 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw67391 [RFC PATCH 8/9] eal/windows: replace sys/queue.h with a complete one from FreeBSD X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/67391 _coding style issues_ WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon #232: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:103: +#define TRACEBUF struct qm_trace trace; ERROR:SPACING: space prohibited before that ',' (ctx:WxE) #233: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:104: +#define TRACEBUF_INITIALIZER { __LINE__, 0, __FILE__, NULL } , ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #233: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:104: +#define TRACEBUF_INITIALIZER { __LINE__, 0, __FILE__, NULL } , ERROR:SPACING: space required after that ';' (ctx:VxV) #258: FILE: lib/librte_eal/windows/eal/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 #258: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:129: +#define TRASHIT(x) do {(x) = (void *)-1;} while (0) ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #314: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:186: + QUEUE_TYPEOF(type) *curelm = SLIST_FIRST(head1); \ ^ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #316: FILE: lib/librte_eal/windows/eal/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 #355: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:227: +#define SLIST_INIT(head) do { \ + SLIST_FIRST((head)) = NULL; \ +} while (0) ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #377: FILE: lib/librte_eal/windows/eal/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 #385: FILE: lib/librte_eal/windows/eal/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 #390: FILE: lib/librte_eal/windows/eal/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) #401: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:273: + QUEUE_TYPEOF(type) *swap_first = SLIST_FIRST(head1); \ ^ ERROR:SPACING: space required before the open parenthesis '(' #450: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:322: + for((var) = STAILQ_FIRST((head)); \ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #475: FILE: lib/librte_eal/windows/eal/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 #481: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:353: + if ((STAILQ_NEXT((elm), field) = STAILQ_FIRST((head))) == NULL) \ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #505: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:377: + QUEUE_TYPEOF(type) *curelm = STAILQ_FIRST(head); \ ^ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #514: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:386: + if ((STAILQ_NEXT(elm, field) = \ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #520: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:392: + if ((STAILQ_FIRST((head)) = \ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #526: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:398: + QUEUE_TYPEOF(type) *swap_first = STAILQ_FIRST(head1); \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #527: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:399: + QUEUE_TYPEOF(type) **swap_last = (head1)->stqh_last; \ ^ ERROR:CODE_INDENT: code indent should use tabs where possible #595: FILE: lib/librte_eal/windows/eal/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 #595: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:467: +^I ^Ipanic("Bad link elm %p next->prev != elm", (elm));^I\$ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #614: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:486: + QUEUE_TYPEOF(type) *curelm = LIST_FIRST(head1); \ ^ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #616: FILE: lib/librte_eal/windows/eal/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 #654: FILE: lib/librte_eal/windows/eal/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 #660: FILE: lib/librte_eal/windows/eal/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 #677: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:549: + if ((LIST_NEXT((elm), field) = LIST_FIRST((head))) != NULL) \ WARNING:SPACE_BEFORE_TAB: please, no space before tabs #696: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:568: +^I^ILIST_NEXT((elm), field)->field.le_prev = ^I^I\$ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #704: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:576: + QUEUE_TYPEOF(type) *swap_tmp = LIST_FIRST(head1); \ ^ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #707: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:579: + if ((swap_tmp = LIST_FIRST((head1))) != NULL) \ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #709: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:581: + if ((swap_tmp = LIST_FIRST((head2))) != NULL) \ ERROR:CODE_INDENT: code indent should use tabs where possible #741: FILE: lib/librte_eal/windows/eal/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 #741: FILE: lib/librte_eal/windows/eal/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 #782: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:738: + if ((TAILQ_NEXT((elm), field) = TAILQ_NEXT((listelm), field)) != NULL)\ WARNING:SPACE_BEFORE_TAB: please, no space before tabs #783: FILE: lib/librte_eal/windows/eal/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 #793: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:763: + if ((TAILQ_NEXT((elm), field) = TAILQ_FIRST((head))) != NULL) \ WARNING:LONG_LINE: line over 90 characters #803: FILE: lib/librte_eal/windows/eal/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 #803: FILE: lib/librte_eal/windows/eal/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 #811: FILE: lib/librte_eal/windows/eal/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 #812: FILE: lib/librte_eal/windows/eal/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 #821: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:812: +^I^ITAILQ_NEXT((elm), field)->field.tqe_prev = ^I^I\$ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #831: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:825: + QUEUE_TYPEOF(type) *swap_first = (head1)->tqh_first; \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #832: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:826: + QUEUE_TYPEOF(type) **swap_last = (head1)->tqh_last; \ ^ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #839: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:831: + if ((swap_first = (head1)->tqh_first) != NULL) \ ERROR:ASSIGN_IN_IF: do not use assignment in if condition #845: FILE: lib/librte_eal/windows/eal/include/sys/queue.h:835: + if ((swap_first = (head2)->tqh_first) != NULL) \ total: 30 errors, 15 warnings, 753 lines checked