|WARNING| pw166935 [PATCH 2/5] capture: infrastructure wireshark packet capture

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Jul 24 23:25:45 CEST 2026


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

_coding style issues_


WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'?
#154: FILE: lib/capture/capture.c:13:
+#include <stdio.h>
           ^^^^^

ERROR:SPACING: space prohibited before that close parenthesis ')'
#568: FILE: lib/capture/capture.c:427:
+capture_pcapng_open(const char *path, int *fd, uint16_t port_id, const char *filter )

WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#912: FILE: lib/capture/capture.c:771:
+			break;
+		} else {

WARNING:TYPO_SPELLING: 'respons' may be misspelled - perhaps 'response'?
#1043: FILE: lib/capture/capture.c:902:
+	/* return 0 since error reported "error":"XXX" in respons */
 	                                                  ^^^^^^^

WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'?
#1247: FILE: lib/capture/filter.c:6:
+#include <stdio.h>
           ^^^^^

WARNING:STRCPY: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
#1295: FILE: lib/capture/filter.c:54:
+	strcpy(flt->expr, filter);

total: 1 errors, 5 warnings, 1239 lines checked
Warning in lib/capture/capture.c:
Using %l format, prefer %PRI*64 if type is [u]int64_t
Warning in lib/capture/capture.c:
Prefer rte_thread_(set_prefixed_name|create_internal_control)


More information about the test-report mailing list