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

[Stefano Sabatini]

On date Sunday 2012-03-25 21:48:03 +1030, Rodney Baker encoded:
> 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. 

Maybe Michael may want to comment on it, otherwise I'll push it in a
few days.
-- 
FFmpeg = Frightening and Fundamental Maxi Peaceless Easy Gorilla