[FFmpeg-devel] [PATCH] libavcodec/qsv.c: Issue fixed: QSV engine does not release display handler under linux platform.

[Gwenole Beauchesne]

Hi,

2015-07-13 16:22 GMT+02:00 Ivan Uskov <ivan.uskov at 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