[FFmpeg-devel] [PATCH] AV_TIME_BASE and av_gettime()
Luca Abeni
lucabe72
Sun Nov 23 12:28:01 CET 2008
Hi,
Stefano Sabatini wrote:
[...]
>> I do not think that defining a new av_gettime2() function just for this
>> purpose is a good idea.
>
> Mmh... yes I agree.
>
> Check the patch, I also changed the name pts -> next, I think it's
> less prone to confusion since "pts" may be confused with the meaning
> of pts in ist->next_pts (which uses another time unit).
The new changes look good to me. I hope the patch can be split so that
Michael can approve it :)
> BTW if that's OK I think it's also safe to remove the "frame" var,
> which is now unused.
Yes, I agree... I do not know if this must be done in a separate patch.
Luca
More information about the ffmpeg-devel
mailing list