[FFmpeg-devel] [PATCH] doc/ffmpeg: update and extend documentation for -copytb

[Michael Niedermayer]

On Tue, Mar 27, 2012 at 08:25:30PM +0200, Stefano Sabatini wrote:
> On date Tuesday 2012-03-27 03:00:27 +0200, Michael Niedermayer encoded:
> > On Sun, Mar 25, 2012 at 11:56:12AM +0200, Stefano Sabatini wrote:
> [...]
> > > Subject: [PATCH] doc/ffmpeg: update and extend documentation for -copytb
> > > 
> > > Address trac ticket #1120.
> > > ---
> > >  doc/ffmpeg.texi |   24 ++++++++++++++++++++++--
> > >  ffmpeg.c        |    2 +-
> > >  2 files changed, 23 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi
> > > index 402a4ae..b32f735 100644
> > > --- a/doc/ffmpeg.texi
> > > +++ b/doc/ffmpeg.texi
> > > @@ -765,8 +765,28 @@ the parameter is the maximum samples per second by which the audio is changed.
> > >  without any later correction.
> > >  @item -copyts
> > >  Copy timestamps from input to output.
> > > - at item -copytb
> > > -Copy input stream time base from input to output when stream copying.
> > > + at item -copytb @var{mode}
> > > +Specify how to set the timebase when stream copying. @var{mode} is an
> > > +integer numeric value, and it can assume one of the following values:
> > > +
> > > + at table @option
> > > + at item 1
> > > +The output time base is copied unchanged from the corresponding input
> > > +stream. This is sometimes required to avoid non monotonically
> > > +increasing timestamps when copying video streams with variable frame
> > > +rate.
> > > +
> > > + at item 0
> > > +The output time base is changed as required in case the output
> > > +supports variable frame rate.
> > > +
> > > + at item -1
> > > +The output time base is changed as with @code{-copytb 0}, but only when
> > > +it seems required to generate sane output files.
> > > + at end table
> > > +
> > > +Default value is -1.
> > 
> > The idea behind the values is:
> > 
> > 1
> > use the demuxer timbase
> > 
> > 0
> > use the codec timebase
> > 
> > -1
> > try to make the choice automatically
> > 
> > your description describes a orthogonal aspect, maybe they can be
> > combined ?
> 
> Updated.
> -- 
> FFmpeg = Fast Faithless Merciless Perennial Enlightening Game

>  doc/ffmpeg.texi |   29 +++++++++++++++++++++++++++--
>  ffmpeg.c        |    2 +-
>  2 files changed, 28 insertions(+), 3 deletions(-)
> e03902ec9462277bf5122e7ffa9fc0121fe3e7cb  0001-doc-ffmpeg-update-and-extend-documentation-for-copyt.patch
> From d651c600bae3a8bcf0e72d31a545507dc0448fc4 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefasab at gmail.com>
> Date: Sat, 24 Mar 2012 16:11:46 +0100
> Subject: [PATCH] doc/ffmpeg: update and extend documentation for -copytb
> 
> Address trac ticket #1120.

LGTM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120327/19bdbb25/attachment.asc>