March 2019 - Wine-Commits - list.winehq.org

Alexandre Julliard : winegcc: Also return Windows link flags in get_link_args().
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: 44839493ae6a99d0e3aa0b2c22a83e1061daff04 URL: https://source.winehq.org/git/wine.git/?a=commit;h=44839493ae6a99d0e3aa0b2c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Sat Mar 9 20:43:43 2019 +0100 winegcc: Also return Windows link flags in get_link_args(). Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winegcc/winegcc.c | 63 ++++++++++++++++++++++++------------------------- 1 file changed, 31 insertions(+), 32 deletions(-) diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index 5eafe54..b58c39a 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -425,6 +425,36 @@ static strarray *get_link_args( struct options *opts, const char *output_name ) strarray_add( flags, strmake( "-Wl,-soname,%s.so", output_name )); break; + case PLATFORM_WINDOWS: + case PLATFORM_CYGWIN: + if (opts->shared) + { + strarray_add( flags, "-shared" ); + strarray_add( flags, "-Wl,--kill-at" ); + } + else strarray_add( flags, opts->gui_app ? "-mwindows" : "-mconsole" ); + + if (opts->unicode_app) strarray_add( flags, "-municode" ); + if (opts->nodefaultlibs) strarray_add( flags, "-nodefaultlibs" ); + if (opts->nostartfiles) strarray_add( flags, "-nostartfiles" ); + + if (opts->subsystem) + { + strarray_add( flags, strmake("-Wl,--subsystem,%s", opts->subsystem )); + if (!strcmp( opts->subsystem, "native" )) + { + const char *entry = opts->target_cpu == CPU_x86 ? "_DriverEntry(a)8" : "DriverEntry"; + strarray_add( flags, strmake( "-Wl,--entry,%s", entry )); + } + } + + if (opts->image_base) strarray_add( flags, strmake("-Wl,--image-base,%s", opts->image_base )); + + if (opts->large_address_aware && opts->target_cpu == CPU_x86) + strarray_add( flags, "-Wl,--large-address-aware" ); + + return flags; + default: if (opts->image_base) { @@ -906,7 +936,6 @@ static void build(struct options* opts) /* mark the files with their appropriate type */ spec_file = lang = 0; files = strarray_alloc(); - link_args = strarray_alloc(); for ( j = 0; j < opts->files->size; j++ ) { const char* file = opts->files->base[j]; @@ -961,7 +990,7 @@ static void build(struct options* opts) if (opts->win16_app) error( "Building 16-bit code is not supported for Windows\n" ); - strarray_addall(link_args, get_translator(opts)); + link_args = get_link_args( opts, output_name ); if (opts->shared) { @@ -979,42 +1008,12 @@ static void build(struct options* opts) spawn(opts->prefix, spec_args, 0); strarray_free(spec_args); - strarray_add(link_args, "-shared"); - if (verbose) strarray_add(link_args, "-v"); - strarray_add(link_args, "-Wl,--kill-at"); strarray_add(link_args, spec_def_name); } - else - { - strarray_add(link_args, opts->gui_app ? "-mwindows" : "-mconsole"); - } - - if (opts->unicode_app) strarray_add(link_args, "-municode"); - if (opts->nodefaultlibs) strarray_add(link_args, "-nodefaultlibs"); - if (opts->nostartfiles) strarray_add(link_args, "-nostartfiles" ); - - if (opts->subsystem) - { - strarray_add(link_args, strmake("-Wl,--subsystem,%s", opts->subsystem)); - if (!strcmp( opts->subsystem, "native" )) - { - const char *entry = opts->target_cpu == CPU_x86 ? "_DriverEntry(a)8" : "DriverEntry"; - strarray_add(link_args, strmake( "-Wl,--entry,%s", entry )); - } - } - - for ( j = 0 ; j < opts->linker_args->size ; j++ ) - strarray_add(link_args, opts->linker_args->base[j]); strarray_add(link_args, "-o"); strarray_add(link_args, output_file); - if (opts->image_base) - strarray_add(link_args, strmake("-Wl,--image-base,%s", opts->image_base)); - - if (opts->large_address_aware && opts->target_cpu == CPU_x86) - strarray_add( link_args, "-Wl,--large-address-aware" ); - for ( j = 0; j < lib_dirs->size; j++ ) strarray_add(link_args, strmake("-L%s", lib_dirs->base[j]));

1 0

