automatic DPDK test reports
 help / color / mirror / Atom feed
* |WARNING| pw129196 [v2 2/5] app/testpmd: support indirect actions list syntax
       [not found] <20230702045758.23244-2-igozlan@nvidia.com>
@ 2023-07-03  9:22 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2023-07-03  9:22 UTC (permalink / raw)
  To: test-report; +Cc: Itamar Gozlan

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

_coding style issues_


ERROR:CODE_INDENT: code indent should use tabs where possible
#321: FILE: app/test-pmd/cmdline_flow.c:7330:
+^I^I^I^I           rte_flow_action_indirect_list)),$

ERROR:C99_COMMENTS: do not use C99 // comments
#341: FILE: app/test-pmd/cmdline_flow.c:7350:
+//		.comp = comp_none,

ERROR:C99_COMMENTS: do not use C99 // comments
#347: FILE: app/test-pmd/cmdline_flow.c:7356:
+//		.comp = comp_none,

ERROR:CODE_INDENT: code indent should use tabs where possible
#369: FILE: app/test-pmd/cmdline_flow.c:7893:
+        case INDIRECT_ACTION_FLOW_CONF:$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#369: FILE: app/test-pmd/cmdline_flow.c:7893:
+        case INDIRECT_ACTION_FLOW_CONF:$

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#382: FILE: app/test-pmd/cmdline_flow.c:11405:
+		    void __rte_unused *buf, unsigned int __rte_unused size)
 		                      ^

ERROR:CODE_INDENT: code indent should use tabs where possible
#403: FILE: app/test-pmd/cmdline_flow.c:11426:
+        action_conf->handle = (typeof(action_conf->handle))$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#403: FILE: app/test-pmd/cmdline_flow.c:11426:
+        action_conf->handle = (typeof(action_conf->handle))$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 8)
#501: FILE: app/test-pmd/cmdline_flow.c:12531:
+	if (indlst_conf)
+        printf("created indirect action list configuration %u
",

ERROR:CODE_INDENT: code indent should use tabs where possible
#502: FILE: app/test-pmd/cmdline_flow.c:12532:
+        printf("created indirect action list configuration %u
",$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#502: FILE: app/test-pmd/cmdline_flow.c:12532:
+        printf("created indirect action list configuration %u
",$

ERROR:CODE_INDENT: code indent should use tabs where possible
#503: FILE: app/test-pmd/cmdline_flow.c:12533:
+               in->args.vc.attr.group);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#503: FILE: app/test-pmd/cmdline_flow.c:12533:
+               in->args.vc.attr.group);$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 8)
#504: FILE: app/test-pmd/cmdline_flow.c:12534:
+	else
+        printf("cannot create indirect action list configuration %u
",

ERROR:CODE_INDENT: code indent should use tabs where possible
#505: FILE: app/test-pmd/cmdline_flow.c:12535:
+        printf("cannot create indirect action list configuration %u
",$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#505: FILE: app/test-pmd/cmdline_flow.c:12535:
+        printf("cannot create indirect action list configuration %u
",$

ERROR:CODE_INDENT: code indent should use tabs where possible
#506: FILE: app/test-pmd/cmdline_flow.c:12536:
+               in->args.vc.attr.group);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#506: FILE: app/test-pmd/cmdline_flow.c:12536:
+               in->args.vc.attr.group);$

total: 10 errors, 8 warnings, 411 lines checked

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-07-03  9:22 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20230702045758.23244-2-igozlan@nvidia.com>
2023-07-03  9:22 ` |WARNING| pw129196 [v2 2/5] app/testpmd: support indirect actions list syntax 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).