On Thu, Aug 25, 2016 at 03:16:19PM +0200, Luca Barbato wrote: > 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?
Translate between vpx/aom and libav pixel formats of course. In any case I have another patch pending. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
