Henri Verbeet : d3d10: Implement d3d10_effect_parse() on top of vkd3d_shader_parse_dxbc().

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 93cc58d12190c4536db34401dee8f157883a6d99
URL:    https://gitlab.winehq.org/wine/wine/-/commit/93cc58d12190c4536db34401dee8f15...

Author: Henri Verbeet <hverbeet(a)codeweavers.com>
Date:   Mon Apr 10 09:33:44 2023 +0200

d3d10: Implement d3d10_effect_parse() on top of vkd3d_shader_parse_dxbc().

---

 dlls/d3d10/effect.c | 45 ++++++++++++++++++++++++++++++---------------
 1 file changed, 30 insertions(+), 15 deletions(-)

diff --git a/dlls/d3d10/effect.c b/dlls/d3d10/effect.c
index 61c76416273..1d2b46523ad 100644
--- a/dlls/d3d10/effect.c
+++ b/dlls/d3d10/effect.c
@@ -19,6 +19,7 @@
  */
 
 #include "d3d10_private.h"
+#include <vkd3d_shader.h>
 
 #include <float.h>
 #include <stdint.h>
@@ -4056,28 +4057,42 @@ static HRESULT parse_fx10(struct d3d10_effect *e, const char *data, size_t data_
     return parse_fx10_body(e, ptr, data_size - (ptr - data));
 }
 
-static HRESULT fx10_chunk_handler(const char *data, size_t data_size, uint32_t tag, void *ctx)
+HRESULT d3d10_effect_parse(struct d3d10_effect *effect, const void *data, SIZE_T data_size)
 {
-    struct d3d10_effect *e = ctx;
-
-    TRACE("tag: %s.\n", debugstr_an((const char *)&tag, 4));
+    const struct vkd3d_shader_code dxbc = {.code = data, .size = data_size};
+    const struct vkd3d_shader_dxbc_section_desc *section;
+    struct vkd3d_shader_dxbc_desc dxbc_desc;
+    HRESULT hr = S_OK;
+    unsigned int i;
+    int ret;
 
-    TRACE("Chunk size: %#Ix.\n", data_size);
+    if ((ret = vkd3d_shader_parse_dxbc(&dxbc, 0, &dxbc_desc, NULL)) < 0)
+    {
+        WARN("Failed to parse DXBC, ret %d.\n", ret);
+        return E_FAIL;
+    }
 
-    switch(tag)
+    for (i = 0; i < dxbc_desc.section_count; ++i)
     {
-        case TAG_FX10:
-            return parse_fx10(e, data, data_size);
+        section = &dxbc_desc.sections[i];
 
-        default:
-            FIXME("Unhandled chunk %s.\n", debugstr_an((const char *)&tag, 4));
-            return S_OK;
+        TRACE("Section %u: tag %s, data {%p, %#Ix}.\n",
+                i, debugstr_an((const char *)§ion->tag, 4),
+                section->data.code, section->data.size);
+
+        if (section->tag != TAG_FX10)
+        {
+            FIXME("Unhandled chunk %s.\n", debugstr_an((const char *)§ion->tag, 4));
+            continue;
+        }
+
+        if (FAILED(hr = parse_fx10(effect, section->data.code, section->data.size)))
+            break;
     }
-}
 
-HRESULT d3d10_effect_parse(struct d3d10_effect *effect, const void *data, SIZE_T data_size)
-{
-    return parse_dxbc(data, data_size, fx10_chunk_handler, effect);
+    vkd3d_shader_free_dxbc(&dxbc_desc);
+
+    return hr;
 }
 
 static void d3d10_effect_shader_variable_destroy(struct d3d10_effect_shader_variable *s,