On Fri, Apr 13, 2018 at 12:24 AM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > 2018-04-13 0:11 GMT+02:00, Alexander Kravchenko <akravchenko...@gmail.com>: >> >> >>> -----Original Message----- >>> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of >>> Carl Eugen Hoyos >>> Sent: Friday, April 13, 2018 12:48 AM >>> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> >>> Subject: Re: [FFmpeg-devel] [PATCH] lavc/amfenc: DXVA2 textures support >>> implementation by AMF encoder >>> >>> 2018-04-12 23:42 GMT+02:00, Alexander Kravchenko >>> <akravchenko...@gmail.com>: >>> > >>> > This patch contains DXVA2 textures support implementation by AMF >>> > encoder (in addition of D3D11 textures) >>> >>> > + if (frames_ctx->device_ctx->hwctx) { #if CONFIG_D3D11VA > > (There is a linebreak in your original and my mail afaict.) > >>> > + if (frames_ctx->device_ctx->type == >>> > AV_HWDEVICE_TYPE_D3D11VA) { >>> >>> if (CONFIG_D3D11VA && frames_ctx->device_ctx->type ==... >>> >>> same below. >>> >> >> Hi Carl, thanks for your feedback >> >> Could you explain the reason replacing >> if (frames_ctx->device_ctx->type == >> to >> if (CONFIG_D3D11VA && frames_ctx->device_ctx->type == > > The code gets more readable / less ugly. >
The code needs to be under an actual preprocessor check though, as the types referenced in there may not be valid when D3D11 is not available on the system. - Hendrik _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel