Quoting Vittorio Giovara (2015-07-01 19:53:21)
> They are used by dnxhd and mpegvideo_enc exclusively, move them to codec
> private options instead.
> 
> Signed-off-by: Vittorio Giovara <[email protected]>
> ---
>  libavcodec/avcodec.h       | 14 ++++++--------
>  libavcodec/dnxhdenc.c      | 17 ++++++++++++-----
>  libavcodec/dnxhdenc.h      |  1 +
>  libavcodec/mpegvideo.h     |  2 ++
>  libavcodec/mpegvideo_enc.c | 10 ++++++++--
>  libavcodec/options_table.h |  2 ++
>  libavcodec/version.h       |  3 +++
>  7 files changed, 34 insertions(+), 15 deletions(-)
> 
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index 39ffb09..6cc47ea 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -1565,20 +1565,18 @@ typedef struct AVCodecContext {
>       */
>      int me_range;
>  
> +#if FF_API_QUANT_BIAS
>      /**
> -     * intra quantizer bias
> -     * - encoding: Set by user.
> -     * - decoding: unused
> +     * @deprecated use encoder private option instead
>       */
> -    int intra_quant_bias;
> +    attribute_deprecated int intra_quant_bias;
>  #define FF_DEFAULT_QUANT_BIAS 999999

Might want to keep the define internally, to avoid the weird-looking
magic number.

Also, this should go with a minor bump.

Oterwise looks ok.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to