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

[Stefano Sabatini]

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.
-- 
FFmpeg = Formidable & Furious Marvellous Problematic Elastic Glue
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-doc-ffmpeg-update-and-extend-documentation-for-copyt.patch
Type: text/x-diff
Size: 2716 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120325/f7be617d/attachment.bin>