[LTP] [PATCH 0/3] tools/sparse: Add static check
Richard Palethorpe
rpalethorpe@suse.com
Tue Nov 23 13:43:45 CET 2021
Hello,
This teaches 'make check' to flag missing static keywords. Note that I
am not sure how this avoids flagging symbols only declared in header
files. However I haven't noticed any false positives and I think false
negatives will only occur when the author uses function prototypes to
reference private symbols before their definition.
The statx patch is just included to show one issue it caught.
Richard Palethorpe (3):
tools/sparse: Add static check
doc: Add LTP-003 and LTP-004 static and tst API prefix rules
statx: Add missing static keyword to tcase variable
doc/library-api-writing-guidelines.txt | 10 +++++
doc/rules.tsv | 2 +
doc/test-writing-guidelines.txt | 8 ++++
testcases/kernel/syscalls/statx/statx01.c | 2 +-
testcases/kernel/syscalls/statx/statx02.c | 2 +-
testcases/kernel/syscalls/statx/statx04.c | 2 +-
testcases/kernel/syscalls/statx/statx05.c | 2 +-
testcases/kernel/syscalls/statx/statx07.c | 2 +-
tools/sparse/sparse-ltp.c | 53 +++++++++++++++++++++++
9 files changed, 78 insertions(+), 5 deletions(-)
--
2.33.1
More information about the ltp
mailing list