On Fri, Nov 20, 2015 at 08:13:11PM +0100, Paul B Mahol wrote:
[...]

> +static av_cold void uninit(AVFilterContext *ctx)
> +{
> +    DeFlashContext *s = ctx->priv;
> +}



> +
> +static const AVFilterPad inputs[] = {
> +    {
> +        .name         = "default",
> +        .type         = AVMEDIA_TYPE_VIDEO,
> +        .filter_frame = filter_frame,
> +        .config_props = config_input,
> +    },
> +    { NULL }
> +};
> +
> +static const AVFilterPad outputs[] = {
> +    {
> +        .name = "default",
> +        .type = AVMEDIA_TYPE_VIDEO,
> +    },
> +    { NULL }
> +};
> +
> +AVFilter ff_vf_deflash = {
> +    .name          = "deflash",
> +    .description   = NULL_IF_CONFIG_SMALL("Deflash."),
> +    .priv_size     = sizeof(DeFlashContext),
> +    .priv_class    = &deflash_class,

> +    .uninit        = uninit,

function is empty, this could be removed, unless you want to
keep it / plan to add something to it

either way, patch shuld be ok


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to