[FFmpeg-devel] [PATCH] avformat: add vapoursynth wrapper

wm4 nfxjfg at googlemail.com
Sat Apr 28 14:51:56 EEST 2018


On Sat, 28 Apr 2018 11:08:01 +0800
Steven Liu <lq at chinaffmpeg.org> wrote:

> > On 28 Apr 2018, at 03:37, wm4 <nfxjfg at googlemail.com> wrote:
> > 

> > +
> > +        if (strncmp(pd->name, "xyz", 3) == 0)
> > +            continue;  
> 
> liuqideMacBook-Pro:xxx liuqi$ ../tools/patcheck ~/Downloads/FFmpeg-devel-avformat-add-vapoursynth-wrapper.patch
> patCHeck 1e10.0
> This tool is intended to help a human check/review patches. It is very far from
> being free of false positives and negatives, and its output are just hints of what
> may or may not be bad. When you use it and it misses something or detects
> something wrong, fix it and send a patch to the ffmpeg-devel mailing list.
> License: GPL, Author: Michael Niedermayer
> egrep: empty (sub)expression
> 
> These functions may need av_cold, please review the whole patch for similar functions needing av_cold
> /Users/liuqi/Downloads/FFmpeg-devel-avformat-add-vapoursynth-wrapper.patch:147:+static int is_native_endian(enum AVPixelFormat pixfmt)

av_cold seems dumb but I guess I can cargo-cult that.

> x==0 / x!=0 can be simplified to !x / x
> /Users/liuqi/Downloads/FFmpeg-devel-avformat-add-vapoursynth-wrapper.patch:193:+        if (strncmp(pd->name, "xyz", 3) == 0)
> /Users/liuqi/Downloads/FFmpeg-devel-avformat-add-vapoursynth-wrapper.patch:204:+                c->offset != 0 || c->shift != 0 ||

I prefer to keep those explicit, especially the strncmp one. But I
don't have that strong feelings about it, so if someone minds I can
still change it.


More information about the ffmpeg-devel mailing list