[FFmpeg-devel] [PATCH] lavc/utils: provide more feedback in case of experimental codec

[Michael Niedermayer]

On Sat, Oct 20, 2012 at 02:54:02PM +0200, Stefano Sabatini wrote:
> ---
>  libavcodec/utils.c |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> index 5e9f38e..ac43843 100644
> --- a/libavcodec/utils.c
> +++ b/libavcodec/utils.c
> @@ -888,7 +888,10 @@ int attribute_align_arg avcodec_open2(AVCodecContext *avctx, const AVCodec *code
>  
>      if (codec->capabilities & CODEC_CAP_EXPERIMENTAL)
>          if (avctx->strict_std_compliance > FF_COMPLIANCE_EXPERIMENTAL) {
> -            av_log(avctx, AV_LOG_ERROR, "Codec is experimental but experimental codecs are not enabled, try -strict -2\n");
> +            av_log(avctx, AV_LOG_ERROR,
> +                   "Codec %s is experimental but experimental codecs are not enabled: "
> +                   "try the \"strict\" option with value \"experimental\"\n",
> +                   codec->name);

do all user applications support named values ?
if not removing -2, would a problem for their users

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121020/160a7875/attachment.asc>