* cfg.mk (false_positives): Enable syntax checker sc_the_the * build-aux/src-sniff.py (checkers): Avoid false positive with sc_the_the.
Signed-off-by: James Youngman <[email protected]> --- ChangeLog | 5 +++++ build-aux/src-sniff.py | 2 +- cfg.mk | 2 +- 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index a5a0fd0..48b7e14 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,10 @@ 2010-04-02 James Youngman <[email protected]> + Enable syntax checker sc_the_the, detecting doubled 'the'. + * cfg.mk (false_positives): Enable syntax checker sc_the_the + * build-aux/src-sniff.py (checkers): Avoid false positive with + sc_the_the. + Avoid false positives for sc_prohibit_atoi_atof. * find/util.c (set_option_defaults): Avoid false positive with sc_prohibit_atoi_atof. diff --git a/build-aux/src-sniff.py b/build-aux/src-sniff.py index 9d26f0b..cfd98c0 100644 --- a/build-aux/src-sniff.py +++ b/build-aux/src-sniff.py @@ -96,7 +96,7 @@ checkers = [ [r'HAVE''_CONFIG_H', "Avoid checking HAVE_CONFIG_H"], # [r'HAVE_FCNTL_H', "Avoid checking HAVE_FCNTL_H"], [r'O_NDELAY', "Avoid using O_NDELAY"], - [r'the *the', "'the the' is probably not deliberate"], + [r'the *the', "'the"+" the' is probably not deliberate"], [r'(?<!\w)error \([^_"]*[^_]"[^"]*[a-z]{3}', "untranslated error message"], [r'^# *if\s+defined *\(', "useless parentheses in '#if defined'"], diff --git a/cfg.mk b/cfg.mk index b43b3e5..47f49e0 100644 --- a/cfg.mk +++ b/cfg.mk @@ -29,7 +29,7 @@ skip_defer = sc_program_name \ # False positives I don't have a workaround for yet. # sc_space_tab: several .xo test output files contain this sequence # for testing xargs's handling of white space. -false_positives = sc_obsolete_symbols sc_prohibit_cvs_keyword sc_the_the \ +false_positives = sc_obsolete_symbols sc_prohibit_cvs_keyword \ sc_two_space_separator_in_usage \ sc_space_tab -- 1.7.0 _______________________________________________ Findutils-patches mailing list [email protected] http://lists.gnu.org/mailman/listinfo/findutils-patches
