[FFmpeg-devel] AVFrac deprecated

[Måns Rullgård]

Michael Niedermayer <michaelni at gmx.at> writes:

> On Thu, Dec 13, 2007 at 04:56:03PM +0100, Thorsten Jordan wrote:
>> Hello,
>> 
>> AVFrac is listed as deprecated in avformat.h with the additional comment
>> that one should use rather AVRational. However AVRational has limited
>> range using only int's, whereas AVFrac.val can hold much bigger numbers,
>> which is needed for timestamp generation when writing e.g. .MP4 containers.
>> What's the point with that comment to use AVRational?
>> Should it rather be replaced by int64_t base and AVRational extension (f
>> = base + AVRational) - but it would be very similar to current AVFrac
>> define.
>
> the idea was to replace int64_t + rational by int64 * AVRational 
> but there are problems with this so maybe the deprecated should simply
> be removed

I have a patch to remove AVFrac, see
http://git.mansr.com/?p=ffmpeg.mru;a=shortlog;h=avfrac

Unfortunately, it breaks regression tests in a few places...

-- 
M?ns Rullg?rd
mans at mansr.com