September 2021 - Wine-commits

Zebediah Figura : d3dcompiler: Handle D3D_COMPILE_STANDARD_FILE_INCLUDE in preprocess_shader().
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: 72d6654c7327b544b7ed8d5c4957882cc73ee44b URL: https://source.winehq.org/git/wine.git/?a=commit;h=72d6654c7327b544b7ed8d5c… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Sep 1 12:12:51 2021 -0500 d3dcompiler: Handle D3D_COMPILE_STANDARD_FILE_INCLUDE in preprocess_shader(). Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dcompiler_43/compiler.c | 164 ++++++++++++++++++++--------------------- 1 file changed, 82 insertions(+), 82 deletions(-) diff --git a/dlls/d3dcompiler_43/compiler.c b/dlls/d3dcompiler_43/compiler.c index 4fae71d4196..d4523edb99c 100644 --- a/dlls/d3dcompiler_43/compiler.c +++ b/dlls/d3dcompiler_43/compiler.c @@ -481,13 +481,95 @@ int wpp_parse( const char *input, FILE *output ) return ret; } +static HRESULT WINAPI d3dcompiler_include_from_file_open(ID3DInclude *iface, D3D_INCLUDE_TYPE include_type, + const char *filename, const void *parent_data, const void **data, UINT *bytes) +{ + char *fullpath, *buffer = NULL, current_dir[MAX_PATH + 1]; + const char *initial_dir; + SIZE_T size; + HANDLE file; + ULONG read; + DWORD len; + + if ((initial_dir = strrchr(initial_filename, '\\'))) + { + len = initial_dir - initial_filename + 1; + initial_dir = initial_filename; + } + else + { + len = GetCurrentDirectoryA(MAX_PATH, current_dir); + current_dir[len] = '\\'; + len++; + initial_dir = current_dir; + } + fullpath = heap_alloc(len + strlen(filename) + 1); + if (!fullpath) + return E_OUTOFMEMORY; + memcpy(fullpath, initial_dir, len); + strcpy(fullpath + len, filename); + + file = CreateFileA(fullpath, GENERIC_READ, FILE_SHARE_READ, 0, OPEN_EXISTING, 0, 0); + if (file == INVALID_HANDLE_VALUE) + goto error; + + TRACE("Include file found at %s.\n", debugstr_a(fullpath)); + + size = GetFileSize(file, NULL); + if (size == INVALID_FILE_SIZE) + goto error; + buffer = heap_alloc(size); + if (!buffer) + goto error; + if (!ReadFile(file, buffer, size, &read, NULL) || read != size) + goto error; + + *bytes = size; + *data = buffer; + + heap_free(fullpath); + CloseHandle(file); + return S_OK; + +error: + heap_free(fullpath); + heap_free(buffer); + CloseHandle(file); + WARN("Returning E_FAIL.\n"); + return E_FAIL; +} + +static HRESULT WINAPI d3dcompiler_include_from_file_close(ID3DInclude *iface, const void *data) +{ + heap_free((void *)data); + return S_OK; +} + +const struct ID3DIncludeVtbl d3dcompiler_include_from_file_vtbl = +{ + d3dcompiler_include_from_file_open, + d3dcompiler_include_from_file_close +}; + +struct d3dcompiler_include_from_file +{ + ID3DInclude ID3DInclude_iface; +}; + static HRESULT preprocess_shader(const void *data, SIZE_T data_size, const char *filename, const D3D_SHADER_MACRO *defines, ID3DInclude *include, ID3DBlob **error_messages) { + struct d3dcompiler_include_from_file include_from_file; int ret; HRESULT hr = S_OK; const D3D_SHADER_MACRO *def = defines; + if (include == D3D_COMPILE_STANDARD_FILE_INCLUDE) + { + include_from_file.ID3DInclude_iface.lpVtbl = &d3dcompiler_include_from_file_vtbl; + include = &include_from_file.ID3DInclude_iface; + } + if (def != NULL) { while (def->Name != NULL) @@ -654,88 +736,12 @@ HRESULT WINAPI D3DAssemble(const void *data, SIZE_T datasize, const char *filena return hr; } -static HRESULT WINAPI d3dcompiler_include_from_file_open(ID3DInclude *iface, D3D_INCLUDE_TYPE include_type, - const char *filename, const void *parent_data, const void **data, UINT *bytes) -{ - char *fullpath, *buffer = NULL, current_dir[MAX_PATH + 1]; - const char *initial_dir; - SIZE_T size; - HANDLE file; - ULONG read; - DWORD len; - - if ((initial_dir = strrchr(initial_filename, '\\'))) - { - len = initial_dir - initial_filename + 1; - initial_dir = initial_filename; - } - else - { - len = GetCurrentDirectoryA(MAX_PATH, current_dir); - current_dir[len] = '\\'; - len++; - initial_dir = current_dir; - } - fullpath = heap_alloc(len + strlen(filename) + 1); - if (!fullpath) - return E_OUTOFMEMORY; - memcpy(fullpath, initial_dir, len); - strcpy(fullpath + len, filename); - - file = CreateFileA(fullpath, GENERIC_READ, FILE_SHARE_READ, 0, OPEN_EXISTING, 0, 0); - if (file == INVALID_HANDLE_VALUE) - goto error; - - TRACE("Include file found at %s.\n", debugstr_a(fullpath)); - - size = GetFileSize(file, NULL); - if (size == INVALID_FILE_SIZE) - goto error; - buffer = heap_alloc(size); - if (!buffer) - goto error; - if (!ReadFile(file, buffer, size, &read, NULL) || read != size) - goto error; - - *bytes = size; - *data = buffer; - - heap_free(fullpath); - CloseHandle(file); - return S_OK; - -error: - heap_free(fullpath); - heap_free(buffer); - CloseHandle(file); - WARN("Returning E_FAIL.\n"); - return E_FAIL; -} - -static HRESULT WINAPI d3dcompiler_include_from_file_close(ID3DInclude *iface, const void *data) -{ - heap_free((void *)data); - return S_OK; -} - -const struct ID3DIncludeVtbl d3dcompiler_include_from_file_vtbl = -{ - d3dcompiler_include_from_file_open, - d3dcompiler_include_from_file_close -}; - -struct d3dcompiler_include_from_file -{ - ID3DInclude ID3DInclude_iface; -}; - HRESULT WINAPI D3DCompile2(const void *data, SIZE_T data_size, const char *filename, const D3D_SHADER_MACRO *defines, ID3DInclude *include, const char *entrypoint, const char *target, UINT sflags, UINT eflags, UINT secondary_flags, const void *secondary_data, SIZE_T secondary_data_size, ID3DBlob **shader, ID3DBlob **error_messages) { - struct d3dcompiler_include_from_file include_from_file; HRESULT hr; TRACE("data %p, data_size %lu, filename %s, defines %p, include %p, entrypoint %s, " @@ -751,12 +757,6 @@ HRESULT WINAPI D3DCompile2(const void *data, SIZE_T data_size, const char *filen if (shader) *shader = NULL; if (error_messages) *error_messages = NULL; - if (include == D3D_COMPILE_STANDARD_FILE_INCLUDE) - { - include_from_file.ID3DInclude_iface.lpVtbl = &d3dcompiler_include_from_file_vtbl; - include = &include_from_file.ID3DInclude_iface; - } - EnterCriticalSection(&wpp_mutex); hr = preprocess_shader(data, data_size, filename, defines, include, error_messages);

