[FFmpeg-devel] Killing AVFrac
Michael Niedermayer
michaelni
Mon Jul 9 00:34:16 CEST 2007
Hi
On Sun, Jul 08, 2007 at 09:26:03PM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
>
> > 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:
>
> But are the values computed there used for anything other than
> comparisons? From a quick glance it didn't seem like they were
> assigned as actual timestamps.
i dont know, i just pointed at all differences i saw after a quick review
also i think such changes would belong into a seperate patch ...
>
> >> @@ -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
>
> Well, there is the den/2 bias thing... What's the purpose of that
> anyway?
i was also wondering about that
i suspect it was supposed to improve the rounding of pts.val, that is
make it round to nearest but maybe it had some other purpose ...
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The worst form of inequality is to try to make unequal things equal.
-- 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/20070709/5aa01331/attachment.pgp>
More information about the ffmpeg-devel
mailing list