[vlc-devel] [PATCH 02/11] Render all rows, even blank ones

Rafaël Carré funman at videolan.org
Mon Dec 31 00:43:03 CET 2012


Le 27/12/2012 07:08, Devin Heitmueller a écrit :
> We need to render all the rows, or else the placement of popup captions will
> not be correct.
> 
> The previous code worked fine when rendering text horizontally aligned at the
> bottom of the screen, but won't work for a real layout.
> ---
>  modules/codec/cc.c |    3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/modules/codec/cc.c b/modules/codec/cc.c
> index e9590bc..86ec51e 100644
> --- a/modules/codec/cc.c
> +++ b/modules/codec/cc.c
> @@ -1098,9 +1098,6 @@ static char *Eia608Text( eia608_t *h, bool b_html )
>          Eia608Strlcat( psz, "<text>", i_size );
>      for( i = 0; i < EIA608_SCREEN_ROWS; i++ )

Change "for ( i" to "for( int i"
And the whole series will merge with last HEAD :)


>      {
> -        if( !screen->row_used[i] )
> -            continue;
> -
>          if( !b_first )
>              Eia608Strlcat( psz, b_html ? "<br />" : "\n", i_size );
>          b_first = false;
> 




More information about the vlc-devel mailing list