contrib/check-code.py
changeset 42870 4257c33e24b7
parent 41826 867883d454ea
child 43060 63e25ae9cdce
equal deleted inserted replaced
42869:5ca351ba2478 42870:4257c33e24b7
   114     (r'tail -n', "don't use the '-n' option to tail, just use '-<num>'"),
   114     (r'tail -n', "don't use the '-n' option to tail, just use '-<num>'"),
   115     (r'sha1sum', "don't use sha1sum, use $TESTDIR/md5sum.py"),
   115     (r'sha1sum', "don't use sha1sum, use $TESTDIR/md5sum.py"),
   116     (r'\bls\b.*-\w*R', "don't use 'ls -R', use 'find'"),
   116     (r'\bls\b.*-\w*R', "don't use 'ls -R', use 'find'"),
   117     (r'printf.*[^\\]\\([1-9]|0\d)', r"don't use 'printf \NNN', use Python"),
   117     (r'printf.*[^\\]\\([1-9]|0\d)', r"don't use 'printf \NNN', use Python"),
   118     (r'printf.*[^\\]\\x', "don't use printf \\x, use Python"),
   118     (r'printf.*[^\\]\\x', "don't use printf \\x, use Python"),
   119     (r'\$\(.*\)', "don't use $(expr), use `expr`"),
       
   120     (r'rm -rf \*', "don't use naked rm -rf, target a directory"),
   119     (r'rm -rf \*', "don't use naked rm -rf, target a directory"),
   121     (r'\[[^\]]+==', '[ foo == bar ] is a bashism, use [ foo = bar ] instead'),
   120     (r'\[[^\]]+==', '[ foo == bar ] is a bashism, use [ foo = bar ] instead'),
   122     (r'(^|\|\s*)grep (-\w\s+)*[^|]*[(|]\w',
   121     (r'(^|\|\s*)grep (-\w\s+)*[^|]*[(|]\w',
   123      "use egrep for extended grep syntax"),
   122      "use egrep for extended grep syntax"),
   124     (r'(^|\|\s*)e?grep .*\\S', "don't use \\S in regular expression"),
   123     (r'(^|\|\s*)e?grep .*\\S', "don't use \\S in regular expression"),