> Am 11.11.2019 um 04:42 schrieb Andriy Gelman <andriy.gel...@gmail.com>:
> 
> +    /* check configured pix_fmt matches avctx->pix_fmt */

Is this comment really useful?

> +    pix_fmt_output = ff_v4l2_format_v4l2_to_avfmt(v4l2_fmt_output, 
> AV_CODEC_ID_RAWVIDEO);
> +    if (pix_fmt_output != avctx->pix_fmt) {
> +        const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(pix_fmt_output);
> +        av_log(priv, AV_LOG_ERROR, "Pixel format not configured. Encoder 
> requires %s "


> +                                   " pixel format. On command line add 
> setting:"
> +                                   " -pix_fmt %s to force correct pixel 
> format.\n",

This should be rarely printed by a library, I don’t think this is a good 
exception.

Thank you for looking into this issue, Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to