Ping
On Fri, Jul 8, 2022 at 8:57 PM Amir Mazzarella <amirm...@google.com> wrote:
> FFVulkanExtensions enum does not have a value for 0 defined, and bitwise
> OR on enums is not safe. The function returns uint64_t, so it makes more
> sense and is safer to do arithmetic in terms of uint64_t
>
> Signed-off-by: Amir Mazzarella <amirm...@google.com>
> ---
> libavutil/vulkan_loader.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavutil/vulkan_loader.h b/libavutil/vulkan_loader.h
> index 3f1ee6aa46..0b2de4ab94 100644
> --- a/libavutil/vulkan_loader.h
> +++ b/libavutil/vulkan_loader.h
> @@ -50,12 +50,12 @@ static inline uint64_t ff_vk_extensions_to_mask(const
> char * const *extensions,
> #endif
> };
>
> - FFVulkanExtensions mask = 0x0;
> + uint64_t mask = 0x0;
>
> for (int i = 0; i < nb_extensions; i++) {
> for (int j = 0; j < FF_ARRAY_ELEMS(extension_map); j++) {
> if (!strcmp(extensions[i], extension_map[j].name)) {
> - mask |= extension_map[j].flag;
> + mask |= (uint64_t) extension_map[j].flag;
> continue;
> }
> }
> --
> 2.37.0.rc0.161.g10f37bed90-goog
>
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
