Oh boy, here we go.

On Wed, 2012-10-24 at 12:10 +0200, Kostya Shishkov wrote:
> @@ -1605,6 +1606,12 @@ static int transcode_init(void)
>                      codec->codec_tag = icodec->codec_tag;
>              }
>  
> +            desc = avcodec_descriptor_get(codec->codec_id);
> +            if (desc && (desc->props & AV_CODEC_PROP_IDIOTIC)) {

AV_CODEC_PROP_IDIOTIC? How professional.

> +                av_log(NULL, AV_LOG_FATAL, "You are trying to copy stream in 
> idiotic format, don't do that!\n");

What is the motivation behind this? To eventually remove support for
remuxing such formats? To tell users not to file tickets for code that
no one wants to touch? I'm curious.

> +                // return AVERROR(EINVAL);

Useless.

> @@ -1810,6 +1818,13 @@ static int transcode_init(void)
>                          ost->file_index, ost->index);
>                  goto dump_format;
>              }
> +
> +            desc = avcodec_descriptor_get(ost->st->codec->codec_id);
> +            if (desc && (desc->props & AV_CODEC_PROP_IDIOTIC)) {
> +                av_log(NULL, AV_LOG_FATAL, "You are trying to encode stream 
> in idiotic format, don't do that!\n");

If so, then why not remove those encoders?

/Tomas

_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to