| OLD | NEW |
| 1 #!/bin/sh | 1 #!/bin/sh |
| 2 | 2 |
| 3 # if no argument provided, write stdin to a file and re-run the script | 3 # if no argument provided, write stdin to a file and re-run the script |
| 4 if [ $# = 0 ]; then | 4 if [ $# = 0 ]; then |
| 5 cat > patcheck.stdout | 5 cat > patcheck.stdout |
| 6 $0 patcheck.stdout | 6 $0 patcheck.stdout |
| 7 rm -f patcheck.stdout | 7 rm -f patcheck.stdout |
| 8 exit | 8 exit |
| 9 fi | 9 fi |
| 10 | 10 |
| (...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 88 hiegrep2 '\.sample_fmts *= *\(' 'const' 'missing const for sample_fmts array' $* | 88 hiegrep2 '\.sample_fmts *= *\(' 'const' 'missing const for sample_fmts array' $* |
| 89 hiegrep2 '\.supported_framerates *= *\(' 'const' 'missing const for supported_fr
amerates array' $* | 89 hiegrep2 '\.supported_framerates *= *\(' 'const' 'missing const for supported_fr
amerates array' $* |
| 90 hiegrep2 '\.channel_layouts *= *\(' 'const' 'missing const for channel_layouts a
rray' $* | 90 hiegrep2 '\.channel_layouts *= *\(' 'const' 'missing const for channel_layouts a
rray' $* |
| 91 | 91 |
| 92 #egrep $OPT '^\+.*const ' $*| grep -v 'static'> $TMP && printf '\nnon static con
st\n' | 92 #egrep $OPT '^\+.*const ' $*| grep -v 'static'> $TMP && printf '\nnon static con
st\n' |
| 93 #cat $TMP | 93 #cat $TMP |
| 94 | 94 |
| 95 hiegrep2 "$ERE_TYPES" '(static|av_|ff_|typedef|:\+[^a-zA-Z_])' 'Non static with
no ff_/av_ prefix' $* | 95 hiegrep2 "$ERE_TYPES" '(static|av_|ff_|typedef|:\+[^a-zA-Z_])' 'Non static with
no ff_/av_ prefix' $* |
| 96 | 96 |
| 97 hiegrep ':\+[^}#]*else' 'missing } prior to else' $* | 97 hiegrep ':\+[^}#]*else' 'missing } prior to else' $* |
| 98 hiegrep '(if|while|for)\(' 'missing whitespace between keyword and (' $* | 98 hiegrep '(if|while|for)\(' 'missing whitespace between keyword and ( (feel free
to ignore)' $* |
| 99 hiegrep '(else|do){' 'missing whitespace between keyword and {' $* | 99 hiegrep '(else|do){' 'missing whitespace between keyword and { (feel free
to ignore)' $* |
| 100 hiegrep '}(else|while)' 'missing whitespace between } and keyword' $* | 100 hiegrep '}(else|while)' 'missing whitespace between } and keyword (feel free
to ignore)' $* |
| 101 | 101 |
| 102 #FIXME this should print the previous statement maybe | 102 #FIXME this should print the previous statement maybe |
| 103 hiegrep ':\+ *{ *$' '{ should be on the same line as the related previous state
ment' $* | 103 hiegrep ':\+ *{ *$' '{ should be on the same line as the related previous state
ment' $* |
| 104 | 104 |
| 105 | 105 |
| 106 rm $TMP | 106 rm $TMP |
| 107 for i in `grep -H '^+.*@param' $*| sed 's/^\([^:]*\):.*@param\(\[.*\]\|\) *\([a-
zA-Z0-9_]*\) .*$/\1:\3/'` ; do | 107 for i in `grep -H '^+.*@param' $*| sed 's/^\([^:]*\):.*@param\(\[.*\]\|\) *\([a-
zA-Z0-9_]*\) .*$/\1:\3/'` ; do |
| 108 doxpar=`echo $i | sed 's/^.*:\(.*\)$/\1/'` | 108 doxpar=`echo $i | sed 's/^.*:\(.*\)$/\1/'` |
| 109 file=`echo $i | sed 's/^\([^:]*\):.*$/\1/'` | 109 file=`echo $i | sed 's/^\([^:]*\):.*$/\1/'` |
| 110 grep " *$doxpar *[),]" $file | grep -v '@param' >/dev/null || grep --color=a
lways "@param *$doxpar" $file >>$TMP | 110 grep " *$doxpar *[),]" $file | grep -v '@param' >/dev/null || grep --color=a
lways "@param *$doxpar" $file >>$TMP |
| (...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 167 | 167 |
| 168 vertical align = | 168 vertical align = |
| 169 /* and * align | 169 /* and * align |
| 170 arrays fitting in smaller types | 170 arrays fitting in smaller types |
| 171 variables written to twice with no interspaced read | 171 variables written to twice with no interspaced read |
| 172 memset(block, 0, 6*64*sizeof(DCTELEM)); -> clear_blocks | 172 memset(block, 0, 6*64*sizeof(DCTELEM)); -> clear_blocks |
| 173 check existence of long_name in AVCodec | 173 check existence of long_name in AVCodec |
| 174 check that the patch does not touch codec & (de)muxer layer at the same time ->s
plit | 174 check that the patch does not touch codec & (de)muxer layer at the same time ->s
plit |
| 175 | 175 |
| 176 write a regression test containing at least a line that triggers each warning on
ce | 176 write a regression test containing at least a line that triggers each warning on
ce |
| OLD | NEW |