Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
小白菜888
Ffmpeg
提交
f11c9ee9
F
Ffmpeg
项目概览
小白菜888
/
Ffmpeg
通知
3
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
DevOps
流水线
流水线任务
计划
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
F
Ffmpeg
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
DevOps
DevOps
流水线
流水线任务
计划
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
流水线任务
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
f11c9ee9
编写于
3月 11, 2009
作者:
A
Alex Converse
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
patcheck: Replace non-POSIX echo -e with printf.
Originally committed as revision 17943 to
svn://svn.ffmpeg.org/ffmpeg/trunk
上级
fe91924d
变更
1
隐藏空白更改
内联
并排
Showing
1 changed file
with
14 addition
and
14 deletion
+14
-14
tools/patcheck
tools/patcheck
+14
-14
未找到文件。
tools/patcheck
浏览文件 @
f11c9ee9
...
...
@@ -19,7 +19,7 @@ hiegrep(){
arg
=
"
$1
"
msg
=
"
$2
"
shift
2
grep
$OPT
'^+'
$*
|
grep
-v
':+++'
| egrep
--color
=
always
--
"
$arg
"
>
$TMP
&&
echo
-e
"
\n
$msg
"
grep
$OPT
'^+'
$*
|
grep
-v
':+++'
| egrep
--color
=
always
--
"
$arg
"
>
$TMP
&&
printf
"
\n
$msg
\n
"
cat
$TMP
}
...
...
@@ -28,7 +28,7 @@ hiegrep2(){
varg
=
"
$2
"
msg
=
"
$3
"
shift
3
grep
$OPT
'^+'
$*
|
grep
-v
':+++'
| egrep
-v
--
"
$varg
"
| egrep
--color
=
always
--
"
$arg
"
>
$TMP
&&
echo
-e
"
\n
$msg
"
grep
$OPT
'^+'
$*
|
grep
-v
':+++'
| egrep
-v
--
"
$varg
"
| egrep
--color
=
always
--
"
$arg
"
>
$TMP
&&
printf
"
\n
$msg
\n
"
cat
$TMP
}
...
...
@@ -48,7 +48,7 @@ hiegrep '\+= *1 *;' 'can be simplified to ++' $*
hiegrep
'-= *1 *;'
'can be simplified to --'
$*
hiegrep
'((!|=)= *(0|NULL)[^0-9a-z]|[^0-9a-z](0|NULL) *(!|=)=)'
'x==0 / x!=0 can be simplified to !x / x'
$*
egrep
$OPT
'^\+ *(const *|)static'
$*
| egrep
--color
=
always
'[^=]= *(0|NULL)[^0-9a-zA-Z]'
>
$TMP
&&
echo
-e
'\nuseless 0 init
'
egrep
$OPT
'^\+ *(const *|)static'
$*
| egrep
--color
=
always
'[^=]= *(0|NULL)[^0-9a-zA-Z]'
>
$TMP
&&
printf
'\nuseless 0 init\n
'
cat
$TMP
hiegrep
'# *ifdef * (HAVE|CONFIG)_'
'ifdefs that should be #if'
$*
...
...
@@ -72,7 +72,7 @@ hiegrep ' *static *'"$ERE_FUNCS"'[^)]*\);' 'static prototype, maybe you should r
hiegrep2
'\.long_name *='
'NULL_IF_CONFIG_SMAL'
'missing NULL_IF_CONFIG_SMAL'
$*
#egrep $OPT '^\+.*const ' $*| grep -v 'static'> $TMP &&
echo -e '\nnon static const
'
#egrep $OPT '^\+.*const ' $*| grep -v 'static'> $TMP &&
printf '\nnon static const\n
'
#cat $TMP
hiegrep2
"
$ERE_TYPES
"
'(static|av_|ff_|typedef|:\+[^a-zA-Z_])'
'Non static with no ff_/av_ prefix'
$*
...
...
@@ -90,11 +90,11 @@ for i in `grep -H '^+.*@param' $*| sed 's/^\([^:]*\):.*@param\(\[.*\]\|\) *\([a-
grep
" *
$doxpar
*[),]"
$file
|
grep
-v
'@param'
>
/dev/null
||
grep
--color
=
always
"@param *
$doxpar
"
$file
>>
$TMP
done
if
test
-e
$TMP
;
then
echo
-e
'\nmismatching doxy params
'
printf
'\nmismatching doxy params\n
'
cat
$TMP
fi
egrep
-B2
$OPT
'^(\+|) *('
"
$ERE_TYPES
"
'|# *define)'
$*
| egrep
-A2
--color
=
always
'(:|-)\+[^/]*/(\*([^*]|$)|/([^/]|$))'
>
$TMP
&&
echo
-e
"
\n
Non doxy comments
"
egrep
-B2
$OPT
'^(\+|) *('
"
$ERE_TYPES
"
'|# *define)'
$*
| egrep
-A2
--color
=
always
'(:|-)\+[^/]*/(\*([^*]|$)|/([^/]|$))'
>
$TMP
&&
printf
"
\n
Non doxy comments
\n
"
cat
$TMP
rm
$TMP
...
...
@@ -112,30 +112,30 @@ for i in \
egrep
-v
$i
' *= *(0x|)[0-9]{1,};'
>
/dev/null
||
echo
"possibly constant :"
$i
>>
$TMP
done
if
test
-e
$TMP
;
then
echo
-e
'\npossibly unused variables
'
printf
'\npossibly unused variables\n
'
cat
$TMP
fi
grep
'^Index:.*Changelog'
$*
>
/dev/null
||
echo
-e
"
\n
Missing changelog entry (ignore if minor change)
"
grep
'^Index:.*Changelog'
$*
>
/dev/null
||
printf
"
\n
Missing changelog entry (ignore if minor change)
\n
"
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'(fprintf|av_log|printf)\([^)]*\)[+ ;@]*\1'
>
$TMP
&&
echo
-e
"
\n
Mergeable calls
"
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'(fprintf|av_log|printf)\([^)]*\)[+ ;@]*\1'
>
$TMP
&&
printf
"
\n
Mergeable calls
\n
"
cat
$TMP
|
tr
'@'
'\n'
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'\+ *if *\( *([A-Za-z0-9_]*) *[<>]=? *[0-9]* *\) * \1 *= *[0-9]* *;[ @\\+]*else *if *\( *\1 *[<>]=? *[0-9]* *\) *\1 *= *[0-9]* *;'
>
$TMP
&&
echo
-e
"
\n
av_clip / av_clip_uint8 / av_clip_int16 / ...
"
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'\+ *if *\( *([A-Za-z0-9_]*) *[<>]=? *[0-9]* *\) * \1 *= *[0-9]* *;[ @\\+]*else *if *\( *\1 *[<>]=? *[0-9]* *\) *\1 *= *[0-9]* *;'
>
$TMP
&&
printf
"
\n
av_clip / av_clip_uint8 / av_clip_int16 / ...
\n
"
cat
$TMP
|
tr
'@'
'\n'
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'\+ *if *\( *([A-Za-z0-9_]*) *[<>]=? *([A-Za-z0-9_]*) *\)[ @\\+]*(\1|\2) *= *(\1|\2) *;'
>
$TMP
&&
echo
-e
"
\n
FFMIN/FFMAX
"
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'\+ *if *\( *([A-Za-z0-9_]*) *[<>]=? *([A-Za-z0-9_]*) *\)[ @\\+]*(\1|\2) *= *(\1|\2) *;'
>
$TMP
&&
printf
"
\n
FFMIN/FFMAX
\n
"
cat
$TMP
|
tr
'@'
'\n'
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'\+ *if *\( *([A-Za-z0-9_]*) *\)[ @\\+]*av_free(p|) *\( *(&|) *\1[^-.]'
>
$TMP
&&
echo
-e
"
\n
av_free(NULL) is safe
"
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'\+ *if *\( *([A-Za-z0-9_]*) *\)[ @\\+]*av_free(p|) *\( *(&|) *\1[^-.]'
>
$TMP
&&
printf
"
\n
av_free(NULL) is safe
\n
"
cat
$TMP
|
tr
'@'
'\n'
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'[^a-zA-Z0-9_]([a-zA-Z0-9_]*) *= *av_malloc *\([^)]*\)[ @;\\+]*memset *\( *\1'
>
$TMP
&&
echo
-e
"
\n
av_mallocz()
"
cat
$*
|
tr
'\n'
'@'
| egrep
--color
=
always
-o
'[^a-zA-Z0-9_]([a-zA-Z0-9_]*) *= *av_malloc *\([^)]*\)[ @;\\+]*memset *\( *\1'
>
$TMP
&&
printf
"
\n
av_mallocz()
\n
"
cat
$TMP
|
tr
'@'
'\n'
# doesnt work
#cat $* | tr '\n' '@' | egrep -o '[^a-zA-Z_0-9]([a-zA-Z][a-zA-Z_0-9]*) *=[^=].*\1' | egrep -o '[^a-zA-Z_0-9]([a-zA-Z][a-zA-Z_0-9]*) *=[^=].*\1 *=[^=]' >$TMP &&
echo -e "\nPossibly written 2x before read
"
#cat $* | tr '\n' '@' | egrep -o '[^a-zA-Z_0-9]([a-zA-Z][a-zA-Z_0-9]*) *=[^=].*\1' | egrep -o '[^a-zA-Z_0-9]([a-zA-Z][a-zA-Z_0-9]*) *=[^=].*\1 *=[^=]' >$TMP &&
printf "\nPossibly written 2x before read\n
"
#cat $TMP | tr '@' '\n'
exit
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录