[FFmpeg-devel] [PATCH v5] Add experimental support for vp9 in iso-bmff

KongQun Yang kqyang at google.com
Wed Jun 15 03:25:34 CEST 2016


-- KongQun Yang (KQ)

On Tue, Jun 14, 2016 at 4:20 PM, Ronald S. Bultje <rsbultje at gmail.com>
wrote:

> Hi,
>
> On Tue, Jun 14, 2016 at 6:05 PM, Kongqun Yang <yangkongqun at gmail.com>
> wrote:
>
>> +    default:
>> +        av_log(NULL, AV_LOG_ERROR, "Unsupported color space (%d)\n",
>> +               color_space);
>> +        return -1;
>>
> [..]
>
>> +    default:
>> +        av_log(NULL, AV_LOG_ERROR, "Unsupported pixel format (%d)\n",
>> +               pixel_format);
>> +        return -1;
>>
> [..]
>
>> +    if (desc == NULL) {
>> +        av_log(NULL, AV_LOG_ERROR, "Unsupported pixel format (%d)\n",
>> +               pixel_format);
>> +        return -1;
>>
>
> You're still logging without a context (first argument), can you please
> provide one so people know which muxer is complaining about these error
> messages?
>

Are you ok with using "AVIOContext" as the context?

>
> Ronald
>


More information about the ffmpeg-devel mailing list