diff options
| author |  Kurtis Rader <krader@skepticism.us> | 2016-04-19 19:49:15 -0700 |
|---|---|---|
| committer | Kurtis Rader <krader@skepticism.us> | 2016-04-26 21:58:59 -0700 |
| commit | df10b53c0caecd43a02b7f24515a9ff9edea7056 (patch) | |
| tree | c4330d36de296ee75a608a6665091134be780167 /build_tools | |
| parent | c2f9d60eb1f04a3aa0b01d785c1e133cac8ecf1d (diff) | |
restyle builtin modules to match project style
Now that the IWYU cleanup has been merged compile all, not just a couple, of
the builtin modules independent of builtin.cpp. That is, no longer `#include
builtin_NAME.cpp` in builtin.cpp. This is more consistent, more in line with
what developers expect, and is likely to reduce mistakes.
Reduces lint errors from 384 to 336 (-13%). Line count from 6307 to 4988 (-21%).
Another step in resolving issue #2902.
Diffstat (limited to 'build_tools')
| -rw-r--r-- | build_tools/iwyu.linux.imp | 2 | ||||
| -rwxr-xr-x | build_tools/style.fish | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/build_tools/iwyu.linux.imp b/build_tools/iwyu.linux.imp index 6fc614ae..27700b67 100644 --- a/build_tools/iwyu.linux.imp +++ b/build_tools/iwyu.linux.imp @@ -14,4 +14,6 @@ { symbol: ["size_t", "private", "<stddef.h>", "public"] }, { symbol: ["size_t", "private", "<stdlib.h>", "public"] }, { symbol: ["uint64_t", "private", "<sys/types.h>", "public"] }, + { symbol: ["memset", "private", "<string.h>", "public"] }, + { symbol: ["strerror", "private", "<string.h>", "public"] }, ] diff --git a/build_tools/style.fish b/build_tools/style.fish index 62d8d5c7..fc9a96e3 100755 --- a/build_tools/style.fish +++ b/build_tools/style.fish @@ -39,7 +39,7 @@ else set git_clang_format yes else # No pending changes so lint the files in the most recent commit. - set files (git show --name-only --pretty=oneline head | tail --lines=+2) + set files (git show --name-only --pretty=oneline | tail --lines=+2) end # Extract just the C/C++ files. |