1 0

Matteo Bruni : d3dcompiler: Clear preprocessor lexer state after running it.
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: 450d435306dae98d86e99bf894a2b14d30395e1e URL: https://source.winehq.org/git/wine.git/?a=commit;h=450d435306dae98d86e99bf8… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Wed Sep 8 12:49:02 2021 +0200 d3dcompiler: Clear preprocessor lexer state after running it. Otherwise state might leak to the next preprocessed shader. Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dcompiler_43/compiler.c | 1 + dlls/d3dcompiler_43/wpp_private.h | 1 + 2 files changed, 2 insertions(+) diff --git a/dlls/d3dcompiler_43/compiler.c b/dlls/d3dcompiler_43/compiler.c index d4523edb99c..b75a1d0a6a5 100644 --- a/dlls/d3dcompiler_43/compiler.c +++ b/dlls/d3dcompiler_43/compiler.c @@ -475,6 +475,7 @@ int wpp_parse( const char *input, FILE *output ) } /* Clean if_stack, it could remain dirty on errors */ while (pp_get_if_depth()) pp_pop_if(); + ppy_lex_destroy(); del_special_defines(); del_cmdline_defines(); pp_pop_define_state(); diff --git a/dlls/d3dcompiler_43/wpp_private.h b/dlls/d3dcompiler_43/wpp_private.h index 325c7f1e510..3b6d947d8a8 100644 --- a/dlls/d3dcompiler_43/wpp_private.h +++ b/dlls/d3dcompiler_43/wpp_private.h @@ -235,6 +235,7 @@ extern includelogicentry_t *pp_includelogiclist; extern FILE *ppy_out; extern char *ppy_text; int ppy_lex(void); +int ppy_lex_destroy(void); void pp_do_include(char *fname, int type); void pp_push_ignore_state(void);

1 0

Zebediah Figura : d3dcompiler/tests: Test include preprocessing with more functions.
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: 0ef10854c8a243896cf1c1ee6008c15161081c3b URL: https://source.winehq.org/git/wine.git/?a=commit;h=0ef10854c8a243896cf1c1ee… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Sep 8 12:49:03 2021 +0200 d3dcompiler/tests: Test include preprocessing with more functions. Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dcompiler_43/tests/hlsl_d3d9.c | 218 +++++++++++++++++++++++----------- 1 file changed, 147 insertions(+), 71 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=0ef10854c8a243896cf1…

1 0

