Hello Derek,
Thanks for review, Do i need to send a new patch or this patch itself can
move into ffmpeg repo?
Regards
Gopu
On Sat, May 9, 2015 at 12:38 AM, Derek Buitenhuis <
derek.buitenh...@gmail.com> wrote:
> On 5/7/2015 10:17 AM, Gopu Govindaswamy wrote:
> > Hello,
> >
> > this patch is ffmpeg can now use the x265 multi-library interface to
> make a
> > run-time selection between a number of libx265 libraries (perhaps 8bpp
> and
> > 16bpp).
> >
> > I have attached the patch with this mail,
>
> [...]
>
> > - x265_picture x265pic_out = { 0 };
> > + x265_picture x265pic_out = { { 0 } };
>
> Unrelated change.
>
> > - ret = x265_encoder_encode(ctx->encoder, &nal, &nnal,
> > + ret = ctx->api->encoder_encode(ctx->encoder, &nal, &nnal,
> > pic ? &x265pic : NULL, &x265pic_out);
>
> Line needs re-indent.
>
> Other than that, I think it can be applied.
>
> - Derek
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
--
Thanks & Regards
Gopu G
Multicoreware Inc
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