Alexandre Julliard : winegcc: Make get_lddllflags() more generic to return all link flags.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: 9ce91cd56eb191eb7f76834ed4452a25fa7d4205 URL: https://source.winehq.org/git/wine.git/?a=commit;h=9ce91cd56eb191eb7f76834e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Sat Mar 9 20:42:37 2019 +0100 winegcc: Make get_lddllflags() more generic to return all link flags. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winegcc/winegcc.c | 130 +++++++++++++++++++++++------------------------- 1 file changed, 63 insertions(+), 67 deletions(-) diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index 4f9498a..5eafe54 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -213,6 +213,7 @@ struct options const char* section_align; const char* lib_suffix; const char* subsystem; + const char* prelink; strarray* prefix; strarray* lib_dirs; strarray* linker_args; @@ -378,9 +379,17 @@ static int try_link( const strarray *prefix, const strarray *link_tool, const ch return ret; } -static const strarray* get_lddllflags( const struct options *opts, const strarray *link_tool ) +static strarray *get_link_args( struct options *opts, const char *output_name ) { + const strarray *link_tool = get_translator( opts ); strarray *flags = strarray_alloc(); + unsigned int i; + + strarray_addall( flags, link_tool ); + for (i = 0; i < opts->linker_args->size; i++) strarray_add( flags, opts->linker_args->base[i] ); + + if (verbose > 1) strarray_add( flags, "-v" ); + switch (opts->target_platform) { case PLATFORM_APPLE: @@ -392,34 +401,64 @@ static const strarray* get_lddllflags( const struct options *opts, const strarra strarray_add( flags, "-read_only_relocs" ); strarray_add( flags, "warning" ); } - break; + if (opts->image_base) + { + strarray_add( flags, "-image_base" ); + strarray_add( flags, opts->image_base ); + } + if (opts->strip) strarray_add( flags, "-x" ); + return flags; - case PLATFORM_ANDROID: case PLATFORM_SOLARIS: - case PLATFORM_UNSPECIFIED: - strarray_add( flags, "-shared" ); - strarray_add( flags, "-Wl,-Bsymbolic" ); - - /* Try all options first - this is likely to succeed on modern compilers */ - if (!try_link( opts->prefix, link_tool, "-fPIC -shared -Wl,-Bsymbolic " - "-Wl,-z,defs -Wl,-init,__wine_spec_init,-fini,_wine_spec_fini" )) { - strarray_add( flags, "-Wl,-z,defs" ); - strarray_add( flags, "-Wl,-init,__wine_spec_init,-fini,__wine_spec_fini" ); + char *mapfile = get_temp_file( output_name, ".map" ); + const char *align = opts->section_align ? opts->section_align : "0x1000"; + + create_file( mapfile, 0644, "text = A%s;\ndata = A%s;\n", align, align ); + strarray_add( flags, strmake("-Wl,-M,%s", mapfile) ); + strarray_add( tmp_files, mapfile ); } - else /* otherwise figure out which ones are allowed */ + break; + + case PLATFORM_ANDROID: + /* the Android loader requires a soname for all libraries */ + strarray_add( flags, strmake( "-Wl,-soname,%s.so", output_name )); + break; + + default: + if (opts->image_base) { - if (!try_link( opts->prefix, link_tool, "-fPIC -shared -Wl,-Bsymbolic -Wl,-z,defs" )) - strarray_add( flags, "-Wl,-z,defs" ); - if (!try_link( opts->prefix, link_tool, "-fPIC -shared -Wl,-Bsymbolic " - "-Wl,-init,__wine_spec_init,-fini,_wine_spec_fini" )) - strarray_add( flags, "-Wl,-init,__wine_spec_init,-fini,__wine_spec_fini" ); + if (!try_link( opts->prefix, link_tool, strmake("-Wl,-Ttext-segment=%s", opts->image_base)) ) + strarray_add( flags, strmake("-Wl,-Ttext-segment=%s", opts->image_base) ); + else + opts->prelink = PRELINK; } + if (!try_link( opts->prefix, link_tool, "-Wl,-z,max-page-size=0x1000")) + strarray_add( flags, "-Wl,-z,max-page-size=0x1000"); break; + } - default: - assert(0); + /* generic Unix shared library flags */ + + strarray_add( flags, "-shared" ); + strarray_add( flags, "-Wl,-Bsymbolic" ); + + /* Try all options first - this is likely to succeed on modern compilers */ + if (!try_link( opts->prefix, link_tool, "-fPIC -shared -Wl,-Bsymbolic " + "-Wl,-z,defs -Wl,-init,__wine_spec_init,-fini,_wine_spec_fini" )) + { + strarray_add( flags, "-Wl,-z,defs" ); + strarray_add( flags, "-Wl,-init,__wine_spec_init,-fini,__wine_spec_fini" ); } + else /* otherwise figure out which ones are allowed */ + { + if (!try_link( opts->prefix, link_tool, "-fPIC -shared -Wl,-Bsymbolic -Wl,-z,defs" )) + strarray_add( flags, "-Wl,-z,defs" ); + if (!try_link( opts->prefix, link_tool, "-fPIC -shared -Wl,-Bsymbolic " + "-Wl,-init,__wine_spec_init,-fini,_wine_spec_fini" )) + strarray_add( flags, "-Wl,-init,__wine_spec_init,-fini,__wine_spec_fini" ); + } + return flags; } @@ -807,7 +846,6 @@ static void build(struct options* opts) char *output_file; const char *spec_o_name; const char *output_name, *spec_file, *lang; - const char *prelink = NULL; int generate_app_loader = 1; int fake_module = 0; unsigned int j; @@ -1151,53 +1189,11 @@ static void build(struct options* opts) if (fake_module) return; /* nothing else to do */ /* link everything together now */ - strarray_addall(link_args, get_translator(opts)); - strarray_addall(link_args, get_lddllflags(opts, link_args)); + link_args = get_link_args( opts, output_name ); strarray_add(link_args, "-o"); strarray_add(link_args, strmake("%s.so", output_file)); - for ( j = 0 ; j < opts->linker_args->size ; j++ ) - strarray_add(link_args, opts->linker_args->base[j]); - - switch (opts->target_platform) - { - case PLATFORM_APPLE: - if (opts->image_base) - { - strarray_add(link_args, "-image_base"); - strarray_add(link_args, opts->image_base); - } - if (opts->strip) - strarray_add(link_args, "-Wl,-x"); - break; - case PLATFORM_SOLARIS: - { - char *mapfile = get_temp_file( output_name, ".map" ); - const char *align = opts->section_align ? opts->section_align : "0x1000"; - - create_file( mapfile, 0644, "text = A%s;\ndata = A%s;\n", align, align ); - strarray_add(link_args, strmake("-Wl,-M,%s", mapfile)); - strarray_add(tmp_files, mapfile); - } - break; - case PLATFORM_ANDROID: - /* the Android loader requires a soname for all libraries */ - strarray_add( link_args, strmake( "-Wl,-soname,%s.so", output_name )); - break; - default: - if (opts->image_base) - { - if (!try_link(opts->prefix, link_args, strmake("-Wl,-Ttext-segment=%s", opts->image_base))) - strarray_add(link_args, strmake("-Wl,-Ttext-segment=%s", opts->image_base)); - else - prelink = PRELINK; - } - if (!try_link(opts->prefix, link_args, "-Wl,-z,max-page-size=0x1000")) - strarray_add(link_args, "-Wl,-z,max-page-size=0x1000"); - break; - } - for ( j = 0; j < lib_dirs->size; j++ ) strarray_add(link_args, strmake("-L%s", lib_dirs->base[j])); @@ -1229,12 +1225,12 @@ static void build(struct options* opts) strarray_free (link_args); /* set the base address with prelink if linker support is not present */ - if (prelink && !opts->target) + if (opts->prelink && !opts->target) { - if (prelink[0] && strcmp(prelink,"false")) + if (opts->prelink[0] && strcmp(opts->prelink,"false")) { strarray *prelink_args = strarray_alloc(); - strarray_add(prelink_args, prelink); + strarray_add(prelink_args, opts->prelink); strarray_add(prelink_args, "--reloc-only"); strarray_add(prelink_args, opts->image_base); strarray_add(prelink_args, strmake("%s.so", output_file));

1 0

