From: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Date: Mon, 30 Jul 2018 13:43:19 +0000 (+0200)
Subject: radv: Remove redundant format check.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=9add63a3a5600d96d96640320706dd2491b9bd64;p=mesa.git

radv: Remove redundant format check.

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>
---

diff --git a/src/amd/vulkan/vk_format_table.py b/src/amd/vulkan/vk_format_table.py
index 604aac8fa75..44c3439e9fc 100644
--- a/src/amd/vulkan/vk_format_table.py
+++ b/src/amd/vulkan/vk_format_table.py
@@ -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)