[Ffmpeg-devel] [PATCH] use av_q2d() instead

Michael Niedermayer michaelni
Thu Mar 8 11:30:05 CET 2007


Hi

On Thu, Mar 08, 2007 at 01:26:25PM +0800, Limin Wang wrote:
> Hi,
> 
> please review the patch.
> 
> 
> Thanks,
> Limin

[...]
> @@ -1079,9 +1079,7 @@
>                          goto discard_packet;
>                      }
>                      if (ist->st->codec->time_base.num != 0) {
> -                        ist->next_pts += ((int64_t)AV_TIME_BASE *
> -                                          ist->st->codec->time_base.num) /
> -                            ist->st->codec->time_base.den;
> +                        ist->next_pts += ((int64_t)AV_TIME_BASE * av_q2d(ist->st->codec->time_base));

replacing exact integer calculations by approximations is not accpetable


[...]
-- 
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/20070308/9e12098e/attachment.pgp>



More information about the ffmpeg-devel mailing list