Alexandre Julliard : winegcc: Get rid of the Mingw Unicode hack.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: d228b3cfd5cf7d758e858f05216cbb349c116c2c URL: https://source.winehq.org/git/wine.git/?a=commit;h=d228b3cfd5cf7d758e858f05… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Sat Mar 9 20:41:03 2019 +0100 winegcc: Get rid of the Mingw Unicode hack. Mingw supports Unicode nowadays. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winegcc/winegcc.c | 25 +------------------------ 1 file changed, 1 insertion(+), 24 deletions(-) diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index 54d4368..4f9498a 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -800,27 +800,6 @@ static void add_library( struct options *opts, strarray *lib_dirs, strarray *fil free(fullname); } -/* hack a main or WinMain function to work around Mingw's lack of Unicode support */ -static const char *mingw_unicode_hack( struct options *opts ) -{ - char *main_stub = get_temp_file( opts->output_name, ".c" ); - - create_file( main_stub, 0644, - "typedef unsigned short wchar_t;\n" - "extern void * __stdcall LoadLibraryA(const char *);\n" - "extern void * __stdcall GetProcAddress(void *,const char *);\n" - "extern int wmain( int argc, wchar_t *argv[] );\n\n" - "int main( int argc, char *argv[] )\n{\n" - " int wargc;\n" - " wchar_t **wargv, **wenv;\n" - " void *msvcrt = LoadLibraryA( \"msvcrt.dll\" );\n" - " void (*__wgetmainargs)(int *argc, wchar_t** *wargv, wchar_t** *wenvp, int expand_wildcards,\n" - " int *new_mode) = GetProcAddress( msvcrt, \"__wgetmainargs\" );\n" - " __wgetmainargs( &wargc, &wargv, &wenv, 0, 0 );\n" - " return wmain( wargc, wargv );\n}\n" ); - return compile_to_object( opts, main_stub, NULL ); -} - static void build(struct options* opts) { strarray *lib_dirs, *files; @@ -972,6 +951,7 @@ static void build(struct options* opts) strarray_add(link_args, opts->gui_app ? "-mwindows" : "-mconsole"); } + if (opts->unicode_app) strarray_add(link_args, "-municode"); if (opts->nodefaultlibs) strarray_add(link_args, "-nodefaultlibs"); if (opts->nostartfiles) strarray_add(link_args, "-nostartfiles" ); @@ -997,9 +977,6 @@ static void build(struct options* opts) if (opts->large_address_aware && opts->target_cpu == CPU_x86) strarray_add( link_args, "-Wl,--large-address-aware" ); - if (opts->unicode_app && !opts->shared) - strarray_add(link_args, mingw_unicode_hack(opts)); - for ( j = 0; j < lib_dirs->size; j++ ) strarray_add(link_args, strmake("-L%s", lib_dirs->base[j]));

1 0

Michael Stefaniuc : d3dx9: Merge the d3dx_effect_GetMatrix() helper.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: 636a2b5a39f81699bddd289b04de9644611b8987 URL: https://source.winehq.org/git/wine.git/?a=commit;h=636a2b5a39f81699bddd289b… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Sat Mar 9 11:21:28 2019 +0100 d3dx9: Merge the d3dx_effect_GetMatrix() helper. Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dx9_36/effect.c | 59 ++++++++++++++++++++++---------------------------- 1 file changed, 26 insertions(+), 33 deletions(-) diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c index 38e6d85..124e1cf 100644 --- a/dlls/d3dx9_36/effect.c +++ b/dlls/d3dx9_36/effect.c @@ -1559,38 +1559,6 @@ static HRESULT d3dx9_base_effect_set_matrix(struct d3dx9_base_effect *base, return D3DERR_INVALIDCALL; } -static HRESULT d3dx9_base_effect_get_matrix(struct d3dx9_base_effect *base, - D3DXHANDLE parameter, D3DXMATRIX *matrix) -{ - struct d3dx_parameter *param = get_valid_parameter(base, parameter); - - if (matrix && param && !param->element_count) - { - TRACE("Class %s\n", debug_d3dxparameter_class(param->class)); - - switch (param->class) - { - case D3DXPC_MATRIX_ROWS: - get_matrix(param, matrix, FALSE); - return D3D_OK; - - case D3DXPC_SCALAR: - case D3DXPC_VECTOR: - case D3DXPC_OBJECT: - case D3DXPC_STRUCT: - break; - - default: - FIXME("Unhandled class %s\n", debug_d3dxparameter_class(param->class)); - break; - } - } - - WARN("Parameter not found.\n"); - - return D3DERR_INVALIDCALL; -} - static HRESULT d3dx9_base_effect_set_matrix_array(struct d3dx9_base_effect *base, D3DXHANDLE parameter, const D3DXMATRIX *matrix, UINT count) { @@ -3564,10 +3532,35 @@ static HRESULT WINAPI d3dx_effect_SetMatrix(ID3DXEffect *iface, D3DXHANDLE param static HRESULT WINAPI d3dx_effect_GetMatrix(ID3DXEffect *iface, D3DXHANDLE parameter, D3DXMATRIX *matrix) { struct d3dx_effect *effect = impl_from_ID3DXEffect(iface); + struct d3dx_parameter *param = get_valid_parameter(&effect->base_effect, parameter); TRACE("iface %p, parameter %p, matrix %p.\n", iface, parameter, matrix); - return d3dx9_base_effect_get_matrix(&effect->base_effect, parameter, matrix); + if (matrix && param && !param->element_count) + { + TRACE("Class %s.\n", debug_d3dxparameter_class(param->class)); + + switch (param->class) + { + case D3DXPC_MATRIX_ROWS: + get_matrix(param, matrix, FALSE); + return D3D_OK; + + case D3DXPC_SCALAR: + case D3DXPC_VECTOR: + case D3DXPC_OBJECT: + case D3DXPC_STRUCT: + break; + + default: + FIXME("Unhandled class %s.\n", debug_d3dxparameter_class(param->class)); + break; + } + } + + WARN("Parameter not found.\n"); + + return D3DERR_INVALIDCALL; } static HRESULT WINAPI d3dx_effect_SetMatrixArray(ID3DXEffect *iface, D3DXHANDLE parameter,

1 0