Nikolay Sivov : d3d10/effect: Fix indexing in GetPixelShader().
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: 288428cc73c5f9fb6746e8e799330ea44c08e55a URL: https://source.winehq.org/git/wine.git/?a=commit;h=288428cc73c5f9fb6746e8e7… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Sep 3 12:30:01 2021 +0300 d3d10/effect: Fix indexing in GetPixelShader(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10/effect.c | 15 ++++++++++----- dlls/d3d10/tests/effect.c | 32 ++++++++++++++++++++++++++++++++ 2 files changed, 42 insertions(+), 5 deletions(-) diff --git a/dlls/d3d10/effect.c b/dlls/d3d10/effect.c index 180d36ce65b..fbfe8d0113a 100644 --- a/dlls/d3d10/effect.c +++ b/dlls/d3d10/effect.c @@ -7104,19 +7104,24 @@ static HRESULT STDMETHODCALLTYPE d3d10_effect_shader_variable_GetPixelShader( ID3D10EffectShaderVariable *iface, UINT index, ID3D10PixelShader **shader) { struct d3d10_effect_variable *v = impl_from_ID3D10EffectShaderVariable(iface); + struct d3d10_effect_shader_variable *s; + D3D10_SHADER_VARIABLE_TYPE basetype; + HRESULT hr; TRACE("iface %p, index %u, shader %p.\n", iface, index, shader); - if (v->type->element_count) - v = impl_from_ID3D10EffectVariable(iface->lpVtbl->GetElement(iface, index)); + *shader = NULL; - if (v->type->basetype != D3D10_SVT_PIXELSHADER) + if (FAILED(hr = d3d10_get_shader_variable(v, index, &s, &basetype))) + return hr; + + if (basetype != D3D10_SVT_PIXELSHADER) { WARN("Shader is not a pixel shader.\n"); - return E_FAIL; + return D3DERR_INVALIDCALL; } - if ((*shader = v->u.shader.shader.ps)) + if ((*shader = s->shader.ps)) ID3D10PixelShader_AddRef(*shader); return S_OK; diff --git a/dlls/d3d10/tests/effect.c b/dlls/d3d10/tests/effect.c index 4583863d59b..603f3dd049a 100644 --- a/dlls/d3d10/tests/effect.c +++ b/dlls/d3d10/tests/effect.c @@ -6015,6 +6015,7 @@ static void test_effect_shader_object(void) ID3D10EffectShaderVariable *s; ID3D10EffectVariable *v; ID3D10VertexShader *vs; + ID3D10PixelShader *ps; ID3D10Effect* effect; ID3D10Device *device; ULONG refcount; @@ -6060,6 +6061,37 @@ static void test_effect_shader_object(void) ok(!!vs, "Unexpected shader object.\n"); ID3D10VertexShader_Release(vs); + ps = (void *)0xdeadbeef; + hr = s->lpVtbl->GetPixelShader(s, 0, &ps); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!ps, "Unexpected shader object.\n"); + + ps = (void *)0xdeadbeef; + hr = s->lpVtbl->GetPixelShader(s, 1, &ps); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + ok(!ps, "Unexpected shader object.\n"); + + ps = (void *)0xdeadbeef; + hr = s->lpVtbl->GetPixelShader(s, 2, &ps); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!ps, "Unexpected shader object.\n"); + + ps = (void *)0xdeadbeef; + hr = s->lpVtbl->GetPixelShader(s, 3, &ps); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!ps, "Unexpected shader object.\n"); + + ps = (void *)0xdeadbeef; + hr = s->lpVtbl->GetPixelShader(s, 4, &ps); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!ps, "Unexpected shader object.\n"); + + ps = NULL; + hr = s->lpVtbl->GetPixelShader(s, 5, &ps); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + ok(!!ps, "Unexpected shader object.\n"); + ID3D10PixelShader_Release(ps); + effect->lpVtbl->Release(effect); refcount = ID3D10Device_Release(device);

1 0

