[FFmpeg-devel] Killing AVFrac

[Michael Niedermayer]

Hi

On Sun, Jul 08, 2007 at 04:21:57PM +0100, M?ns Rullg?rd wrote:
> Attached patch is an attempt to remove use of AVFrac.  It works, apart
> >from audio timestamps occasionally being off by one.  Can anyone see
> what might be causing the difference?

well the code in ffmpeg.c does convert to double before the divide and:

[...]

> @@ -2177,7 +2175,7 @@ static int compute_pkt_fields2(AVStream *st, AVPacket *pkt){
>  
>  //    av_log(NULL, AV_LOG_DEBUG, "av_write_frame: pts2:%"PRId64" dts2:%"PRId64"\n", pkt->pts, pkt->dts);
>      st->cur_dts= pkt->dts;
> -    st->pts.val= pkt->dts;
> +    st->pts.num= pkt->dts * st->pts.den;
>  
>      /* update pts */
>      switch (st->codec->codec_type) {

is also not what it was, that said iam not saying the code makes sense as it
is currently


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I count him braver who overcomes his desires than him who conquers his
enemies for the hardest victory is over self. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070708/46637b9a/attachment.pgp>