comparison contrib/check-code.py @ 12743:4c4aeaab2339

check-code: add 'no tab indent' check for unified tests and fix the offending tests accordingly
author Adrian Buehlmann <adrian@cadifra.com>
date Sat, 16 Oct 2010 18:09:01 +0200
parents 3acd5f7ab9d0
children 614f0d8724ab
comparison
equal deleted inserted replaced
12742:6ab4a7d3c179 12743:4c4aeaab2339
70 (r"( *)(#([^\n]*\S)?)", repcomment), 70 (r"( *)(#([^\n]*\S)?)", repcomment),
71 (r"<<(\S+)((.|\n)*?\n\1)", rephere), 71 (r"<<(\S+)((.|\n)*?\n\1)", rephere),
72 ] 72 ]
73 73
74 uprefix = r"^ \$ " 74 uprefix = r"^ \$ "
75 uprefixc = r"^ > "
75 utestpats = [ 76 utestpats = [
76 (uprefix + r'.*\|\s*sed', "use regex test output patterns instead of sed"), 77 (uprefix + r'.*\|\s*sed', "use regex test output patterns instead of sed"),
77 (uprefix + r'(true|exit 0)', "explicit zero exit unnecessary"), 78 (uprefix + r'(true|exit 0)', "explicit zero exit unnecessary"),
78 (uprefix + r'.*\$\?', "explicit exit code checks unnecessary"), 79 (uprefix + r'.*\$\?', "explicit exit code checks unnecessary"),
79 (uprefix + r'.*\|\| echo.*(fail|error)', 80 (uprefix + r'.*\|\| echo.*(fail|error)',
80 "explicit exit code checks unnecessary"), 81 "explicit exit code checks unnecessary"),
81 (uprefix + r'set -e', "don't use set -e"), 82 (uprefix + r'set -e', "don't use set -e"),
83 (uprefixc + r'( *)\t', "don't use tabs to indent"),
82 ] 84 ]
83 85
84 for p, m in testpats: 86 for p, m in testpats:
85 if p.startswith('^'): 87 if p.startswith('^'):
86 p = uprefix + p[1:] 88 p = uprefix + p[1:]