[FFmpeg-devel] [PATCH] patcheck: warn missing whitespace between ) and {

[Steven Liu]

> 2020年12月30日 下午6:54，Andreas Rheinhardt <andreas.rheinhardt at gmail.com> 写道：
> 
> liuqi05:
>> Signed-off-by: liuqi05 <liuqi05 at kuaishou.com>
>> ---
>> tools/patcheck | 1 +
>> 1 file changed, 1 insertion(+)
>> 
>> diff --git a/tools/patcheck b/tools/patcheck
>> index fe52938f29..ca7b57f29f 100755
>> --- a/tools/patcheck
>> +++ b/tools/patcheck
>> @@ -101,6 +101,7 @@ hiegrep2 "$ERE_TYPES" '(static|av_|ff_|typedef|:\+[^a-zA-Z_])' 'Non static with
>> hiegrep ':\+[^}#]*else' 'missing } prior to else' $*
>> hiegrep '(if|while|for)\(' 'missing whitespace between keyword and ( (feel free to ignore)' $*
>> hiegrep '(else|do){'       'missing whitespace between keyword and { (feel free to ignore)' $*
>> +hiegrep '\){'              'missing whitespace between ) and { (feel free to ignore)' $*
>> hiegrep '}(else|while)'    'missing whitespace between } and keyword (feel free to ignore)' $*
>> 
>> #FIXME this should print the previous statement maybe
>> Missing whitespace is actually common in case of compound literals.
So (feel free to ignore) as above line.
> 
> - Andreas
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".

Thanks

Steven Liu