Alexandre Julliard : configure: Remove a few no longer used definitions.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: 2f149571d5627add6954a7b6bb71306bb779c652 URL: https://source.winehq.org/git/wine.git/?a=commit;h=2f149571d5627add6954a7b6… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Sat Mar 9 21:11:25 2019 +0100 configure: Remove a few no longer used definitions. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure | 17 ----------------- configure.ac | 14 -------------- include/config.h.in | 3 --- 3 files changed, 34 deletions(-) diff --git a/configure b/configure index 1d80c01..9988dcd 100755 --- a/configure +++ b/configure @@ -7788,7 +7788,6 @@ WINEPRELOADER_LDFLAGS="" LIBEXT="so" DLLEXT=".so" -IMPLIBEXT="def" # Extract the first word of "ldd", so it can be a program name with args. set dummy ldd; ac_word=$2 { $as_echo "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5 @@ -8153,7 +8152,6 @@ fi LIBEXT="dll" DLLEXT="" - IMPLIBEXT="a" enable_iphlpapi=${enable_iphlpapi:-no} enable_kernel32=${enable_kernel32:-no} enable_msvcrt=${enable_msvcrt:-no} @@ -8164,12 +8162,6 @@ fi with_x=${with_x:-no} with_pthread=${with_pthread:-no} - crtlibs="" - case $host_os in - mingw32*) - crtlibs="-lmsvcrt" ;; - esac - LIBWINE_SHAREDLIB="libwine.dll" LIBWINE_IMPORTLIB="libwine.a" @@ -16560,23 +16552,14 @@ fi { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_cv_asm_func_def" >&5 $as_echo "$ac_cv_asm_func_def" >&6; } - case "$ac_cv_asm_func_def" in ".def") - $as_echo "#define __ASM_FUNC(name) \".def \" __ASM_NAME(name) \"; .scl 2; .type 32; .endef\"" >>confdefs.h - asm_func_header=".def $asm_name_prefix\" #name suffix \"; .scl 2; .type 32; .endef" ;; ".type @function") - $as_echo "#define __ASM_FUNC(name) \".type \" __ASM_NAME(name) \",@function\"" >>confdefs.h - asm_func_header=".type $asm_name_prefix\" #name suffix \",@function" ;; ".type 2") - $as_echo "#define __ASM_FUNC(name) \".type \" __ASM_NAME(name) \",2\"" >>confdefs.h - asm_func_header=".type $asm_name_prefix\" #name suffix \",2" ;; *) - $as_echo "#define __ASM_FUNC(name) \"\"" >>confdefs.h - asm_func_header="" ;; esac diff --git a/configure.ac b/configure.ac index 71e8606..25581a9 100644 --- a/configure.ac +++ b/configure.ac @@ -710,7 +710,6 @@ AC_SUBST(WINELOADER_LDFLAGS,"") AC_SUBST(WINEPRELOADER_LDFLAGS,"") LIBEXT="so" DLLEXT=".so" -IMPLIBEXT="def" WINE_PATH_SONAME_TOOLS WINE_CONFIG_HELPERS @@ -724,7 +723,6 @@ case $host_os in AC_CHECK_TOOL(DLLTOOL,dlltool,false) LIBEXT="dll" DLLEXT="" - IMPLIBEXT="a" dnl Disable modules that can't be used on Windows enable_iphlpapi=${enable_iphlpapi:-no} enable_kernel32=${enable_kernel32:-no} @@ -737,13 +735,6 @@ case $host_os in with_x=${with_x:-no} with_pthread=${with_pthread:-no} - dnl Mingw needs explicit msvcrt for linking libwine and winsock for wininet - crtlibs="" - case $host_os in - mingw32*) - crtlibs="-lmsvcrt" ;; - esac - AC_SUBST(LIBWINE_SHAREDLIB,"libwine.dll") AC_SUBST(LIBWINE_IMPORTLIB,"libwine.a") AC_SUBST(LIBWINE_LDFLAGS,"-shared \$(srcdir)/wine.def") @@ -2058,19 +2049,14 @@ AC_CACHE_CHECK([how to define a function in assembly code], ac_cv_asm_func_def, ac_cv_asm_func_def=".type 2", ac_cv_asm_func_def="unknown")])])) -AH_TEMPLATE(__ASM_FUNC,[Define to a macro to generate an assembly function directive]) case "$ac_cv_asm_func_def" in ".def") - AC_DEFINE([__ASM_FUNC(name)], [".def " __ASM_NAME(name) "; .scl 2; .type 32; .endef"]) asm_func_header=".def $asm_name_prefix\" #name suffix \"; .scl 2; .type 32; .endef" ;; ".type @function") - AC_DEFINE([__ASM_FUNC(name)], [".type " __ASM_NAME(name) ",@function"]) asm_func_header=".type $asm_name_prefix\" #name suffix \",@function" ;; ".type 2") - AC_DEFINE([__ASM_FUNC(name)], [".type " __ASM_NAME(name) ",2"]) asm_func_header=".type $asm_name_prefix\" #name suffix \",2" ;; *) - AC_DEFINE([__ASM_FUNC(name)], [""]) asm_func_header="" ;; esac diff --git a/include/config.h.in b/include/config.h.in index 999f4a6..1d69e2e 100644 --- a/include/config.h.in +++ b/include/config.h.in @@ -1631,9 +1631,6 @@ /* Define to a macro to define an assembly function */ #undef __ASM_DEFINE_FUNC -/* Define to a macro to generate an assembly function directive */ -#undef __ASM_FUNC - /* Define to a macro to generate an assembly function with C calling convention */ #undef __ASM_GLOBAL_FUNC

1 0

