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

[Rodney Baker]

On Sun, 25 Mar 2012 20:26:12 Stefano Sabatini wrote:
> On date Sunday 2012-03-25 02:04:48 +1030, Rodney Baker encoded:
> > On Sun, 25 Mar 2012 01:52:28 Stefano Sabatini wrote:
> [...]
> 
> > > +If positive, the output time base is copied unchanged from the
> > > +corresponding input stream. A positive value is sometimes required for
> > > +avoiding non monotonically increasing timestamps when copying video
> > > +streams with variable frame rate.
> > > +
> > > +If zero, changes the time base as required in case the output supports
> > > +variable frame rate.
> > > +
> > > +If negative, changes the time base as with @code{-copytb 0} but only
> > > +when it seems required for generating sane output files.
> > > +
> > > +Default value is -1.
> > > +
> > 
> > So, what are the legal values? -1,0,1? Does specifying a positive integer
> > >1 cause any different behaviour than 1? If not, why not specify "if 1"
> > rather than "if positive" to avoid ambiguity?
> > 
> > As a user who is not a developer or an expert, one thing I find very
> > valuable in documentation is a clear articulation of what are acceptable
> > values or ranges of values when required for specific parameters and
> > just what they mean.
> 
> Updated taking into consideration your feedback, thanks.

Much clearer - thankyou. 

-- 
==========================================================================
Rodney Baker VK5ZTV
rodney.baker at iinet.net.au
==========================================================================