Nikolay Sivov : d3d10/effect: Fix indexing in GetGeometryShader().
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: 3b074fa3beee132c3eb80a6ee7a2964170dbcd23 URL: https://source.winehq.org/git/wine.git/?a=commit;h=3b074fa3beee132c3eb80a6e… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Sep 3 12:30:02 2021 +0300 d3d10/effect: Fix indexing in GetGeometryShader(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10/effect.c | 15 ++++++++++----- dlls/d3d10/tests/effect.c | 43 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 53 insertions(+), 5 deletions(-) diff --git a/dlls/d3d10/effect.c b/dlls/d3d10/effect.c index fbfe8d0113a..6e429c32b2d 100644 --- a/dlls/d3d10/effect.c +++ b/dlls/d3d10/effect.c @@ -7082,19 +7082,24 @@ static HRESULT STDMETHODCALLTYPE d3d10_effect_shader_variable_GetGeometryShader( ID3D10EffectShaderVariable *iface, UINT index, ID3D10GeometryShader **shader) { struct d3d10_effect_variable *v = impl_from_ID3D10EffectShaderVariable(iface); + struct d3d10_effect_shader_variable *s; + D3D10_SHADER_VARIABLE_TYPE basetype; + HRESULT hr; TRACE("iface %p, index %u, shader %p.\n", iface, index, shader); - if (v->type->element_count) - v = impl_from_ID3D10EffectVariable(iface->lpVtbl->GetElement(iface, index)); + *shader = NULL; - if (v->type->basetype != D3D10_SVT_GEOMETRYSHADER) + if (FAILED(hr = d3d10_get_shader_variable(v, index, &s, &basetype))) + return hr; + + if (basetype != D3D10_SVT_GEOMETRYSHADER) { WARN("Shader is not a geometry shader.\n"); - return E_FAIL; + return D3DERR_INVALIDCALL; } - if ((*shader = v->u.shader.shader.gs)) + if ((*shader = s->shader.gs)) ID3D10GeometryShader_AddRef(*shader); return S_OK; diff --git a/dlls/d3d10/tests/effect.c b/dlls/d3d10/tests/effect.c index 603f3dd049a..df9777491b6 100644 --- a/dlls/d3d10/tests/effect.c +++ b/dlls/d3d10/tests/effect.c @@ -6013,6 +6013,7 @@ static void test_effect_shader_description(void) static void test_effect_shader_object(void) { ID3D10EffectShaderVariable *s; + ID3D10GeometryShader *gs; ID3D10EffectVariable *v; ID3D10VertexShader *vs; ID3D10PixelShader *ps; @@ -6092,6 +6093,48 @@ static void test_effect_shader_object(void) ok(!!ps, "Unexpected shader object.\n"); ID3D10PixelShader_Release(ps); + gs = (void *)0xdeadbeef; + hr = s->lpVtbl->GetGeometryShader(s, 0, &gs); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!gs, "Unexpected shader object.\n"); + + gs = (void *)0xdeadbeef; + hr = s->lpVtbl->GetGeometryShader(s, 1, &gs); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!gs, "Unexpected shader object.\n"); + + gs = (void *)0xdeadbeef; + hr = s->lpVtbl->GetGeometryShader(s, 2, &gs); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + ok(!gs, "Unexpected shader object.\n"); + + gs = (void *)0xdeadbeef; + hr = s->lpVtbl->GetGeometryShader(s, 3, &gs); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!gs, "Unexpected shader object.\n"); + + gs = (void *)0xdeadbeef; + hr = s->lpVtbl->GetGeometryShader(s, 4, &gs); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!gs, "Unexpected shader object.\n"); + + gs = (void *)0xdeadbeef; + hr = s->lpVtbl->GetGeometryShader(s, 5, &gs); + ok(hr == D3DERR_INVALIDCALL, "Unexpected hr %#x.\n", hr); + ok(!gs, "Unexpected shader object.\n"); + + gs = NULL; + hr = s->lpVtbl->GetGeometryShader(s, 6, &gs); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + ok(!!gs, "Unexpected shader object.\n"); + ID3D10GeometryShader_Release(gs); + + gs = NULL; + hr = s->lpVtbl->GetGeometryShader(s, 7, &gs); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); + ok(!!gs, "Unexpected shader object.\n"); + ID3D10GeometryShader_Release(gs); + effect->lpVtbl->Release(effect); refcount = ID3D10Device_Release(device);

1 0