Erich E. Hoover : msidb: Add stub tool for manipulating MSI databases.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: 3d5b606ddea7ede27eb2320e2755666bdbed1f33 URL: https://source.winehq.org/git/wine.git/?a=commit;h=3d5b606ddea7ede27eb2320e… Author: Erich E. Hoover <erich.e.hoover(a)wine-staging.com> Date: Tue Sep 15 17:44:33 2015 -0600 msidb: Add stub tool for manipulating MSI databases. The "Windows SDK Components for Windows Installer Developers" has a command line tool called msidb that is incredibly useful for creating, editing, and exporting MSI installer databases, think of it as winemsibuilder on steroids. This patch series implements much of the functionality of the msidb tool, maintains compatible CLI flags, and the underlying MSI functionality necessary to support these features. Jacek expressed an interest in having these patches resurrected for use by the Gecko build scripts and Austin's VS builds of Valgrind. With this patch series all the existing winemsibuilder functionality is available, plus the ability to drop streams, export the _SummaryInformation table, and export binary streams (Binary/Icon tables). A big feature of the implementation is that it allows you to edit existing installer databases, rather than just creating new ones. Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure | 3 + configure.ac | 2 + programs/msidb/Makefile.in | 5 ++ programs/msidb/main.c | 153 +++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 163 insertions(+) diff --git a/configure b/configure index 9988dcd..8a07748 100755 --- a/configure +++ b/configure @@ -1715,6 +1715,7 @@ enable_ipconfig enable_lodctr enable_mofcomp enable_mshta +enable_msidb enable_msiexec enable_msinfo32 enable_net @@ -20292,6 +20293,7 @@ wine_fn_config_makefile programs/ipconfig enable_ipconfig wine_fn_config_makefile programs/lodctr enable_lodctr wine_fn_config_makefile programs/mofcomp enable_mofcomp wine_fn_config_makefile programs/mshta enable_mshta +wine_fn_config_makefile programs/msidb enable_msidb wine_fn_config_makefile programs/msiexec enable_msiexec wine_fn_config_makefile programs/msinfo32 enable_msinfo32 wine_fn_config_makefile programs/net enable_net @@ -20443,6 +20445,7 @@ else fonts \ loader/l_intl.nls \ loader/wine.inf \ +programs/msidb/msidb \ programs/msiexec/msiexec \ programs/notepad/notepad \ programs/regedit/regedit \ diff --git a/configure.ac b/configure.ac index 25581a9..db8bb6c 100644 --- a/configure.ac +++ b/configure.ac @@ -3938,6 +3938,7 @@ WINE_CONFIG_MAKEFILE(programs/ipconfig) WINE_CONFIG_MAKEFILE(programs/lodctr) WINE_CONFIG_MAKEFILE(programs/mofcomp) WINE_CONFIG_MAKEFILE(programs/mshta) +WINE_CONFIG_MAKEFILE(programs/msidb) WINE_CONFIG_MAKEFILE(programs/msiexec) WINE_CONFIG_MAKEFILE(programs/msinfo32) WINE_CONFIG_MAKEFILE(programs/net) @@ -4106,6 +4107,7 @@ else fonts \ loader/l_intl.nls \ loader/wine.inf \ +programs/msidb/msidb \ programs/msiexec/msiexec \ programs/notepad/notepad \ programs/regedit/regedit \ diff --git a/programs/msidb/Makefile.in b/programs/msidb/Makefile.in new file mode 100644 index 0000000..bb74041 --- /dev/null +++ b/programs/msidb/Makefile.in @@ -0,0 +1,5 @@ +MODULE = msidb.exe +APPMODE = -mconsole -municode +IMPORTS = msi + +C_SRCS = main.c diff --git a/programs/msidb/main.c b/programs/msidb/main.c new file mode 100644 index 0000000..a431fd5 --- /dev/null +++ b/programs/msidb/main.c @@ -0,0 +1,153 @@ +/* + * msidb - command line tool for assembling MSI packages + * + * Copyright 2015 Erich E. Hoover + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#define WIN32_LEAN_AND_MEAN + +#include <stdlib.h> +#include <windows.h> +#include <msi.h> +#include <msiquery.h> + +#include "wine/debug.h" +#include "wine/unicode.h" + +WINE_DEFAULT_DEBUG_CHANNEL(msidb); + +struct msidb_state +{ + WCHAR *database_file; + WCHAR *table_folder; + MSIHANDLE database_handle; + BOOL create_database; +}; + +static void show_usage( void ) +{ + WINE_MESSAGE( + "Usage: msidb [OPTION]...[OPTION]...\n" + "Options:\n" + " -? Show this usage message and exit.\n" + " -c Create database file (instead of opening existing file).\n" + " -d package.msi Path to the database file.\n" + " -f folder Folder in which to open/save the tables.\n" + ); +} + +static int valid_state( struct msidb_state *state ) +{ + if (state->database_file == NULL) + { + FIXME( "GUI operation is not currently supported.\n" ); + return 0; + } + if (state->table_folder == NULL) + { + ERR( "No table folder specified (-f option).\n" ); + show_usage(); + return 0; + } + return 1; +} + +static int process_argument( struct msidb_state *state, int i, int argc, WCHAR *argv[] ) +{ + /* msidb accepts either "-" or "/" style flags */ + if (strlenW(argv[i]) != 2 || (argv[i][0] != '-' && argv[i][0] != '/')) + { + WINE_FIXME( "Table names are not currently supported.\n" ); + show_usage(); + exit( 1 ); + } + switch( argv[i][1] ) + { + case '?': + show_usage(); + exit( 0 ); + case 'c': + state->create_database = TRUE; + return 1; + case 'd': + if (i + 1 >= argc) return 0; + state->database_file = argv[i + 1]; + return 2; + case 'f': + if (i + 1 >= argc) return 0; + state->table_folder = argv[i + 1]; + return 2; + default: + break; + } + show_usage(); + exit( 1 ); +} + +static int open_database( struct msidb_state *state ) +{ + LPCWSTR db_mode = state->create_database ? MSIDBOPEN_CREATE : MSIDBOPEN_TRANSACT; + UINT ret; + + ret = MsiOpenDatabaseW( state->database_file, db_mode, &state->database_handle ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to open database '%s', error %d\n", wine_dbgstr_w(state->database_file), ret ); + return 0; + } + return 1; +} + +static void close_database( struct msidb_state *state ) +{ + UINT ret; + + ret = MsiDatabaseCommit( state->database_handle ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to commit changes to database.\n" ); + return; + } + ret = MsiCloseHandle( state->database_handle ); + if (ret != ERROR_SUCCESS) + { + WARN( "Failed to close database handle.\n" ); + return; + } +} + +int wmain( int argc, WCHAR *argv[] ) +{ + struct msidb_state state; + int i, n = 1; + + memset( &state, 0x0, sizeof(state) ); + /* process and validate all the command line flags */ + for (i = 1; n && i < argc; i += n) + n = process_argument( &state, i, argc, argv ); + if (!valid_state( &state )) + return 1; + + /* perform the requested operations */ + if (!open_database( &state )) + { + ERR( "Failed to open database '%s'.\n", wine_dbgstr_w(state.database_file) ); + return 1; + } + close_database( &state ); + return 0; +}

1 0

