[Ffmpeg-devel] [PATCH] cosmetics and minor bugfix to True Audio Demuxer

Diego Biurrun diego
Sat Mar 31 13:50:47 CEST 2007


On Sat, Mar 31, 2007 at 08:16:41PM +0900, Masahiro Kiyota wrote:
> 
> --- libavformat/tta.c.orig	2007-03-31 16:53:16.000000000 +0900
> +++ libavformat/tta.c	2007-03-31 20:03:12.000000000 +0900
> @@ -21,6 +21,8 @@
>  
> +#define FRAME_TIME 1.04489795918367346939
> +
> @@ -64,7 +66,7 @@ static int tta_read_header(AVFormatConte
>  
> -    framelen = 1.04489795918367346939 * samplerate;
> +    framelen = FRAME_TIME * samplerate;

Michael already fixed this differently.

> @@ -77,7 +79,7 @@ static int tta_read_header(AVFormatConte
>  
>      for (i = 0; i < c->totalframes; i++)
> -            c->seektable[i] = get_le32(&s->pb);
> +        c->seektable[i] = get_le32(&s->pb);

This hunk applied.

> @@ -109,7 +111,7 @@ static int tta_read_packet(AVFormatConte
>  
>      // FIXME!
> -    if (c->currentframe > c->totalframes)
> +    if (c->currentframe >= c->totalframes)

What's the problem here and why the FIXME?

Diego





More information about the ffmpeg-devel mailing list