radv: Remove redundant format check.
authorBas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Mon, 30 Jul 2018 13:43:19 +0000 (15:43 +0200)
committerBas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Mon, 17 Dec 2018 20:09:38 +0000 (20:09 +0000)
The switch directly after the check has a default case that returns
NULL too, so the effective return value is not changed. Also this
check is wrong once we start dealing with formats introduced by an
extension (e.g. YUV formats).

Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
src/amd/vulkan/vk_format_table.py

index 604aac8..44c3439 100644 (file)
@@ -146,10 +146,6 @@ def write_format_table(formats):
     print("const struct vk_format_description *")
     print("vk_format_description(VkFormat format)")
     print("{")
-    print("   if (format > VK_FORMAT_END_RANGE) {")
-    print("      return NULL;")
-    print("   }")
-    print()
     print("   switch (format) {")
     for format in formats:
         print("   case %s:" % format.name)