Erich E. Hoover : msidb: Add support for importing database tables.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: 68e1ad07750bd6f9fa7b529398f79db1622c4ad5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=68e1ad07750bd6f9fa7b5293… Author: Erich E. Hoover <erich.e.hoover(a)wine-staging.com> Date: Wed Sep 16 11:58:50 2015 -0600 msidb: Add support for importing database tables. msidb uses a nice CLI syntax for adding multiple database tables in one call with the "-i" mode flag, this patch implements that syntax. For example, this call would import three tables from the current directory (ActionTe.idt, Componen.idt, and InstallE.idt): msidb -d package.msi -f . -i ActionText Component InstallExecuteSequence Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/msidb/main.c | 113 ++++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 100 insertions(+), 13 deletions(-) diff --git a/programs/msidb/main.c b/programs/msidb/main.c index a431fd5..9c4628b 100644 --- a/programs/msidb/main.c +++ b/programs/msidb/main.c @@ -27,26 +27,61 @@ #include "wine/debug.h" #include "wine/unicode.h" +#include "wine/list.h" WINE_DEFAULT_DEBUG_CHANNEL(msidb); +struct msidb_listentry +{ + struct list entry; + WCHAR *name; +}; + struct msidb_state { WCHAR *database_file; WCHAR *table_folder; MSIHANDLE database_handle; BOOL create_database; + BOOL import_tables; + struct list table_list; }; +static void list_free( struct list *list ) +{ + struct msidb_listentry *data, *next; + + LIST_FOR_EACH_ENTRY_SAFE( data, next, list, struct msidb_listentry, entry ) + { + list_remove( &data->entry ); + HeapFree( GetProcessHeap(), 0, data ); + } +} + +static void list_append( struct list *list, WCHAR *name ) +{ + struct msidb_listentry *data; + + data = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(struct msidb_listentry) ); + if (!data) + { + ERR( "Out of memory for list.\n" ); + exit( 1 ); + } + data->name = name; + list_add_tail( list, &data->entry ); +} + static void show_usage( void ) { WINE_MESSAGE( - "Usage: msidb [OPTION]...[OPTION]...\n" + "Usage: msidb [OPTION]...[OPTION]... [TABLE]...[TABLE]\n" "Options:\n" " -? Show this usage message and exit.\n" " -c Create database file (instead of opening existing file).\n" " -d package.msi Path to the database file.\n" " -f folder Folder in which to open/save the tables.\n" + " -i Import tables into database.\n" ); } @@ -63,6 +98,17 @@ static int valid_state( struct msidb_state *state ) show_usage(); return 0; } + if (!state->create_database && !state->import_tables) + { + ERR( "No mode flag specified (-c, -i).\n" ); + show_usage(); + return 0; + } + if (list_empty( &state->table_list )) + { + ERR( "No tables specified.\n" ); + return 0; + } return 1; } @@ -70,11 +116,7 @@ static int process_argument( struct msidb_state *state, int i, int argc, WCHAR * { /* msidb accepts either "-" or "/" style flags */ if (strlenW(argv[i]) != 2 || (argv[i][0] != '-' && argv[i][0] != '/')) - { - WINE_FIXME( "Table names are not currently supported.\n" ); - show_usage(); - exit( 1 ); - } + return 0; switch( argv[i][1] ) { case '?': @@ -91,6 +133,9 @@ static int process_argument( struct msidb_state *state, int i, int argc, WCHAR * if (i + 1 >= argc) return 0; state->table_folder = argv[i + 1]; return 2; + case 'i': + state->import_tables = TRUE; + return 1; default: break; } @@ -112,11 +157,14 @@ static int open_database( struct msidb_state *state ) return 1; } -static void close_database( struct msidb_state *state ) +static void close_database( struct msidb_state *state, BOOL commit_changes ) { - UINT ret; + UINT ret = ERROR_SUCCESS; - ret = MsiDatabaseCommit( state->database_handle ); + if (state->database_handle == 0) + return; + if (commit_changes) + ret = MsiDatabaseCommit( state->database_handle ); if (ret != ERROR_SUCCESS) { ERR( "Failed to commit changes to database.\n" ); @@ -130,24 +178,63 @@ static void close_database( struct msidb_state *state ) } } +static int import_table( struct msidb_state *state, const WCHAR *table_name ) +{ + const WCHAR format[] = { '%','.','8','s','.','i','d','t',0 }; /* truncate to 8 characters */ + WCHAR table_path[MAX_PATH]; + UINT ret; + + snprintfW( table_path, sizeof(table_path)/sizeof(WCHAR), format, table_name ); + ret = MsiDatabaseImportW( state->database_handle, state->table_folder, table_path ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to import table '%s', error %d.\n", wine_dbgstr_w(table_name), ret ); + return 0; + } + return 1; +} + +static int import_tables( struct msidb_state *state ) +{ + struct msidb_listentry *data; + + LIST_FOR_EACH_ENTRY( data, &state->table_list, struct msidb_listentry, entry ) + { + if (!import_table( state, data->name )) + return 0; /* failed, do not commit changes */ + } + return 1; +} + int wmain( int argc, WCHAR *argv[] ) { struct msidb_state state; int i, n = 1; + int ret = 1; memset( &state, 0x0, sizeof(state) ); + list_init( &state.table_list ); /* process and validate all the command line flags */ for (i = 1; n && i < argc; i += n) n = process_argument( &state, i, argc, argv ); + /* process all remaining arguments as table names */ + for (; i < argc; i++) + list_append( &state.table_list, argv[i] ); if (!valid_state( &state )) - return 1; + goto cleanup; /* perform the requested operations */ if (!open_database( &state )) { ERR( "Failed to open database '%s'.\n", wine_dbgstr_w(state.database_file) ); - return 1; + goto cleanup; } - close_database( &state ); - return 0; + if (state.import_tables && !import_tables( &state )) + goto cleanup; /* failed, do not commit changes */ + ret = 0; + +cleanup: + close_database( &state, ret == 0 ); + list_free( &state.table_list ); + return ret; }

1 0

