[FFmpeg-devel] [PATCH 2/2] lavfi/vf_thumbnail_cuda: fix operator precedence bug

[Philip Langdale]

On 2019-07-30 15:51, Rodger Combs wrote:
> Discovered via a warning when building with clang
> ---
>  libavfilter/vf_thumbnail_cuda.cu | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavfilter/vf_thumbnail_cuda.cu 
> b/libavfilter/vf_thumbnail_cuda.cu
> index c73e49fbc6..d4d4d791f6 100644
> --- a/libavfilter/vf_thumbnail_cuda.cu
> +++ b/libavfilter/vf_thumbnail_cuda.cu
> @@ -71,7 +71,7 @@ __global__ void
> Thumbnail_ushort2(cudaTextureObject_t ushort2_tex,
>      {
>          ushort2 pixel = tex2D<ushort2>(ushort2_tex, x, y);
>          atomicAdd(&histogram[(pixel.x + 128) >> 8], 1);
> -        atomicAdd(&histogram[256 + (pixel.y + 128) >> 8], 1);
> +        atomicAdd(&histogram[256 + ((pixel.y + 128) >> 8)], 1);
>      }
>  }

LGTM

--phil