[FFmpeg-cvslog] lavc: Move prediction_method to codec private options

Michael Niedermayer michael at niedermayer.cc
Thu Feb 4 02:56:32 CET 2016


On Wed, Feb 03, 2016 at 05:49:56PM +0100, Vittorio Giovara wrote:
> ffmpeg | branch: master | Vittorio Giovara <vittorio.giovara at gmail.com> | 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 <vittorio.giovara at gmail.com>
> 
> > 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20160204/8efa788c/attachment.sig>


More information about the ffmpeg-cvslog mailing list