On 1/8/17, Mark Thompson <s...@jkqxz.net> wrote:
> On 08/01/17 19:27, Paul B Mahol wrote:
>> On 1/8/17, Mark Thompson <s...@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.
IIRC I see no other filter uses this approach.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
