[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