diff options
author | Caio Marcelo de Oliveira Filho <[email protected]> | 2018-07-16 13:50:07 -0700 |
---|---|---|
committer | Caio Marcelo de Oliveira Filho <[email protected]> | 2018-07-18 08:29:51 -0700 |
commit | 01d02e8906a9bc327d257c7bccba39f6af74045f (patch) | |
tree | a22981ff15c7362a16fef4bbde4b459eab10be8d | |
parent | df8f1637fa08dcf876840a315a5b8461a6ce890a (diff) |
anv: avoid warning when switching in VkStructureType
When one of the cases is not part of the enum, the compilar complains:
../../src/intel/vulkan/anv_formats.c: In function ‘anv_GetPhysicalDeviceFormatProperties2’:
../../src/intel/vulkan/anv_formats.c:728:7: warning: case value ‘1000001004’ not in enumerated type ‘VkStructureType’ {aka ‘enum VkStructureType’} [-Wswitch]
case VK_STRUCTURE_TYPE_WSI_FORMAT_MODIFIER_PROPERTIES_LIST_MESA:
^~~~
Given the switch has an "default:" case, we don't lose anything by
switching on the unsigned value to avoid the warning.
Reviewed-by: Anuj Phogat <[email protected]>
-rw-r--r-- | src/intel/vulkan/anv_formats.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/intel/vulkan/anv_formats.c b/src/intel/vulkan/anv_formats.c index 0c5b8d3a2c8..815b320a826 100644 --- a/src/intel/vulkan/anv_formats.c +++ b/src/intel/vulkan/anv_formats.c @@ -724,7 +724,8 @@ void anv_GetPhysicalDeviceFormatProperties2( &pFormatProperties->formatProperties); vk_foreach_struct(ext, pFormatProperties->pNext) { - switch (ext->sType) { + /* Use unsigned since some cases are not in the VkStructureType enum. */ + switch ((unsigned)ext->sType) { case VK_STRUCTURE_TYPE_WSI_FORMAT_MODIFIER_PROPERTIES_LIST_MESA: get_wsi_format_modifier_properties_list(physical_device, format, (void *)ext); |