[PATCH 3 of 4] check-code: capture "wc" as a word
Pierre-Yves David
pierre-yves.david at ens-lyon.org
Fri Aug 8 22:27:57 UTC 2014
# HG changeset patch
# User Pierre-Yves David <pierre-yves.david at fb.com>
# Date 1407024115 25200
# Sat Aug 02 17:01:55 2014 -0700
# Node ID 8fc5b99e6d48a053331c6f28cdd43aa255211255
# Parent ef62b424770799e3e9e970f7e0ef4a3493f72211
check-code: capture "wc" as a word
Otherwise entries such as "wcchange" give false negative
diff --git a/contrib/check-code.py b/contrib/check-code.py
--- a/contrib/check-code.py
+++ b/contrib/check-code.py
@@ -92,11 +92,11 @@ testpats = [
(r'grep.*-q', "don't use 'grep -q', redirect to /dev/null"),
(r'(?<!hg )grep.*-a', "don't use 'grep -a', use in-line python"),
(r'sed.*-i', "don't use 'sed -i', use a temporary file"),
(r'\becho\b.*\\n', "don't use 'echo \\n', use printf"),
(r'echo -n', "don't use 'echo -n', use printf"),
- (r'(^| )wc[^|]*$\n(?!.*\(re\))', "filter wc output"),
+ (r'(^| )\bwc\b[^|]*$\n(?!.*\(re\))', "filter wc output"),
(r'head -c', "don't use 'head -c', use 'dd'"),
(r'tail -n', "don't use the '-n' option to tail, just use '-<num>'"),
(r'sha1sum', "don't use sha1sum, use $TESTDIR/md5sum.py"),
(r'ls.*-\w*R', "don't use 'ls -R', use 'find'"),
(r'printf.*[^\\]\\([1-9]|0\d)', "don't use 'printf \NNN', use Python"),
More information about the Mercurial-devel
mailing list