[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