[FFmpeg-devel] [PATCH 1/5] avformat: add codec_time_base

Gonzalo Garramuno ggarra13 at gmail.com
Wed Jun 18 23:33:35 CEST 2014


On 18/06/14 17:53, Michael Niedermayer wrote:
> On Wed, Jun 18, 2014 at 02:37:02PM -0500, Hendrik Leppkes wrote:
>> On Wed, Jun 18, 2014 at 2:35 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>>> This is unambigously the codec timebase, while AVStream.time_base
>>> changes its meaning when avpriv_set_pts_info() is called
>>>
>>> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
>> And why wouldn't I simply access st->codec->time_base instead of
>> st->codec_time_base?
>> Seems a bit redundant, doesn't it?
> absolutely yes, accessing it directly is the best solution
> but libav moves away from this it seems, see
FWIW, I second the motion to remain with st->codec->time_base.  This 
seems an API change with no benefits.
Why does the new API reads only one time base?



More information about the ffmpeg-devel mailing list