[FFmpeg-soc] [soc]: r4714 - in afilters: af_null.c dummy.c

Benoit Fouet benoit.fouet at free.fr
Wed Jul 15 20:43:05 CEST 2009


kdub wrote :
> Author: kdub
> Date: Tue Jul 14 10:09:58 2009
> New Revision: 4714
>
> Log:
> memcpy the instance of the filter instead of having code duplication
>
> Modified:
>    afilters/af_null.c
>    afilters/dummy.c
>
>   

[...]

> Modified: afilters/dummy.c
> ==============================================================================
> --- afilters/dummy.c	Mon Jul 13 20:36:13 2009	(r4713)
> +++ afilters/dummy.c	Tue Jul 14 10:09:58 2009	(r4714)
> @@ -102,14 +102,15 @@ int main()
>      /* set up the first filter */
>      AVFilterContext * avfiltcont=NULL;
>      AVFilter *avfilt;
> -    avfilt = &avfilter_af_null;
> +    avfilt = (AVFilter*)malloc(sizeof(AVFilter));
>   

useless cast; and I would have written sizeof(*avfilt)

> +    memcpy(avfilt, &avfilter_af_null, sizeof(AVFilter));
>   

sizeof(*avfilt)

>      avfiltcont = avfilter_open(avfilt, "filterID1234");
>      avfilter_register(avfilt);
>  
>      /* set up the first filter */
>      AVFilterContext * avfiltcont2=NULL;
>      AVFilter *avfilt2;
> -    avfilt2 = &avfilter_af_null2;
> +    avfilt2 = (AVFilter*)malloc(sizeof(AVFilter));
>   

same here

>      memcpy(avfilt2, &avfilter_af_null, sizeof(AVFilter));
>   

and here

Ben


More information about the FFmpeg-soc mailing list