Nikolay Sivov : d3d10/effect: Use case-insensitive comparison in GetVariableBySemantic().
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: a32a495012e9d846daa4b43143f779bebba46f30 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a32a495012e9d846daa4b431… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Sep 3 12:30:03 2021 +0300 d3d10/effect: Use case-insensitive comparison in GetVariableBySemantic(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10/effect.c | 4 ++-- dlls/d3d10/tests/effect.c | 6 ++++++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/dlls/d3d10/effect.c b/dlls/d3d10/effect.c index 6e429c32b2d..5efeb2901c0 100644 --- a/dlls/d3d10/effect.c +++ b/dlls/d3d10/effect.c @@ -3322,7 +3322,7 @@ static struct ID3D10EffectVariable * STDMETHODCALLTYPE d3d10_effect_GetVariableB { struct d3d10_effect_variable *v = &l->members[j]; - if (v->semantic && !strcmp(v->semantic, semantic)) + if (v->semantic && !stricmp(v->semantic, semantic)) { TRACE("Returning variable %p.\n", v); return &v->ID3D10EffectVariable_iface; @@ -3334,7 +3334,7 @@ static struct ID3D10EffectVariable * STDMETHODCALLTYPE d3d10_effect_GetVariableB { struct d3d10_effect_variable *v = &This->local_variables[i]; - if (v->semantic && !strcmp(v->semantic, semantic)) + if (v->semantic && !stricmp(v->semantic, semantic)) { TRACE("Returning variable %p.\n", v); return &v->ID3D10EffectVariable_iface; diff --git a/dlls/d3d10/tests/effect.c b/dlls/d3d10/tests/effect.c index df9777491b6..a448ba83714 100644 --- a/dlls/d3d10/tests/effect.c +++ b/dlls/d3d10/tests/effect.c @@ -3850,6 +3850,9 @@ static void test_effect_get_variable_by(void) variable = effect->lpVtbl->GetVariableBySemantic(effect, "SV_POSITION"); ok(variable_by_index == variable, "GetVariableBySemantic got %p, expected %p\n", variable, variable_by_index); + variable = effect->lpVtbl->GetVariableBySemantic(effect, "sv_POSITION"); + ok(variable_by_index == variable, "GetVariableBySemantic got %p, expected %p\n", variable, variable_by_index); + /* variable f2 */ variable_by_index = effect->lpVtbl->GetVariableByIndex(effect, 1); ok(null_variable != variable_by_index, "GetVariableByIndex failed %p\n", variable_by_index); @@ -3860,6 +3863,9 @@ static void test_effect_get_variable_by(void) variable = effect->lpVtbl->GetVariableBySemantic(effect, "COLOR0"); ok(variable_by_index == variable, "GetVariableBySemantic got %p, expected %p\n", variable, variable_by_index); + variable = effect->lpVtbl->GetVariableBySemantic(effect, "color0"); + ok(variable_by_index == variable, "GetVariableBySemantic got %p, expected %p\n", variable, variable_by_index); + /* variable f3 */ variable_by_index = effect->lpVtbl->GetVariableByIndex(effect, 2); ok(null_variable != variable_by_index, "GetVariableByIndex failed %p\n", variable_by_index);

1 0

Nikolay Sivov : d3d10/effect: Use case-insensitive comparison in GetMemberTypeBySemantic().
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: af20f75f7c1b6a693b9b5d5480b52dd3926f2f57 URL: https://source.winehq.org/git/wine.git/?a=commit;h=af20f75f7c1b6a693b9b5d54… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Sep 3 12:30:04 2021 +0300 d3d10/effect: Use case-insensitive comparison in GetMemberTypeBySemantic(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10/effect.c | 2 +- dlls/d3d10/tests/effect.c | 8 +++++--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/dlls/d3d10/effect.c b/dlls/d3d10/effect.c index 5efeb2901c0..5ade60537ef 100644 --- a/dlls/d3d10/effect.c +++ b/dlls/d3d10/effect.c @@ -8277,7 +8277,7 @@ static struct ID3D10EffectType * STDMETHODCALLTYPE d3d10_effect_type_GetMemberTy { struct d3d10_effect_type_member *typem = &This->members[i]; - if (typem->semantic && !strcmp(typem->semantic, semantic)) + if (typem->semantic && !stricmp(typem->semantic, semantic)) { TRACE("Returning type %p.\n", typem->type); return &typem->type->ID3D10EffectType_iface; diff --git a/dlls/d3d10/tests/effect.c b/dlls/d3d10/tests/effect.c index a448ba83714..029d4271b14 100644 --- a/dlls/d3d10/tests/effect.c +++ b/dlls/d3d10/tests/effect.c @@ -390,11 +390,12 @@ static void test_effect_variable_type(void) ok(type_desc.UnpackedSize == 0x8, "UnpackedSize is %#x, expected 0x8\n", type_desc.UnpackedSize); ok(type_desc.Stride == 0x10, "Stride is %x, expected 0x10\n", type_desc.Stride); - for (i = 0; i < 3; ++i) + for (i = 0; i < 4; ++i) { if (i == 0) type3 = type2->lpVtbl->GetMemberTypeByIndex(type2, 0); else if (i == 1) type3 = type2->lpVtbl->GetMemberTypeByName(type2, "f3"); - else type3 = type2->lpVtbl->GetMemberTypeBySemantic(type2, "SV_POSITION"); + else if (i == 2) type3 = type2->lpVtbl->GetMemberTypeBySemantic(type2, "SV_POSITION"); + else type3 = type2->lpVtbl->GetMemberTypeBySemantic(type2, "sv_POSITION"); hr = type3->lpVtbl->GetDesc(type3, &type_desc); ok(SUCCEEDED(hr), "GetDesc failed (%x)\n", hr); @@ -413,7 +414,8 @@ static void test_effect_variable_type(void) if (i == 0) type3 = type2->lpVtbl->GetMemberTypeByIndex(type2, 1); else if (i == 1) type3 = type2->lpVtbl->GetMemberTypeByName(type2, "f4"); - else type3 = type2->lpVtbl->GetMemberTypeBySemantic(type2, "COLOR0"); + else if (i == 2) type3 = type2->lpVtbl->GetMemberTypeBySemantic(type2, "COLOR0"); + else type3 = type2->lpVtbl->GetMemberTypeBySemantic(type2, "color0"); hr = type3->lpVtbl->GetDesc(type3, &type_desc); ok(SUCCEEDED(hr), "GetDesc failed (%x)\n", hr);

1 0

Nikolay Sivov : d3d10/effect: Use case-insensitive comparison in GetMemberBySemantic().
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: e8843de0f94bb3546607a3a1277fd17cba9bafd4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e8843de0f94bb3546607a3a1… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Sep 3 12:30:05 2021 +0300 d3d10/effect: Use case-insensitive comparison in GetMemberBySemantic(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10/effect.c | 2 +- dlls/d3d10/tests/effect.c | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/dlls/d3d10/effect.c b/dlls/d3d10/effect.c index 5ade60537ef..6d12920c762 100644 --- a/dlls/d3d10/effect.c +++ b/dlls/d3d10/effect.c @@ -4109,7 +4109,7 @@ static struct ID3D10EffectVariable * STDMETHODCALLTYPE d3d10_effect_variable_Get { struct d3d10_effect_variable *m = &This->members[i]; - if (m->semantic && !strcmp(m->semantic, semantic)) + if (m->semantic && !stricmp(m->semantic, semantic)) { TRACE("Returning member %p\n", m); return &m->ID3D10EffectVariable_iface; diff --git a/dlls/d3d10/tests/effect.c b/dlls/d3d10/tests/effect.c index 029d4271b14..60db5ad55be 100644 --- a/dlls/d3d10/tests/effect.c +++ b/dlls/d3d10/tests/effect.c @@ -651,6 +651,9 @@ static void test_effect_variable_member(void) variable = variable2->lpVtbl->GetMemberBySemantic(variable2, "SV_POSITION"); ok(variable == variable3, "GetMemberBySemantic got %p, expected %p\n", variable, variable3); + variable = variable2->lpVtbl->GetMemberBySemantic(variable2, "sv_POSITION"); + ok(variable == variable3, "GetMemberBySemantic got %p, expected %p\n", variable, variable3); + variable = variable2->lpVtbl->GetMemberByIndex(variable2, 0); ok(variable == variable3, "GetMemberByIndex got %p, expected %p\n", variable, variable3); @@ -668,6 +671,9 @@ static void test_effect_variable_member(void) variable = variable2->lpVtbl->GetMemberBySemantic(variable2, "COLOR0"); ok(variable == variable3, "GetMemberBySemantic got %p, expected %p\n", variable, variable3); + variable = variable2->lpVtbl->GetMemberBySemantic(variable2, "color0"); + ok(variable == variable3, "GetMemberBySemantic got %p, expected %p\n", variable, variable3); + variable = variable2->lpVtbl->GetMemberByIndex(variable2, 1); ok(variable == variable3, "GetMemberByIndex got %p, expected %p\n", variable, variable3);

1 0

Nikolay Sivov : d3dx10_38/tests: Enable tests.
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: 3defadee9fdb4bad8f0acf7e333277765d205a2c URL: https://source.winehq.org/git/wine.git/?a=commit;h=3defadee9fdb4bad8f0acf7e… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Sep 3 12:31:31 2021 +0300 d3dx10_38/tests: Enable tests. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure | 1 + configure.ac | 1 + dlls/d3dx10_38/Makefile.in | 1 + dlls/d3dx10_38/tests/Makefile.in | 6 ++++++ 4 files changed, 9 insertions(+) diff --git a/configure b/configure index b879415ff6c..ee1efef77c2 100755 --- a/configure +++ b/configure @@ -20376,6 +20376,7 @@ wine_fn_config_makefile dlls/d3dx10_35 enable_d3dx10_35 wine_fn_config_makefile dlls/d3dx10_36 enable_d3dx10_36 wine_fn_config_makefile dlls/d3dx10_37 enable_d3dx10_37 wine_fn_config_makefile dlls/d3dx10_38 enable_d3dx10_38 +wine_fn_config_makefile dlls/d3dx10_38/tests enable_tests wine_fn_config_makefile dlls/d3dx10_39 enable_d3dx10_39 wine_fn_config_makefile dlls/d3dx10_39/tests enable_tests wine_fn_config_makefile dlls/d3dx10_40 enable_d3dx10_40 diff --git a/configure.ac b/configure.ac index 8c4985c0294..15b7c2add2a 100644 --- a/configure.ac +++ b/configure.ac @@ -3075,6 +3075,7 @@ WINE_CONFIG_MAKEFILE(dlls/d3dx10_35) WINE_CONFIG_MAKEFILE(dlls/d3dx10_36) WINE_CONFIG_MAKEFILE(dlls/d3dx10_37) WINE_CONFIG_MAKEFILE(dlls/d3dx10_38) +WINE_CONFIG_MAKEFILE(dlls/d3dx10_38/tests) WINE_CONFIG_MAKEFILE(dlls/d3dx10_39) WINE_CONFIG_MAKEFILE(dlls/d3dx10_39/tests) WINE_CONFIG_MAKEFILE(dlls/d3dx10_40) diff --git a/dlls/d3dx10_38/Makefile.in b/dlls/d3dx10_38/Makefile.in index 5942b7c7c42..a1e7028de86 100644 --- a/dlls/d3dx10_38/Makefile.in +++ b/dlls/d3dx10_38/Makefile.in @@ -1,4 +1,5 @@ MODULE = d3dx10_38.dll +IMPORTLIB = d3dx10_38 EXTRADLLFLAGS = -mno-cygwin -Wb,--prefer-native diff --git a/dlls/d3dx10_38/tests/Makefile.in b/dlls/d3dx10_38/tests/Makefile.in new file mode 100644 index 00000000000..9e911457c81 --- /dev/null +++ b/dlls/d3dx10_38/tests/Makefile.in @@ -0,0 +1,6 @@ +TESTDLL = d3dx10_38.dll +IMPORTS = d3dx10_38 ole32 gdi32 +PARENTSRC = ../../d3dx10_43/tests + +C_SRCS = \ + d3dx10.c

1 0

Nikolay Sivov : d3dx10: Partially implement D3DX10CreateEffectFromResource().
by Alexandre Julliard 08 Sep '21

08 Sep '21

Module: wine Branch: master Commit: 737da3eb1ce5521bda1351771ec8d6ab34390080 URL: https://source.winehq.org/git/wine.git/?a=commit;h=737da3eb1ce5521bda135177… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Sep 3 15:46:57 2021 +0300 d3dx10: Partially implement D3DX10CreateEffectFromResource(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dx10_43/compiler.c | 82 +++++++++++++++++++++++++++++++++++++++++++ dlls/d3dx10_43/d3dx10_43.spec | 4 +-- dlls/d3dx10_43/tests/d3dx10.c | 22 ++++++++++++ include/d3dx10async.h | 12 +++++++ 4 files changed, 118 insertions(+), 2 deletions(-) diff --git a/dlls/d3dx10_43/compiler.c b/dlls/d3dx10_43/compiler.c index 0bda2712d76..2a306fa8f57 100644 --- a/dlls/d3dx10_43/compiler.c +++ b/dlls/d3dx10_43/compiler.c @@ -123,3 +123,85 @@ HRESULT WINAPI D3DX10CreateEffectFromFileA(const char *filename, const D3D10_SHA return hr; } + +static HRESULT get_resource_data(HMODULE module, HRSRC resinfo, void **buffer, DWORD *length) +{ + HGLOBAL resource; + + *length = SizeofResource(module, resinfo); + if (!*length) + return D3DX10_ERR_INVALID_DATA; + + resource = LoadResource(module, resinfo); + if (!resource) + return D3DX10_ERR_INVALID_DATA; + + *buffer = LockResource(resource); + if (!*buffer) + return D3DX10_ERR_INVALID_DATA; + + return S_OK; +} + +HRESULT WINAPI D3DX10CreateEffectFromResourceA(HMODULE module, const char *resource_name, + const char *filename, const D3D10_SHADER_MACRO *defines, ID3D10Include *include, + const char *profile, UINT shader_flags, UINT effect_flags, ID3D10Device *device, + ID3D10EffectPool *effect_pool, ID3DX10ThreadPump *pump, ID3D10Effect **effect, + ID3D10Blob **errors, HRESULT *hresult) +{ + HRSRC resinfo; + void *data; + DWORD size; + HRESULT hr; + + TRACE("module %p, resource_name %s, filename %s, defines %p, include %p, profile %s, " + "shader_flags %#x, effect_flags %#x, device %p, effect_pool %p, pump %p, effect %p, " + "errors %p, hresult %p.\n", module, debugstr_a(resource_name), debugstr_a(filename), + defines, include, debugstr_a(profile), shader_flags, effect_flags, + device, effect_pool, pump, effect, errors, hresult); + + if (!(resinfo = FindResourceA(module, resource_name, (const char *)RT_RCDATA))) + return D3DX10_ERR_INVALID_DATA; + + if (FAILED(hr = get_resource_data(module, resinfo, &data, &size))) + return hr; + + return D3DX10CreateEffectFromMemory(data, size, filename, defines, include, profile, + shader_flags, effect_flags, device, effect_pool, pump, effect, errors, hresult); +} + +HRESULT WINAPI D3DX10CreateEffectFromResourceW(HMODULE module, const WCHAR *resource_name, + const WCHAR *filenameW, const D3D10_SHADER_MACRO *defines, ID3D10Include *include, + const char *profile, UINT shader_flags, UINT effect_flags, ID3D10Device *device, + ID3D10EffectPool *effect_pool, ID3DX10ThreadPump *pump, ID3D10Effect **effect, + ID3D10Blob **errors, HRESULT *hresult) +{ + char *filename; + HRSRC resinfo; + void *data; + DWORD size; + HRESULT hr; + int len; + + TRACE("module %p, resource_name %s, filename %s, defines %p, include %p, profile %s, " + "shader_flags %#x, effect_flags %#x, device %p, effect_pool %p, pump %p, effect %p, " + "errors %p, hresult %p.\n", module, debugstr_w(resource_name), debugstr_w(filenameW), + defines, include, debugstr_a(profile), shader_flags, effect_flags, + device, effect_pool, pump, effect, errors, hresult); + + if (!(resinfo = FindResourceW(module, resource_name, (const WCHAR *)RT_RCDATA))) + return D3DX10_ERR_INVALID_DATA; + + if (FAILED(hr = get_resource_data(module, resinfo, &data, &size))) + return hr; + + len = WideCharToMultiByte(CP_ACP, 0, filenameW, -1, NULL, 0, NULL, NULL); + if (!(filename = heap_alloc(len))) + return E_OUTOFMEMORY; + WideCharToMultiByte(CP_ACP, 0, filenameW, -1, filename, len, NULL, NULL); + + hr = D3DX10CreateEffectFromMemory(data, size, filename, defines, include, profile, + shader_flags, effect_flags, device, effect_pool, pump, effect, errors, hresult); + heap_free(filename); + return hr; +} diff --git a/dlls/d3dx10_43/d3dx10_43.spec b/dlls/d3dx10_43/d3dx10_43.spec index da6629ad1a0..8bc9451aeab 100644 --- a/dlls/d3dx10_43/d3dx10_43.spec +++ b/dlls/d3dx10_43/d3dx10_43.spec @@ -23,8 +23,8 @@ @ stdcall D3DX10CreateEffectFromFileA(str ptr ptr str long long ptr ptr ptr ptr ptr ptr) @ stdcall D3DX10CreateEffectFromFileW(wstr ptr ptr str long long ptr ptr ptr ptr ptr ptr) @ stdcall D3DX10CreateEffectFromMemory(ptr long str ptr ptr str long long ptr ptr ptr ptr ptr ptr) -@ stub D3DX10CreateEffectFromResourceA(long str str ptr ptr str long long ptr ptr ptr ptr ptr ptr) -@ stub D3DX10CreateEffectFromResourceW(long wstr wstr ptr ptr str long long ptr ptr ptr ptr ptr ptr) +@ stdcall D3DX10CreateEffectFromResourceA(long str str ptr ptr str long long ptr ptr ptr ptr ptr ptr) +@ stdcall D3DX10CreateEffectFromResourceW(long wstr wstr ptr ptr str long long ptr ptr ptr ptr ptr ptr) @ stdcall D3DX10CreateEffectPoolFromFileA(str ptr ptr str long long ptr ptr ptr ptr ptr) @ stdcall D3DX10CreateEffectPoolFromFileW(wstr ptr ptr str long long ptr ptr ptr ptr ptr) @ stdcall D3DX10CreateEffectPoolFromMemory(ptr long str ptr ptr str long long ptr ptr ptr ptr ptr) diff --git a/dlls/d3dx10_43/tests/d3dx10.c b/dlls/d3dx10_43/tests/d3dx10.c index bc5ca6747f5..70827056f5f 100644 --- a/dlls/d3dx10_43/tests/d3dx10.c +++ b/dlls/d3dx10_43/tests/d3dx10.c @@ -3153,6 +3153,27 @@ todo_wine ok(!refcount, "Unexpected refcount.\n"); } +static void test_create_effect_from_resource(void) +{ + ID3D10Device *device; + ID3D10Effect *effect; + ULONG refcount; + HRESULT hr; + + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } + + hr = D3DX10CreateEffectFromResourceA(GetModuleHandleA(NULL), "resource", NULL, NULL, NULL, + "fx_4_0", 0, 0, device, NULL, NULL, &effect, NULL, NULL); + ok(hr == D3DX10_ERR_INVALID_DATA, "Unexpected hr %#x.\n", hr); + + refcount = ID3D10Device_Release(device); + ok(!refcount, "Unexpected refcount.\n"); +} + START_TEST(d3dx10) { test_D3DX10UnsetAllDeviceObjects(); @@ -3163,4 +3184,5 @@ START_TEST(d3dx10) test_create_texture(); test_font(); test_sprite(); + test_create_effect_from_resource(); } diff --git a/include/d3dx10async.h b/include/d3dx10async.h index c8204bc7798..0b7aca9c023 100644 --- a/include/d3dx10async.h +++ b/include/d3dx10async.h @@ -58,6 +58,18 @@ HRESULT WINAPI D3DX10PreprocessShaderFromMemory(const char *data, SIZE_T data_si const D3D10_SHADER_MACRO *defines, ID3DInclude *include, ID3DX10ThreadPump *pump, ID3D10Blob **shader_text, ID3D10Blob **errors, HRESULT *hresult); +HRESULT WINAPI D3DX10CreateEffectFromResourceA(HMODULE module, const char *resource_name, + const char *filename, const D3D10_SHADER_MACRO *defines, ID3D10Include *include, + const char *profile, UINT shader_flags, UINT effect_flags, ID3D10Device *device, + ID3D10EffectPool *effect_pool, ID3DX10ThreadPump *pump, ID3D10Effect **effect, + ID3D10Blob **errors, HRESULT *hresult); + +HRESULT WINAPI D3DX10CreateEffectFromResourceW(HMODULE module, const WCHAR *resource_name, + const WCHAR *filename, const D3D10_SHADER_MACRO *defines, ID3D10Include *include, + const char *profile, UINT shader_flags, UINT effect_flags, ID3D10Device *device, + ID3D10EffectPool *effect_pool, ID3DX10ThreadPump *pump, ID3D10Effect **effect, + ID3D10Blob **errors, HRESULT *hresult); + HRESULT WINAPI D3DX10CreateAsyncFileLoaderW(const WCHAR *filename, ID3DX10DataLoader **loader); HRESULT WINAPI D3DX10CreateAsyncFileLoaderA(const char *filename, ID3DX10DataLoader **loader); HRESULT WINAPI D3DX10CreateAsyncMemoryLoader(const void *data, SIZE_T datasize, ID3DX10DataLoader **loader);

1 0