[FFmpeg-devel] [PATCH] ffprobe: print start_pts and duration_pts stream information

[Clément Bœsch]

On Tue, Sep 11, 2012 at 11:01:03AM +0200, Stefano Sabatini wrote:
> On date Saturday 2012-09-08 17:34:52 +0200, Michael Niedermayer encoded:
> > On Sat, Sep 08, 2012 at 01:02:25PM +0200, Stefano Sabatini wrote:
> > > Add start_pts and duration_pts fields. A duration_time field is added for
> > > overall consistency ("duration" alone does not tell if the duration is a
> > > time or a timestamp), "duration" is kept for backward compatibility but
> > > should be considered deprecated.
> > > ---
> > >  doc/ffprobe.xsd |    3 +++
> > >  ffprobe.c       |    6 +++++-
> > >  2 files changed, 8 insertions(+), 1 deletions(-)
> > 
> > good idea ...
> 
> Updated, "duration_pts" renamed to "duration_ts" as it makes slightly
> more sense.
> 
> Will apply soon if I read no comments.
> -- 
> FFmpeg = Fantastic Faithless Maxi Philosophical Enlightened Guide

> From 68302f62a774909b25d31c6476442211f2469605 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefasab at gmail.com>
> Date: Sat, 8 Sep 2012 13:00:13 +0200
> Subject: [PATCH] ffprobe: print start_pts and duration_ts stream information
> 
> Add start_pts and duration_ts fields. The old "duration" field is kept
> for backward compatibility, but is deprecated in favor of
> "duration_time".

Why duration wouldn't be expressed in time by default? I mean what's the
purpose? I don't see much point in deprecating something we will likely
never remove (for various reasons you are aware of) and is now duplicated.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120911/bd1bc22b/attachment.asc>