]> granicus.if.org Git - libx264/commitdiff
Get rid of "CBR" descriptor from qcomp
authorFiona Glaser <fiona@x264.com>
Tue, 1 Sep 2009 05:44:45 +0000 (22:44 -0700)
committerFiona Glaser <fiona@x264.com>
Tue, 1 Sep 2009 05:44:45 +0000 (22:44 -0700)
Though technically accurate in some vague way, I have never actually seen this
option used correctly, rather it has been used by hundreds of people who can't
read the documentation and believe that qcomp=0 is what should be used for CBR
encoding.

x264.c

diff --git a/x264.c b/x264.c
index b7083c0293d44e98c372fd90bbaf2d118a4c21a4..76e53072d231c3862b267fa38cd48ceaf3cabc8a 100644 (file)
--- a/x264.c
+++ b/x264.c
@@ -233,7 +233,7 @@ static void Help( x264_param_t *defaults, int b_longhelp )
         "                                  - 3: Nth pass, overwrites stats file\n" );
     H0( "      --stats <string>        Filename for 2 pass stats [\"%s\"]\n", defaults->rc.psz_stat_out );
     H0( "      --no-mbtree             Disable mb-tree ratecontrol.\n");
-    H0( "      --qcomp <float>         QP curve compression: 0.0 => CBR, 1.0 => CQP [%.2f]\n", defaults->rc.f_qcompress );
+    H1( "      --qcomp <float>         QP curve compression [%.2f]\n", defaults->rc.f_qcompress );
     H1( "      --cplxblur <float>      Reduce fluctuations in QP (before curve compression) [%.1f]\n", defaults->rc.f_complexity_blur );
     H1( "      --qblur <float>         Reduce fluctuations in QP (after curve compression) [%.1f]\n", defaults->rc.f_qblur );
     H0( "      --zones <zone0>/<zone1>/...  Tweak the bitrate of some regions of the video\n" );