[FFmpeg-devel] parameter ranges bug / minrate q

[Michael Niedermayer]

On Sat, Jul 19, 2008 at 05:59:45PM -0700, tripp wrote:
[...]
> > >  @item -qcomp @var{compression}
> > > -video quantizer scale compression (VBR)
> > > -
> > > +video quantizer scale compression (VBR) (range 0.0 - 1.0,  default 0.5)
> > 
> > not strictly true
> 
> regarding default value?

the range is wrong


> 
> > > +it's not recommended to set this value at its extremes.
> > >  @item -lmin @var{lambda}
> > >  minimum video lagrange factor (VBR)
> > >  @item -lmax @var{lambda}
> > 
> > why?
> 
> lifted straight from mencoder docs,
> where it also says qcomp 1 = CQ encoding
> qcomp 0 = CBR
> (assuming default rc_eq)
> 
> it's not recommended to set this value at its extremes
> with the dafault rc_eq?

"with the dafault rc_eq" is missing from yours at least


> 
> > [and yes you will have to split the seperate things unless
> > diego or another is in very good mood and does it ...]
> 
> meaning i have to split it into like 4 patches?
> minrate
> bitstream filter add
> issue 312 partial fix
> defaults and ranges
> 
> 
> i couldn't find patch submission guidelines.

http://ffmpeg.mplayerhq.hu/general.html

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080720/944cc564/attachment.pgp>