Hi,

2015-07-13 16:22 GMT+02:00 Ivan Uskov <ivan.us...@nablet.com>:
> Hello All,
>
> Current QSV engine implementation does allocate but never does release
> a display handler under linux platforms.
> The attached patch solved this issue, please review.

LGTM. However, missing newline IMHO before
ff_qsv_close_internal_session() and after QSVSession definition.

Note it would also be interesting to provide a means to import a
VADisplay from the user through an additional function. e.g. in view
to interop VA decode to MSDK encode and/or video processing.

Regards,
-- 
Gwenole Beauchesne
Intel Corporation SAS / 2 rue de Paris, 92196 Meudon Cedex, France
Registration Number (RCS): Nanterre B 302 456 199
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to