[FFmpeg-devel] [PATCH 4/5] lavfi: add ProcAmp(color balance) vaapi video filter.

Carl Eugen Hoyos ceffmpeg at gmail.com
Mon Jan 15 01:17:42 EET 2018


2018-01-15 0:14 GMT+01:00 Mark Thompson <sw at jkqxz.net>:
> On 14/01/18 22:59, Carl Eugen Hoyos wrote:
>>> On 08/01/18 08:35, Jun Zhao wrote:
>>
>>>> +    VAStatus vas;
>>
>>>> +        av_log(avctx, AV_LOG_ERROR, "Failed to Query procamp "
>>>> +               "query caps: %d (%s).\n", vas, vaErrorStr(vas));
>>
>> Is "%d" what it's supposed to be? Why is VAStatus needed then?
>
> Yes: <https://github.com/01org/libva/blob/master/va/va.h#L170>.

Ok, I expected this.
But my question was actually:
If there is a typedef for a return value that you may be interested
in printing, shouldn't there be a define that is the correct format?

Carl Eugen


More information about the ffmpeg-devel mailing list