On Wed, Feb 03, 2016 at 05:49:56PM +0100, Vittorio Giovara wrote: > ffmpeg | branch: master | Vittorio Giovara <[email protected]> | Mon > Nov 9 03:15:06 2015 +0100| [2862b63783b5556f7f3fb2d097629bc6879f833a] | > committer: Vittorio Giovara > > lavc: Move prediction_method to codec private options > > This options is only used by huffyuv, ffvhuv, jpegls, mjpeg, > mpegvideoenc, png, utvideo. > It is a very codec-specific options, so deprecate the global variant. > Set proper limits to the maximum allowed values, and update utvideoenc > tests to use the new option name. > > Signed-off-by: Vittorio Giovara <[email protected]> > > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=2862b63783b5556f7f3fb2d097629bc6879f833a [...]
> @@ -631,12 +638,32 @@ FF_ENABLE_DEPRECATION_WARNINGS
> return 0;
> }
>
> +#define OFFSET(x) offsetof(UtvideoContext, x)
> +#define VE AV_OPT_FLAG_VIDEO_PARAM | AV_OPT_FLAG_ENCODING_PARAM
> +static const AVOption options[] = {
> +{ "pred", "Prediction method", OFFSET(frame_pred), AV_OPT_TYPE_INT, { .i64 =
> PRED_LEFT }, PRED_NONE, PRED_MEDIAN, VE, "pred" },
> + { "none", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = PRED_NONE }, INT_MIN,
> INT_MAX, VE, "pred" },
> + { "left", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = PRED_LEFT }, INT_MIN,
> INT_MAX, VE, "pred" },
> + { "gradient", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = PRED_GRADIENT },
> INT_MIN, INT_MAX, VE, "pred" },
> + { "median", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = PRED_MEDIAN },
> INT_MIN, INT_MAX, VE, "pred" },
> +
> + { NULL},
> +};
this breaks existing command line usage like -pred 3
and "3" previously lacked a named constant so people had to use a
litteral 3 i think
this change should be documented if its left that way
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
