From 93b26b63811fca32b98be9e4ee7d338bd7b99fca Mon Sep 17 00:00:00 2001 From: Panagiotis Issaris Date: Thu, 28 Sep 2006 17:45:51 +0000 Subject: [PATCH] Remove the "mblmin" OptionDef. The equally named AVOption takes over its function. A unit change is involved, but as AVOption parameters are now parsed with ff_eval2(), you can use the QP2LAMBDA constant to ease the change. Originally committed as revision 6369 to svn://svn.ffmpeg.org/ffmpeg/trunk --- ffmpeg.c | 13 ------------- libavcodec/utils.c | 2 +- 2 files changed, 1 insertion(+), 14 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index 2037902..ca7b162 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -107,7 +107,6 @@ static int max_frames[4] = {INT_MAX, INT_MAX, INT_MAX, INT_MAX}; static int frame_rate = 25; static int frame_rate_base = 1; static float video_qscale = 0; -static int video_mb_lmin = 2*FF_QP2LAMBDA; static int video_mb_lmax = 31*FF_QP2LAMBDA; static int video_qdiff = 3; static uint16_t *intra_matrix = NULL; @@ -2322,16 +2321,6 @@ static void opt_qscale(const char *arg) } } -static void opt_mb_lmin(const char *arg) -{ - video_mb_lmin = atof(arg)*FF_QP2LAMBDA; - if (video_mb_lmin < 1 || - video_mb_lmin > FF_LAMBDA_MAX) { - fprintf(stderr, "mblmin must be >= 1 and <= %d\n", FF_LAMBDA_MAX / FF_QP2LAMBDA); - exit(1); - } -} - static void opt_mb_lmax(const char *arg) { video_mb_lmax = atof(arg)*FF_QP2LAMBDA; @@ -2881,7 +2870,6 @@ static void new_video_stream(AVFormatContext *oc) if(inter_matrix) video_enc->inter_matrix = inter_matrix; - video_enc->mb_lmin = video_mb_lmin; video_enc->mb_lmax = video_mb_lmax; video_enc->max_qdiff = video_qdiff; video_enc->rc_eq = video_rc_eq; @@ -3804,7 +3792,6 @@ const OptionDef options[] = { { "vn", OPT_BOOL | OPT_VIDEO, {(void*)&video_disable}, "disable video" }, { "vdt", OPT_INT | HAS_ARG | OPT_EXPERT | OPT_VIDEO, {(void*)&video_discard}, "discard threshold", "n" }, { "qscale", HAS_ARG | OPT_EXPERT | OPT_VIDEO, {(void*)opt_qscale}, "use fixed video quantiser scale (VBR)", "q" }, - { "mblmin", HAS_ARG | OPT_EXPERT | OPT_VIDEO, {(void*)opt_mb_lmin}, "min macroblock quantiser scale (VBR)", "q" }, { "mblmax", HAS_ARG | OPT_EXPERT | OPT_VIDEO, {(void*)opt_mb_lmax}, "max macroblock quantiser scale (VBR)", "q" }, { "qdiff", HAS_ARG | OPT_EXPERT | OPT_VIDEO, {(void*)opt_qdiff}, "max difference between the quantiser scale (VBR)", "q" }, { "rc_eq", HAS_ARG | OPT_EXPERT | OPT_VIDEO, {(void*)opt_video_rc_eq}, "set rate control equation", "equation" }, diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 3a929e2..8fa729f 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -677,7 +677,7 @@ static const AVOption options[]={ {"skip_exp", "frame skip exponent", OFFSET(frame_skip_exp), FF_OPT_TYPE_INT, DEFAULT, INT_MIN, INT_MAX, V|E}, {"skipcmp", "frame skip compare function", OFFSET(frame_skip_cmp), FF_OPT_TYPE_INT, FF_CMP_DCTMAX, INT_MIN, INT_MAX, V|E, "cmp_func"}, {"border_mask", NULL, OFFSET(border_masking), FF_OPT_TYPE_FLOAT, DEFAULT, -FLT_MAX, FLT_MAX, V|E}, -{"mblmin", "min macroblock quantiser scale (VBR)", OFFSET(mb_lmin), FF_OPT_TYPE_INT, FF_QP2LAMBDA * 2, INT_MIN, INT_MAX, V|E}, +{"mblmin", "min macroblock quantiser scale (VBR)", OFFSET(mb_lmin), FF_OPT_TYPE_INT, FF_QP2LAMBDA * 2, 1, FF_LAMBDA_MAX, V|E}, {"mblmax", "max macroblock quantiser scale (VBR)", OFFSET(mb_lmax), FF_OPT_TYPE_INT, FF_QP2LAMBDA * 31, INT_MIN, INT_MAX, V|E}, {"mepc", "motion estimation bitrate penalty compensation (1.0 = 256)", OFFSET(me_penalty_compensation), FF_OPT_TYPE_INT, 256, INT_MIN, INT_MAX, V|E}, {"bidir_refine", NULL, OFFSET(bidir_refine), FF_OPT_TYPE_INT, DEFAULT, 0, 4, V|E}, -- 2.7.4