On Mon, Jul 13, 2015 at 11:26:41PM +0200, Gwenole Beauchesne wrote: > 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.
newlines added applied thanks > > 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 -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically faster algorithms should always be preferred if you have asymptotical amounts of data
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