Erich E. Hoover : msidb: Add support for adding stream/ cabinet files to MSI databases.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: 344bd3b1ccf4ba3d0ab40d5788ccbc3228c5b76d URL: https://source.winehq.org/git/wine.git/?a=commit;h=344bd3b1ccf4ba3d0ab40d57… Author: Erich E. Hoover <erich.e.hoover(a)wine-staging.com> Date: Wed Sep 16 14:43:18 2015 -0600 msidb: Add support for adding stream/cabinet files to MSI databases. msidb allows developers to add "streams" (cabinet files) to a database with the "-a" mode flag followed by a filename, example: msidb -d package.msi -a cabinet.cab Signed-off-by: Erich E. Hoover <erich.e.hoover(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/msidb/main.c | 90 ++++++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 86 insertions(+), 4 deletions(-) diff --git a/programs/msidb/main.c b/programs/msidb/main.c index 9c4628b..d837299 100644 --- a/programs/msidb/main.c +++ b/programs/msidb/main.c @@ -42,8 +42,10 @@ struct msidb_state WCHAR *database_file; WCHAR *table_folder; MSIHANDLE database_handle; + BOOL add_streams; BOOL create_database; BOOL import_tables; + struct list add_stream_list; struct list table_list; }; @@ -78,6 +80,7 @@ static void show_usage( void ) "Usage: msidb [OPTION]...[OPTION]... [TABLE]...[TABLE]\n" "Options:\n" " -? Show this usage message and exit.\n" + " -a file.cab Add stream/cabinet file to _Streams table.\n" " -c Create database file (instead of opening existing file).\n" " -d package.msi Path to the database file.\n" " -f folder Folder in which to open/save the tables.\n" @@ -92,19 +95,19 @@ static int valid_state( struct msidb_state *state ) FIXME( "GUI operation is not currently supported.\n" ); return 0; } - if (state->table_folder == NULL) + if (state->table_folder == NULL && !state->add_streams) { ERR( "No table folder specified (-f option).\n" ); show_usage(); return 0; } - if (!state->create_database && !state->import_tables) + if (!state->create_database && !state->import_tables && !state->add_streams) { - ERR( "No mode flag specified (-c, -i).\n" ); + ERR( "No mode flag specified (-a, -c, -i).\n" ); show_usage(); return 0; } - if (list_empty( &state->table_list )) + if (list_empty( &state->table_list ) && !state->add_streams) { ERR( "No tables specified.\n" ); return 0; @@ -122,6 +125,11 @@ static int process_argument( struct msidb_state *state, int i, int argc, WCHAR * case '?': show_usage(); exit( 0 ); + case 'a': + if (i + 1 >= argc) return 0; + state->add_streams = TRUE; + list_append( &state->add_stream_list, argv[i + 1] ); + return 2; case 'c': state->create_database = TRUE; return 1; @@ -178,6 +186,76 @@ static void close_database( struct msidb_state *state, BOOL commit_changes ) } } +static const WCHAR *basenameW( const WCHAR *filename ) +{ + const WCHAR *dir_end; + + dir_end = strrchrW( filename, '/' ); + if (dir_end) return dir_end + 1; + dir_end = strrchrW( filename, '\\' ); + if (dir_end) return dir_end + 1; + return filename; +} + +static int add_stream( struct msidb_state *state, const WCHAR *stream_filename ) +{ + static const char insert_command[] = "INSERT INTO _Streams (Name, Data) VALUES (?, ?)"; + MSIHANDLE view = 0, record = 0; + UINT ret; + + ret = MsiDatabaseOpenViewA( state->database_handle, insert_command, &view ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to open _Streams table.\n" ); + goto cleanup; + } + record = MsiCreateRecord( 2 ); + if (record == 0) + { + ERR( "Failed to create MSI record.\n" ); + ret = ERROR_OUTOFMEMORY; + goto cleanup; + } + ret = MsiRecordSetStringW( record, 1, basenameW( stream_filename ) ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to add stream filename to MSI record.\n" ); + goto cleanup; + } + ret = MsiRecordSetStreamW( record, 2, stream_filename ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to add stream to MSI record.\n" ); + goto cleanup; + } + ret = MsiViewExecute( view, record ); + if (ret != ERROR_SUCCESS) + { + ERR( "Failed to add stream to _Streams table.\n" ); + goto cleanup; + } + +cleanup: + if (record) + MsiCloseHandle( record ); + if (view) + MsiViewClose( view ); + + return (ret == ERROR_SUCCESS); +} + +static int add_streams( struct msidb_state *state ) +{ + struct msidb_listentry *data; + + LIST_FOR_EACH_ENTRY( data, &state->add_stream_list, struct msidb_listentry, entry ) + { + if (!add_stream( state, data->name )) + return 0; /* failed, do not commit changes */ + } + return 1; +} + static int import_table( struct msidb_state *state, const WCHAR *table_name ) { const WCHAR format[] = { '%','.','8','s','.','i','d','t',0 }; /* truncate to 8 characters */ @@ -213,6 +291,7 @@ int wmain( int argc, WCHAR *argv[] ) int ret = 1; memset( &state, 0x0, sizeof(state) ); + list_init( &state.add_stream_list ); list_init( &state.table_list ); /* process and validate all the command line flags */ for (i = 1; n && i < argc; i += n) @@ -229,12 +308,15 @@ int wmain( int argc, WCHAR *argv[] ) ERR( "Failed to open database '%s'.\n", wine_dbgstr_w(state.database_file) ); goto cleanup; } + if (state.add_streams && !add_streams( &state )) + goto cleanup; /* failed, do not commit changes */ if (state.import_tables && !import_tables( &state )) goto cleanup; /* failed, do not commit changes */ ret = 0; cleanup: close_database( &state, ret == 0 ); + list_free( &state.add_stream_list ); list_free( &state.table_list ); return ret; }

1 0

