[FFmpeg-soc] [soc]: r5821 - libavfilter/vf_overlay.c

Stefano Sabatini stefano.sabatini-lala at poste.it
Thu Jun 10 00:22:04 CEST 2010


On date Friday 2010-06-04 22:15:35 +0200, bcoudurier encoded:
> Author: bcoudurier
> Date: Fri Jun  4 22:15:35 2010
> New Revision: 5821
> 
> Log:
> Direct rendering in overlay filter.
> RGB24 support.
> Doesn't work with movie in movie yet, needs loop input feature for logos
> either in movie filter or here.

This broke movie in movie.

Again, please *don't* make massive changes and introduce regressions
without not even posting and discussing the changes before, testing
with ffplay is a bonus.

Consider to fix the regression or revert the patch if you can't do it
in a reasonable amount of time, or at least try to justify because
this change should be an improvement over the previous behaviour.

> Modified:
>    libavfilter/vf_overlay.c
> 
> Modified: libavfilter/vf_overlay.c
> ==============================================================================
> --- libavfilter/vf_overlay.c	Fri Jun  4 22:10:23 2010	(r5820)
> +++ libavfilter/vf_overlay.c	Fri Jun  4 22:15:35 2010	(r5821)
> @@ -1,5 +1,6 @@
>  /*
>   * copyright (c) 2007 Bobby Bingham
> + * copyright (c) 2010 Baptiste Coudurier
>   *
>   * This file is part of FFmpeg.
>   *
> @@ -44,13 +45,9 @@ enum var_name {
>  };
>  
>  typedef struct {
> -    int x, y;                   //< position of subpicture
> +    unsigned x, y;              //< position of subpicture
>  
> -    /** pics[0][0..1] are pictures for the main image.
> -     *  pics[1][0..1] are pictures for the sub image.
> -     *  pics[x][0]    are previously outputted images.
> -     *  pics[x][1]    are queued, yet unused frames for each input. */
> -    AVFilterPicRef *pics[2][2];
> +    AVFilterPicRef *overlay;
>  
>      int bpp;                    //< bytes per pixel

bpp (whose name and documentation was just wrong) is not used anymore.

Regards.


More information about the FFmpeg-soc mailing list