[FFmpeg-devel] [PATCH] AV_TIME_BASE and av_gettime()

Stefano Sabatini stefano.sabatini-lala
Sat Nov 22 10:25:33 CET 2008


On date Friday 2008-11-21 08:53:27 +0100, Luca Abeni encoded:
[...]
> This looks good, and is a nice improvement respect to the current state
> of things...
> Just one comment: I think you are assuming AV_TIME_BASE == 1000000 (since
> you are comparing next_pts with the result of av_gettime()). This is the
> case now, but in the future the AV_TIME_BASE value can change (I know this
> is a problem, because I did the same mistake in the past, and I've been
> asked to fix it :)
> I am not ffmpeg.c maintainer (so, I think Michael's approval is needed),
> but I think the patch should be applied (after fixing the
> AV_TIME_BASE -> microseconds conversion).

Check the patch attached.

Regards.
-- 
FFmpeg = Furious and Friendly Merciful Patchable Emblematic Geek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: implement-av-gettime2-00.patch
Type: text/x-diff
Size: 1026 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081122/7c5e899e/attachment.patch>



More information about the ffmpeg-devel mailing list