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 604aac8fa755ddf9da4545425e00a00a15ff5e2a..44c3439e9fc76c4be5be2674f3347f1f4a9e8046 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)