[FFmpeg-devel] [PATCH] lavfi: Add VAAPI deinterlacer

[Mark Thompson]

On 08/01/17 19:27, Paul B Mahol wrote:
> On 1/8/17, Mark Thompson <sw at jkqxz.net> wrote:
>> +
>> +AVFilter ff_vf_deinterlace_vaapi = {
>> +    .name           = "deinterlace_vaapi",
>> +    .description    = NULL_IF_CONFIG_SMALL("Deinterlacing of VAAPI
>> surfaces"),
>> +    .priv_size      = sizeof(DeintVAAPIContext),
>> +    .init           = &deint_vaapi_init,
>> +    .uninit         = &deint_vaapi_uninit,
> 
> Why & here?

Explicitly taking the address of the function?  I weakly prefer this style, because the type isn't really matching otherwise (without it you are invoking an implicit conversion from "void f(void)" to "void (*f)(void)" or similar).

I can change it easily if you object.

Thanks,

- Mark