diff mbox series

[FFmpeg-devel] avutil/hwcontext_vulkan: fix format specifiers for some printed variables

Message ID 20210429163302.58691-1-jamrial@gmail.com
State Accepted
Commit ffeeff4fbccbf9c9269ac24fe56285b224426141
Headers show
Series [FFmpeg-devel] avutil/hwcontext_vulkan: fix format specifiers for some printed variables | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

James Almer April 29, 2021, 4:33 p.m. UTC 
VkPhysicalDeviceLimits.optimalBufferCopyRowPitchAlignment and
VkPhysicalDeviceExternalMemoryHostPropertiesEXT.minImportedHostPointerAlignment are of type
VkDeviceSize (a typedef uint64_t).
VkPhysicalDeviceLimits.minMemoryMapAlignment is of type size_t.

Signed-off-by: James Almer <jamrial@gmail.com>
---
 libavutil/hwcontext_vulkan.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Lynne April 29, 2021, 4:56 p.m. UTC | #1 
Apr 29, 2021, 18:33 by jamrial@gmail.com:

> VkPhysicalDeviceLimits.optimalBufferCopyRowPitchAlignment and
> VkPhysicalDeviceExternalMemoryHostPropertiesEXT.minImportedHostPointerAlignment are of type
> VkDeviceSize (a typedef uint64_t).
> VkPhysicalDeviceLimits.minMemoryMapAlignment is of type size_t.
>
> Signed-off-by: James Almer <jamrial@gmail.com>
>

LGTM
James Almer April 29, 2021, 5:04 p.m. UTC | #2 
On 4/29/2021 1:56 PM, Lynne wrote:
> Apr 29, 2021, 18:33 by jamrial@gmail.com:
> 
>> VkPhysicalDeviceLimits.optimalBufferCopyRowPitchAlignment and
>> VkPhysicalDeviceExternalMemoryHostPropertiesEXT.minImportedHostPointerAlignment are of type
>> VkDeviceSize (a typedef uint64_t).
>> VkPhysicalDeviceLimits.minMemoryMapAlignment is of type size_t.
>>
>> Signed-off-by: James Almer <jamrial@gmail.com>
>>
> 
> LGTM

Applied, thanks.
diff mbox series

Patch

diff --git a/libavutil/hwcontext_vulkan.c b/libavutil/hwcontext_vulkan.c
index 48327ad98f..444dbb5a58 100644
--- a/libavutil/hwcontext_vulkan.c
+++ b/libavutil/hwcontext_vulkan.c
@@ -1123,12 +1123,12 @@  static int vulkan_device_init(AVHWDeviceContext *ctx)
     av_log(ctx, AV_LOG_VERBOSE, "Using device: %s\n",
            p->props.properties.deviceName);
     av_log(ctx, AV_LOG_VERBOSE, "Alignments:\n");
-    av_log(ctx, AV_LOG_VERBOSE, "    optimalBufferCopyRowPitchAlignment: %li\n",
+    av_log(ctx, AV_LOG_VERBOSE, "    optimalBufferCopyRowPitchAlignment: %"PRIu64"\n",
            p->props.properties.limits.optimalBufferCopyRowPitchAlignment);
-    av_log(ctx, AV_LOG_VERBOSE, "    minMemoryMapAlignment:              %li\n",
+    av_log(ctx, AV_LOG_VERBOSE, "    minMemoryMapAlignment:              %"SIZE_SPECIFIER"\n",
            p->props.properties.limits.minMemoryMapAlignment);
     if (p->extensions & EXT_EXTERNAL_HOST_MEMORY)
-        av_log(ctx, AV_LOG_VERBOSE, "    minImportedHostPointerAlignment:    %li\n",
+        av_log(ctx, AV_LOG_VERBOSE, "    minImportedHostPointerAlignment:    %"PRIu64"\n",
                p->hprops.minImportedHostPointerAlignment);
 
     p->dev_is_nvidia = (p->props.properties.vendorID == 0x10de);