[PATCH 0/3] MR591: vkd3d-shader/dxil: Do not use ascii_strcasecmp().

[PATCH 1/3] vkd3d-shader/dxil: Do not use ascii_strcasecmp() on the entry point.

[Conor McCarthy]

From: Conor McCarthy

Function names are case-sensitive.
---
 libs/vkd3d-shader/dxil.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libs/vkd3d-shader/dxil.c b/libs/vkd3d-shader/dxil.c
index 15cc380f5..1e0a1e682 100644
--- a/libs/vkd3d-shader/dxil.c
+++ b/libs/vkd3d-shader/dxil.c
@@ -6726,7 +6726,7 @@ static enum vkd3d_result sm6_parser_entry_point_init(struct sm6_parser *sm6)

sm6->entry_point = value->u.function.name;
     if (!sm6_metadata_value_is_string(entry_node->operands[1])
-            || ascii_strcasecmp(sm6->entry_point, entry_node->operands[1]->u.string_value))
+            || strcmp(sm6->entry_point, entry_node->operands[1]->u.string_value))
     {
         WARN("Entry point function name %s mismatch.\n", sm6->entry_point);
         vkd3d_shader_parser_warning(&sm6->p, VKD3D_SHADER_WARNING_DXIL_ENTRY_POINT_MISMATCH,

[PATCH 2/3] vkd3d-shader/dxil: Do not use ascii_strcasecmp() to find functions.

[Conor McCarthy]

From: Conor McCarthy

Function names are case-sensitive.
---
 libs/vkd3d-shader/dxil.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libs/vkd3d-shader/dxil.c b/libs/vkd3d-shader/dxil.c
index 1e0a1e682..77d092510 100644
--- a/libs/vkd3d-shader/dxil.c
+++ b/libs/vkd3d-shader/dxil.c
@@ -6916,7 +6916,7 @@ static struct sm6_function *sm6_parser_get_function(const struct sm6_parser *sm6
 {
     size_t i;
     for (i = 0; i < sm6->function_count; ++i)
-        if (!ascii_strcasecmp(sm6->functions[i].declaration->u.function.name, name))
+        if (!strcmp(sm6->functions[i].declaration->u.function.name, name))
             return &sm6->functions[i];
     return NULL;
 }

[PATCH 3/3] vkd3d-shader/dxil: Do not use ascii_strcasecmp() to find the handle type.

[Conor McCarthy]

From: Conor McCarthy

We use strcmp() on the same type name elsewhere.
---
 libs/vkd3d-shader/dxil.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libs/vkd3d-shader/dxil.c b/libs/vkd3d-shader/dxil.c
index 77d092510..b6a9d49e2 100644
--- a/libs/vkd3d-shader/dxil.c
+++ b/libs/vkd3d-shader/dxil.c
@@ -1628,7 +1628,7 @@ static enum vkd3d_result sm6_parser_type_table_init(struct sm6_parser *sm6)
                     break;
                 }

-                if (!ascii_strcasecmp(struct_name, "dx.types.Handle"))
+                if (!strcmp(struct_name, "dx.types.Handle"))
                     sm6->handle_type = type;

type->u.struc->name = struct_name;

[PATCH 0/3] MR591: vkd3d-shader/dxil: Do not use ascii_strcasecmp(). - approved

[Giovanni Mascellani (＠giomasce)]

This merge request was approved by Giovanni Mascellani.

[Giovanni Mascellani (＠giomasce)]

I'd just recommend stressing more what you positively do in the commit, rather than what you don't do, in the commit subject. For instance, "Compare {entry point, function, handle} names case-sensitively".