[FFmpeg-devel] [PATCH] ffprobe: show codec_time_base in streams section

[Stefano Sabatini]

On date Friday 2012-09-14 02:32:00 +0200, Clément Bœsch encoded:
> On Thu, Sep 13, 2012 at 11:36:59AM +0200, Stefano Sabatini wrote:
> > The field is useful to compute times from values expressed in codec time
> > base units, for example the duration field in the "frame" section.
> > ---
> >  doc/ffprobe.xsd |    1 +
> >  ffprobe.c       |    1 +
> >  2 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/doc/ffprobe.xsd b/doc/ffprobe.xsd
> > index 9549bdb..6ea27a5 100644
> > --- a/doc/ffprobe.xsd
> > +++ b/doc/ffprobe.xsd
> > @@ -123,6 +123,7 @@
> >        <xsd:attribute name="r_frame_rate"     type="xsd:string" use="required"/>
> >        <xsd:attribute name="avg_frame_rate"   type="xsd:string" use="required"/>
> >        <xsd:attribute name="time_base"        type="xsd:string" use="required"/>
> > +      <xsd:attribute name="codec_time_base"  type="xsd:string" use="required"/>
> >        <xsd:attribute name="start_pts"        type="xsd:long"/>
> >        <xsd:attribute name="start_time"       type="xsd:float"/>
> >        <xsd:attribute name="duration_ts"      type="xsd:long"/>
> > diff --git a/ffprobe.c b/ffprobe.c
> > index a770a25..6b7e3ff 100644
> > --- a/ffprobe.c
> > +++ b/ffprobe.c
> > @@ -1717,6 +1717,7 @@ static void show_stream(WriterContext *w, AVFormatContext *fmt_ctx, int stream_i
> >      print_q("r_frame_rate",   stream->r_frame_rate,   '/');
> >      print_q("avg_frame_rate", stream->avg_frame_rate, '/');
> >      print_q("time_base",      stream->time_base,      '/');
> > +    print_q("codec_time_base", stream->codec->time_base, '/');
> >      print_ts  ("start_pts",   stream->start_time);
> >      print_time("start_time",  stream->start_time, &stream->time_base);
> >      print_ts  ("duration_ts", stream->duration);
> 
> This field seems already printed, why the dup?

codec time-base != stream time-base
-- 
FFmpeg = Formidable Fierce Magnificient Porno Elegant Gangster