|WARNING| pw140666 [PATCH v21 12/14] log: add optional support of syslog

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Jun 4 02:51:23 CEST 2024


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

_coding style issues_


WARNING:STATIC_CONST_CHAR_ARRAY: char * array declaration might be better as static const
#115: FILE: app/test/test_eal_flags.c:990:
+	const char *argv4[] = {prgname, prefix, mp_flag, "--syslog=both"};

WARNING:STATIC_CONST_CHAR_ARRAY: char * array declaration might be better as static const
#118: FILE: app/test/test_eal_flags.c:992:
+	const char *argv5[] = {prgname, prefix, mp_flag, "--syslog=invalid"};

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 8)
#261: FILE: lib/log/log.c:513:
+	else
[...]
 	if (log_timestamp_enabled())

WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'?
#289: FILE: lib/log/log_stubs.c:5:
+#include <stdio.h>

WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'?
#321: FILE: lib/log/log_syslog.c:3:
+#include <stdio.h>

total: 0 errors, 5 warnings, 240 lines checked


More information about the test-report mailing list