[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