[FFmpeg-devel] [PATCH v3 1/4] lavu/hwcontext_opecl: fix the build warning
Mark Thompson
sw at jkqxz.net
Sun Jun 3 20:42:24 EEST 2018
On 03/06/18 04:15, Jun Zhao wrote:
> fix the build warning when use Portable Computing Language (pocl).
>
> Signed-off-by: Jun Zhao <mypopydev at gmail.com>
> ---
> libavutil/hwcontext_opencl.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/libavutil/hwcontext_opencl.c b/libavutil/hwcontext_opencl.c
> index 43b5c5a..9ecc361 100644
> --- a/libavutil/hwcontext_opencl.c
> +++ b/libavutil/hwcontext_opencl.c
> @@ -2809,7 +2809,11 @@ static int opencl_map_from(AVHWFramesContext *hwfc, AVFrame *dst,
> static int opencl_map_to(AVHWFramesContext *hwfc, AVFrame *dst,
> const AVFrame *src, int flags)
> {
> +#if HAVE_OPENCL_DRM_BEIGNET || HAVE_OPENCL_VAAPI_BEIGNET || \
> + HAVE_OPENCL_VAAPI_INTEL_MEDIA || HAVE_OPENCL_DXVA2 || \
> + HAVE_OPENCL_D3D11 || HAVE_OPENCL_DRM_ARM
> OpenCLDeviceContext *priv = hwfc->device_ctx->internal->priv;
> +#endif
> av_assert0(dst->format == AV_PIX_FMT_OPENCL);
> switch (src->format) {
> #if HAVE_OPENCL_DRM_BEIGNET
> @@ -2850,7 +2854,11 @@ static int opencl_map_to(AVHWFramesContext *hwfc, AVFrame *dst,
> static int opencl_frames_derive_to(AVHWFramesContext *dst_fc,
> AVHWFramesContext *src_fc, int flags)
> {
> +#if HAVE_OPENCL_DRM_BEIGNET || HAVE_OPENCL_VAAPI_BEIGNET || \
> + HAVE_OPENCL_VAAPI_INTEL_MEDIA || HAVE_OPENCL_DXVA2 || \
> + HAVE_OPENCL_D3D11 || HAVE_OPENCL_DRM_ARM
> OpenCLDeviceContext *priv = dst_fc->device_ctx->internal->priv;
> +#endif
> switch (src_fc->device_ctx->type) {
> #if HAVE_OPENCL_DRM_BEIGNET
> case AV_HWDEVICE_TYPE_DRM:
>
The condition is horrible and would need to be changed on any addition. Just mark the variables unused instead.
- Mark
More information about the ffmpeg-devel
mailing list