diff options
author | Jim Meyering <meyering@redhat.com> | 2012-05-24 16:21:34 +0200 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2012-06-10 22:02:26 +0200 |
commit | 5b9206c1a851a29d7088bcef000e0c887200c5df (patch) | |
tree | 4590f18b336c61e1af104fbf4f54d94f815d07fe /cfg.mk | |
parent | 928dd73762e69cfeaab4a7ec9dd8f30f86a45ed4 (diff) | |
download | coreutils-5b9206c1a851a29d7088bcef000e0c887200c5df.tar.xz |
maint: prohibit use of strncpy
* cfg.mk (sc_prohibit_strncpy): New syntax-check rule.
Exempt pinky.c and who.c, at least for now.
Diffstat (limited to 'cfg.mk')
-rw-r--r-- | cfg.mk | 9 |
1 files changed, 9 insertions, 0 deletions
@@ -403,6 +403,12 @@ sc_prohibit_strncmp: { echo '$(ME): use STREQ_LEN or STRPREFIX instead of str''ncmp' \ 1>&2; exit 1; } || : +# Really. You don't want to use this function. +sc_prohibit_strncpy: + @prohibit='\<str''ncpy *\(' \ + halt='do not use str''ncpy, period' \ + $(_sc_search_regexp) + # Enforce recommended preprocessor indentation style. sc_preprocessor_indentation: @if cppi --version >/dev/null 2>&1; then \ @@ -485,3 +491,6 @@ exclude_file_name_regexp--sc_prohibit_test_backticks = \ # Exempt test.c, since it's nominally shared, and relatively static. exclude_file_name_regexp--sc_prohibit_operator_at_end_of_line = \ ^src/(ptx|test|head)\.c$$ + +# Exempt pinky and who: their uses of this function appear to be correct. +exclude_file_name_regexp--sc_prohibit_strncpy = ^src/(pinky|who)\.c$$ |