devtools: check %l format specifier
%lx or %llx tend to be wrong for 32-bit platform if used for fixed size variable like uint64_t. A checkpatch warning will avoid this common mistake. Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
This commit is contained in:
parent
fcc5489c18
commit
d991d879f9
@ -69,6 +69,14 @@ check_forbidden_additions() { # <patch>
|
||||
-f $(dirname $(readlink -f $0))/check-forbidden-tokens.awk \
|
||||
"$1" || res=1
|
||||
|
||||
# check %l or %ll format specifier
|
||||
awk -v FOLDERS='lib drivers app examples' \
|
||||
-v EXPRESSIONS='%ll*[xud]' \
|
||||
-v RET_ON_FAIL=1 \
|
||||
-v MESSAGE='Using %l format, prefer %PRI*64 if type is [u]int64_t' \
|
||||
-f $(dirname $(readlink -f $0))/check-forbidden-tokens.awk \
|
||||
"$1" || res=1
|
||||
|
||||
# forbid variable declaration inside "for" loop
|
||||
awk -v FOLDERS='.' \
|
||||
-v EXPRESSIONS='for[[:space:]]*\\((char|u?int|unsigned|s?size_t)' \
|
||||
|
Loading…
Reference in New Issue
Block a user