Michael Stefaniuc : d3dx9: Merge the d3dx_effect_SetIntArray() helper.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: 8ea0f9d4dd3e4d74af51df0a2dc2723f2be0da8c URL: https://source.winehq.org/git/wine.git/?a=commit;h=8ea0f9d4dd3e4d74af51df0a… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Sat Mar 9 11:21:27 2019 +0100 d3dx9: Merge the d3dx_effect_SetIntArray() helper. Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dx9_36/effect.c | 69 ++++++++++++++++++++++---------------------------- 1 file changed, 30 insertions(+), 39 deletions(-) diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c index f77348c..38e6d85 100644 --- a/dlls/d3dx9_36/effect.c +++ b/dlls/d3dx9_36/effect.c @@ -1345,44 +1345,6 @@ static HRESULT d3dx9_base_effect_get_value(struct d3dx9_base_effect *base, return D3DERR_INVALIDCALL; } -static HRESULT d3dx9_base_effect_set_int_array(struct d3dx9_base_effect *base, - D3DXHANDLE parameter, const INT *n, UINT count) -{ - struct d3dx_parameter *param = get_valid_parameter(base, parameter); - - if (param) - { - UINT i, size = min(count, param->bytes / sizeof(DWORD)); - - TRACE("Class %s\n", debug_d3dxparameter_class(param->class)); - - switch (param->class) - { - case D3DXPC_SCALAR: - case D3DXPC_VECTOR: - case D3DXPC_MATRIX_ROWS: - for (i = 0; i < size; ++i) - { - set_number((DWORD *)param->data + i, param->type, &n[i], D3DXPT_INT); - } - set_dirty(param); - return D3D_OK; - - case D3DXPC_OBJECT: - case D3DXPC_STRUCT: - break; - - default: - FIXME("Unhandled class %s\n", debug_d3dxparameter_class(param->class)); - break; - } - } - - WARN("Parameter not found.\n"); - - return D3DERR_INVALIDCALL; -} - static HRESULT d3dx9_base_effect_set_vector(struct d3dx9_base_effect *base, D3DXHANDLE parameter, const D3DXVECTOR4 *vector) { @@ -3388,10 +3350,39 @@ static HRESULT WINAPI d3dx_effect_GetInt(ID3DXEffect *iface, D3DXHANDLE paramete static HRESULT WINAPI d3dx_effect_SetIntArray(ID3DXEffect *iface, D3DXHANDLE parameter, const INT *n, UINT count) { struct d3dx_effect *effect = impl_from_ID3DXEffect(iface); + struct d3dx_parameter *param = get_valid_parameter(&effect->base_effect, parameter); TRACE("iface %p, parameter %p, n %p, count %u.\n", iface, parameter, n, count); - return d3dx9_base_effect_set_int_array(&effect->base_effect, parameter, n, count); + if (param) + { + unsigned int i, size = min(count, param->bytes / sizeof(DWORD)); + + TRACE("Class %s.\n", debug_d3dxparameter_class(param->class)); + + switch (param->class) + { + case D3DXPC_SCALAR: + case D3DXPC_VECTOR: + case D3DXPC_MATRIX_ROWS: + for (i = 0; i < size; ++i) + set_number((DWORD *)param->data + i, param->type, &n[i], D3DXPT_INT); + set_dirty(param); + return D3D_OK; + + case D3DXPC_OBJECT: + case D3DXPC_STRUCT: + break; + + default: + FIXME("Unhandled class %s.\n", debug_d3dxparameter_class(param->class)); + break; + } + } + + WARN("Parameter not found.\n"); + + return D3DERR_INVALIDCALL; } static HRESULT WINAPI d3dx_effect_GetIntArray(ID3DXEffect *iface, D3DXHANDLE parameter, INT *n, UINT count)

1 0

Michael Stefaniuc : d3dx9: Merge the d3dx_effect_SetMatrix() helper.
by Alexandre Julliard March 11, 2019

March 11, 2019

Module: wine Branch: master Commit: b1621b2c28c57af09064d69e6cbe8573db88b555 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b1621b2c28c57af09064d69e… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Sat Mar 9 11:21:31 2019 +0100 d3dx9: Merge the d3dx_effect_SetMatrix() helper. Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dx9_36/effect.c | 61 ++++++++++++++++++++++---------------------------- 1 file changed, 27 insertions(+), 34 deletions(-) diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c index 8702e2b..343260b 100644 --- a/dlls/d3dx9_36/effect.c +++ b/dlls/d3dx9_36/effect.c @@ -1526,39 +1526,6 @@ static HRESULT d3dx9_base_effect_get_vector_array(struct d3dx9_base_effect *base return D3DERR_INVALIDCALL; } -static HRESULT d3dx9_base_effect_set_matrix(struct d3dx9_base_effect *base, - D3DXHANDLE parameter, const D3DXMATRIX *matrix) -{ - struct d3dx_parameter *param = get_valid_parameter(base, parameter); - - if (param && !param->element_count) - { - TRACE("Class %s\n", debug_d3dxparameter_class(param->class)); - - switch (param->class) - { - case D3DXPC_MATRIX_ROWS: - set_matrix(param, matrix); - set_dirty(param); - return D3D_OK; - - case D3DXPC_SCALAR: - case D3DXPC_VECTOR: - case D3DXPC_OBJECT: - case D3DXPC_STRUCT: - break; - - default: - FIXME("Unhandled class %s\n", debug_d3dxparameter_class(param->class)); - break; - } - } - - WARN("Parameter not found.\n"); - - return D3DERR_INVALIDCALL; -} - static HRESULT d3dx9_base_effect_set_matrix_array(struct d3dx9_base_effect *base, D3DXHANDLE parameter, const D3DXMATRIX *matrix, UINT count) { @@ -3446,10 +3413,36 @@ static HRESULT WINAPI d3dx_effect_GetVectorArray(ID3DXEffect *iface, D3DXHANDLE static HRESULT WINAPI d3dx_effect_SetMatrix(ID3DXEffect *iface, D3DXHANDLE parameter, const D3DXMATRIX *matrix) { struct d3dx_effect *effect = impl_from_ID3DXEffect(iface); + struct d3dx_parameter *param = get_valid_parameter(&effect->base_effect, parameter); TRACE("iface %p, parameter %p, matrix %p.\n", iface, parameter, matrix); - return d3dx9_base_effect_set_matrix(&effect->base_effect, parameter, matrix); + if (param && !param->element_count) + { + TRACE("Class %s.\n", debug_d3dxparameter_class(param->class)); + + switch (param->class) + { + case D3DXPC_MATRIX_ROWS: + set_matrix(param, matrix); + set_dirty(param); + return D3D_OK; + + case D3DXPC_SCALAR: + case D3DXPC_VECTOR: + case D3DXPC_OBJECT: + case D3DXPC_STRUCT: + break; + + default: + FIXME("Unhandled class %s.\n", debug_d3dxparameter_class(param->class)); + break; + } + } + + WARN("Parameter not found.\n"); + + return D3DERR_INVALIDCALL; } static HRESULT WINAPI d3dx_effect_GetMatrix(ID3DXEffect *iface, D3DXHANDLE parameter, D3DXMATRIX *matrix)

1 0