On 25/08/16 10:45, Diego Biurrun wrote:
> On Wed, Aug 24, 2016 at 08:01:52PM +0200, Luca Barbato wrote:
>> On 24/08/16 18:28, Diego Biurrun wrote:
>>> --- a/libavcodec/libvpxenc.c
>>> +++ b/libavcodec/libvpxenc.c
>>> @@ -375,7 +375,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
>>>      codecctl_int(avctx, VP8E_SET_CQ_LEVEL,          ctx->crf);
>>>  
>>>      //provide dummy value to initialize wrapper, values will be updated 
>>> each _encode()
>>> -    vpx_img_wrap(&ctx->rawimg, ff_vpx_pixfmt_to_imgfmt(avctx->pix_fmt),
>>> +    vpx_img_wrap(&ctx->rawimg, ff_aom_pixfmt_to_imgfmt(avctx->pix_fmt),
>>>                   avctx->width, avctx->height, 1, (unsigned char *)1);
>>
>> Looks wrong to me, and I'm not sure it simplifies a lot.
> 
> I can call it ff_aom_vpx_pixfmt_to_imgfmt(), or anything else really, if
> you prefer.
> 

If you have both vpx and aom enabled what that function is supposed to do?

lu

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to