diff options
author | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-08-07 15:26:19 +0200 |
---|---|---|
committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-08-07 15:32:36 +0200 |
commit | b413ba027312bc21e35c37e74a071c47d67676c8 (patch) | |
tree | 8d69aeedf2876c59499fb6943f119baf97410643 /src/validation.cpp | |
parent | 51c693d49ef1e9aabbc35669d86d952dcebeb61c (diff) | |
parent | bcd4b0f5cdde2a1b562a612c78ec1ef1fe47d3dd (diff) |
Merge #13705: build: Add format string linter
bcd4b0f5cdde2a1b562a612c78ec1ef1fe47d3dd Add linting of WalletLogPrintf(...) format strings (practicalswift)
a3e455694901a887e0feef69bd63e3aa122ea44b build: Add format string linter (practicalswift)
Pull request description:
Add format string linter.
This linter checks that the number of arguments passed to each variadic format string function matches the number of format specifiers in the format string.
Example output:
```
$ test/lint/lint-format-strings.sh
src/init.cpp: Expected 2 argument(s) after format string but found 1 argument(s):
LogPrintf("We have a mismatch here: foo=%s bar=%d\n", foo)
src/init.cpp: Expected 1 argument(s) after format string but found 2 argument(s):
LogPrint(BCLog::RPC, "RPC stopped. This is a mismatch: %s\n", s1, s2)
$ echo $?
1
```
Tree-SHA512: 19ab844a63f04bf193d66682ca42745a1c7d6c454b30222491b9fe8dc047054c4a6d3ee7921ec0676fb9ca2e7f6f93bd6c97996fb09667269bd491cb875349f3
Diffstat (limited to 'src/validation.cpp')
0 files changed, 0 insertions, 0 deletions