September 2019 - Wine-commits

Matteo Bruni : d3dx9: Compute the correct source rect in D3DXLoadSurfaceFromFileInMemory().
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: f934ef1a14108aa535e76900b29cba0dd8253f80 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f934ef1a14108aa535e76900… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Fri Sep 27 20:40:05 2019 +0200 d3dx9: Compute the correct source rect in D3DXLoadSurfaceFromFileInMemory(). Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3dx9_36/surface.c | 2 +- dlls/d3dx9_36/tests/texture.c | 22 ++++++++++++++++++++++ 2 files changed, 23 insertions(+), 1 deletion(-) diff --git a/dlls/d3dx9_36/surface.c b/dlls/d3dx9_36/surface.c index bd2055d54d..b47509aa38 100644 --- a/dlls/d3dx9_36/surface.c +++ b/dlls/d3dx9_36/surface.c @@ -1168,7 +1168,7 @@ HRESULT WINAPI D3DXLoadSurfaceFromFileInMemory(IDirect3DSurface9 *pDestSurface, wicrect.Height = imginfo.Height; } - SetRect(&rect, 0, 0, wicrect.Width, wicrect.Height); + SetRect(&rect, wicrect.X, wicrect.Y, wicrect.X + wicrect.Width, wicrect.Y + wicrect.Height); if (imginfo.ImageFileFormat == D3DXIFF_DDS) { diff --git a/dlls/d3dx9_36/tests/texture.c b/dlls/d3dx9_36/tests/texture.c index 6d39bf72b9..229c857ac2 100644 --- a/dlls/d3dx9_36/tests/texture.c +++ b/dlls/d3dx9_36/tests/texture.c @@ -1824,6 +1824,28 @@ static void test_D3DXCreateTextureFromFileInMemory(IDirect3DDevice9 *device) hr = IDirect3DSurface9_UnlockRect(uncompressed_surface); ok(hr == D3D_OK, "Got unexpected hr %#x.\n", hr); + hr = D3DXLoadSurfaceFromFileInMemory(surface, NULL, &rect, dds_dxt5_8_8, + sizeof(dds_dxt5_8_8), &rect, D3DX_FILTER_POINT, 0, NULL); + ok(hr == D3D_OK, "Got unexpected hr %#x.\n", hr); + hr = D3DXLoadSurfaceFromSurface(uncompressed_surface, NULL, NULL, surface, NULL, NULL, D3DX_FILTER_NONE, 0); + ok(hr == D3D_OK, "Got unexpected hr %#x.\n", hr); + + hr = IDirect3DSurface9_LockRect(uncompressed_surface, &lock_rect, NULL, D3DLOCK_READONLY); + ok(hr == D3D_OK, "Got unexpected hr %#x.\n", hr); + for (y = 0; y < 8; ++y) + { + for (x = 0; x < 8; ++x) + { + ok(compare_color(((DWORD *)lock_rect.pBits)[lock_rect.Pitch / 4 * y + x], + dds_dxt5_8_8_expected_misaligned_3[y * 8 + x], 0), + "Color at position %u, %u is 0x%08x, expected 0x%08x.\n", + x, y, ((DWORD *)lock_rect.pBits)[lock_rect.Pitch / 4 * y + x], + dds_dxt5_8_8_expected_misaligned_3[y * 8 + x]); + } + } + hr = IDirect3DSurface9_UnlockRect(uncompressed_surface); + ok(hr == D3D_OK, "Got unexpected hr %#x.\n", hr); + IDirect3DSurface9_Release(uncompressed_surface); IDirect3DSurface9_Release(surface); IDirect3DTexture9_Release(texture);

1 0

Alexandre Julliard : Release 4.17.
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: 65fba0987310e635f27468a5b4a448794b5871e5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=65fba0987310e635f27468a5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 27 22:54:58 2019 +0200 Release 4.17. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- ANNOUNCE | 636 +++++++++++++++++++++++++++++++++++--------------------------- VERSION | 2 +- configure | 18 +- 3 files changed, 366 insertions(+), 290 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=65fba0987310e635f274…

1 0

Tag wine-4.17 : Alexandre Julliard : Release 4.17
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: refs/tags/wine-4.17 Tag: 662cffca4b7f6c3f300043192a6041e5b65a92ce URL: https://source.winehq.org/git/wine.git/?a=tag;h=662cffca4b7f6c3f300043192a6… Tagger: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 27 22:54:58 2019 +0200 Release 4.17

1 0

Alexandre Julliard : winegcc: Update __stdcall defines for ARM platforms.
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: c3072237911eba11195cd3aa0018186022cac869 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c3072237911eba11195cd3aa… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 27 12:01:40 2019 +0200 winegcc: Update __stdcall defines for ARM platforms. Reported by Kevin Puetz. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winegcc/winegcc.c | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index f1798a8978..6f26f60709 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -723,32 +723,29 @@ static void compile(struct options* opts, const char* lang) switch (opts->target_cpu) { case CPU_x86_64: + case CPU_ARM64: strarray_add(comp_args, "-D__stdcall=__attribute__((ms_abi))"); - strarray_add(comp_args, "-D__cdecl=__attribute__((ms_abi))"); - strarray_add(comp_args, "-D_stdcall=__attribute__((ms_abi))"); - strarray_add(comp_args, "-D_cdecl=__attribute__((ms_abi))"); - strarray_add(comp_args, "-D__fastcall=__attribute__((ms_abi))"); - strarray_add(comp_args, "-D_fastcall=__attribute__((ms_abi))"); + strarray_add(comp_args, "-D__cdecl=__stdcall"); + strarray_add(comp_args, "-D__fastcall=__stdcall"); break; case CPU_x86: strarray_add(comp_args, "-D__stdcall=__attribute__((__stdcall__)) __attribute__((__force_align_arg_pointer__))"); strarray_add(comp_args, "-D__cdecl=__attribute__((__cdecl__)) __attribute__((__force_align_arg_pointer__))"); - strarray_add(comp_args, "-D_stdcall=__attribute__((__stdcall__)) __attribute__((__force_align_arg_pointer__))"); - strarray_add(comp_args, "-D_cdecl=__attribute__((__cdecl__)) __attribute__((__force_align_arg_pointer__))"); strarray_add(comp_args, "-D__fastcall=__attribute__((__fastcall__))"); - strarray_add(comp_args, "-D_fastcall=__attribute__((__fastcall__))"); break; case CPU_ARM: - case CPU_ARM64: + strarray_add(comp_args, "-D__stdcall=__attribute__((pcs(\"aapcs-vfp\")))"); + strarray_add(comp_args, "-D__cdecl=__stdcall"); + strarray_add(comp_args, "-D__fastcall=__stdcall"); case CPU_POWERPC: strarray_add(comp_args, "-D__stdcall="); strarray_add(comp_args, "-D__cdecl="); - strarray_add(comp_args, "-D_stdcall="); - strarray_add(comp_args, "-D_cdecl="); strarray_add(comp_args, "-D__fastcall="); - strarray_add(comp_args, "-D_fastcall="); break; } + strarray_add(comp_args, "-D_stdcall=__stdcall"); + strarray_add(comp_args, "-D_cdecl=__cdecl"); + strarray_add(comp_args, "-D_fastcall=__fastcall"); strarray_add(comp_args, "-D__declspec(x)=__declspec_##x"); strarray_add(comp_args, "-D__declspec_align(x)=__attribute__((aligned(x)))"); strarray_add(comp_args, "-D__declspec_allocate(x)=__attribute__((section(x)))");

1 0

Alexandre Julliard : oleaut32: Return value pointer needs to be in x8 on ARM64.
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: d01470ad451b5aff7b1586f35ef90ff039458d02 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d01470ad451b5aff7b1586f3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 27 12:00:09 2019 +0200 oleaut32: Return value pointer needs to be in x8 on ARM64. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/oleaut32/typelib.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/dlls/oleaut32/typelib.c b/dlls/oleaut32/typelib.c index d40c0efb0f..7ad92837da 100644 --- a/dlls/oleaut32/typelib.c +++ b/dlls/oleaut32/typelib.c @@ -6823,7 +6823,7 @@ __ASM_GLOBAL_FUNC( call_method, "ldr x4, [x2, x1, lsl #3]\n\t" "str x4, [sp, x1, lsl #3]\n\t" "cbnz x1, 1b\n" - "2:\tmov x8, x0\n\t" + "2:\tmov x16, x0\n\t" "mov x9, x3\n\t" "ldp d0, d1, [x9]\n\t" "ldp d2, d3, [x9, #0x10]\n\t" @@ -6833,7 +6833,8 @@ __ASM_GLOBAL_FUNC( call_method, "ldp x2, x3, [x9, #0x50]\n\t" "ldp x4, x5, [x9, #0x60]\n\t" "ldp x6, x7, [x9, #0x70]\n\t" - "blr x8\n\t" + "ldr x8, [x9, #0x80]\n\t" + "blr x16\n\t" "mov sp, x29\n\t" "ldp x29, x30, [sp], #16\n\t" "ret" ) @@ -6856,7 +6857,7 @@ HRESULT WINAPI DispCallFunc( void *instance, ULONG_PTR offset, CALLCONV cc, VART float f; double d; } fp[8]; - DWORD_PTR x[8]; + DWORD_PTR x[9]; } regs; int rcount; /* 64-bit register index count */ int fpcount = 0; /* float register index count */ @@ -6887,7 +6888,7 @@ HRESULT WINAPI DispCallFunc( void *instance, ULONG_PTR offset, CALLCONV cc, VART { case VT_DECIMAL: case VT_VARIANT: - regs.x[rcount++] = (DWORD_PTR)result; /* arg 0 is a pointer to the result */ + regs.x[8] = (DWORD_PTR)result; /* x8 is a pointer to the result */ break; case VT_HRESULT: WARN("invalid return type %u\n", ret_type);

1 0

Alexandre Julliard : rpcrt4: Add stubless proxy support for ARM64.
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: d9adc902f88445fb1a33825775884de394124d71 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d9adc902f88445fb1a338257… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 27 12:39:49 2019 +0200 rpcrt4: Add stubless proxy support for ARM64. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/rpcrt4/cproxy.c | 44 ++++++++++++++++++++++++++++++++++++++++++++ dlls/rpcrt4/cstub.c | 17 +++++++++++++++++ dlls/rpcrt4/ndr_stubless.c | 29 +++++++++++++++++++++++++++++ 3 files changed, 90 insertions(+) diff --git a/dlls/rpcrt4/cproxy.c b/dlls/rpcrt4/cproxy.c index 3b9467735b..968c7b5156 100644 --- a/dlls/rpcrt4/cproxy.c +++ b/dlls/rpcrt4/cproxy.c @@ -196,6 +196,50 @@ static inline void init_thunk( struct thunk *thunk, unsigned int index ) thunk->func = call_stubless_func; } +#elif defined(__aarch64__) + +extern void call_stubless_func(void); +__ASM_GLOBAL_FUNC( call_stubless_func, + "stp x29, x30, [sp, #-0x90]!\n\t" + "mov x29, sp\n\t" + "stp d0, d1, [sp, #0x10]\n\t" + "stp d2, d3, [sp, #0x20]\n\t" + "stp d4, d5, [sp, #0x30]\n\t" + "stp d6, d7, [sp, #0x40]\n\t" + "stp x0, x1, [sp, #0x50]\n\t" + "stp x2, x3, [sp, #0x60]\n\t" + "stp x4, x5, [sp, #0x70]\n\t" + "stp x6, x7, [sp, #0x80]\n\t" + "ldr x0, [x0]\n\t" /* This->lpVtbl */ + "ldr x0, [x0, #-16]\n\t" /* MIDL_STUBLESS_PROXY_INFO */ + "ldp x1, x4, [x0, #8]\n\t" /* info->ProcFormatString, FormatStringOffset */ + "ldrh w4, [x4, x16, lsl #1]\n\t" /* info->FormatStringOffset[index] */ + "add x1, x1, x4\n\t" /* info->ProcFormatString + offset */ + "ldr x0, [x0]\n\t" /* info->pStubDesc */ + "add x2, sp, #0x50\n\t" /* stack */ + "add x3, sp, #0x10\n\t" /* fpu_stack */ + "bl " __ASM_NAME("ndr_client_call") "\n\t" + "ldp x29, x30, [sp], #0x90\n\t" + "ret" ) + +struct thunk +{ + DWORD ldr_index; /* ldr w16, index */ + DWORD ldr_func; /* ldr x17, func */ + DWORD br; /* br x17 */ + DWORD index; + void *func; +}; + +static inline void init_thunk( struct thunk *thunk, unsigned int index ) +{ + thunk->ldr_index = 0x18000070; /* ldr w16,index */ + thunk->ldr_func = 0x58000071; /* ldr x17,func */ + thunk->br = 0xd61f0220; /* br x17 */ + thunk->index = index; + thunk->func = call_stubless_func; +} + #else /* __i386__ */ #warning You must implement stubless proxies for your CPU diff --git a/dlls/rpcrt4/cstub.c b/dlls/rpcrt4/cstub.c index 607e9d4bc8..1903a5f0c9 100644 --- a/dlls/rpcrt4/cstub.c +++ b/dlls/rpcrt4/cstub.c @@ -183,6 +183,23 @@ typedef struct DWORD offset; } vtbl_method_t; +#elif defined(__aarch64__) + +static const DWORD opcodes[] = +{ + 0xf9401000, /* ldr x0, [x0,#32] */ + 0xf9400010, /* ldr x16, [x0] */ + 0x18000071, /* ldr w17, offset */ + 0xf8716a10, /* ldr x16, [x16,x17] */ + 0xd61f0200 /* br x16 */ +}; + +typedef struct +{ + DWORD opcodes[ARRAY_SIZE(opcodes)]; + DWORD offset; +} vtbl_method_t; + #else #warning You must implement delegated proxies/stubs for your CPU diff --git a/dlls/rpcrt4/ndr_stubless.c b/dlls/rpcrt4/ndr_stubless.c index bceecdc827..cc52ee27e8 100644 --- a/dlls/rpcrt4/ndr_stubless.c +++ b/dlls/rpcrt4/ndr_stubless.c @@ -1169,6 +1169,35 @@ __ASM_GLOBAL_FUNC( call_server_func, "5:\tblx r4\n\t" "mov SP, r5\n\t" "pop {r4, r5, PC}" ) +#elif defined __aarch64__ +LONG_PTR __cdecl call_server_func(SERVER_ROUTINE func, unsigned char *args, unsigned int stack_size); +__ASM_GLOBAL_FUNC( call_server_func, + "stp x29, x30, [sp, #-16]!\n\t" + "mov x29, sp\n\t" + "add x3, x2, #15\n\t" + "lsr x3, x3, #4\n\t" + "sub sp, sp, x3, lsl #4\n\t" + "cbz x2, 2f\n" + "1:\tsub x2, x2, #8\n\t" + "ldr x4, [x1, x2]\n\t" + "str x4, [sp, x2]\n\t" + "cbnz x2, 1b\n" + "2:\tmov x8, x0\n\t" + "cbz x3, 3f\n\t" + "ldp x0, x1, [sp], #16\n\t" + "cmp x3, #1\n\t" + "b.le 3f\n\t" + "ldp x2, x3, [sp], #16\n\t" + "cmp x3, #2\n\t" + "b.le 3f\n\t" + "ldp x4, x5, [sp], #16\n\t" + "cmp x3, #3\n\t" + "b.le 3f\n\t" + "ldp x6, x7, [sp], #16\n" + "3:\tblr x8\n\t" + "mov sp, x29\n\t" + "ldp x29, x30, [sp], #16\n\t" + "ret" ) #else #warning call_server_func not implemented for your architecture LONG_PTR __cdecl call_server_func(SERVER_ROUTINE func, unsigned char * args, unsigned short stack_size)

1 0

Alexandre Julliard : winegcc: Centralize argument parsing into the main switch.
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: 0bfee2c6eb3aa9a688e9493eff7565b5cdafb8f0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0bfee2c6eb3aa9a688e9493e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 27 13:25:59 2019 +0200 winegcc: Centralize argument parsing into the main switch. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/winegcc/winegcc.c | 90 ++++++++++++++++++------------------------------- 1 file changed, 33 insertions(+), 57 deletions(-) diff --git a/tools/winegcc/winegcc.c b/tools/winegcc/winegcc.c index 6f26f60709..0ca553318e 100644 --- a/tools/winegcc/winegcc.c +++ b/tools/winegcc/winegcc.c @@ -1304,7 +1304,6 @@ static int is_linker_arg(const char* arg) { case 'R': case 'z': - case 'l': case 'u': return 1; case 'W': @@ -1324,43 +1323,6 @@ static int is_linker_arg(const char* arg) return 0; } -/* - * Target Options - * -b machine -V version - */ -static int is_target_arg(const char* arg) -{ - return arg[1] == 'b' || arg[1] == 'V'; -} - - -/* - * Directory Options - * -Bprefix -Idir -I- -Ldir -specs=file - */ -static int is_directory_arg(const char* arg) -{ - return arg[1] == 'B' || arg[1] == 'L' || arg[1] == 'I' || strncmp("-specs=", arg, 7) == 0; -} - -/* - * MinGW Options - * -mno-cygwin -mwindows -mconsole -mthreads -municode - */ -static int is_mingw_arg(const char* arg) -{ - static const char* mingw_switches[] = - { - "-mno-cygwin", "-mwindows", "-mconsole", "-mthreads", "-municode" - }; - unsigned int j; - - for (j = 0; j < ARRAY_SIZE(mingw_switches); j++) - if (strcmp(mingw_switches[j], arg) == 0) return 1; - - return 0; -} - static void parse_target_option( struct options *opts, const char *target ) { char *p, *platform, *spec = xstrdup( target ); @@ -1524,25 +1486,8 @@ int main(int argc, char **argv) } /* determine what options go 'as is' to the linker & the compiler */ - raw_compiler_arg = raw_linker_arg = 0; - if (is_linker_arg(argv[i])) - { - raw_linker_arg = 1; - } - else - { - if (is_directory_arg(argv[i]) || is_target_arg(argv[i])) - raw_linker_arg = 1; - raw_compiler_arg = !is_mingw_arg(argv[i]); - } - - /* these things we handle explicitly so we don't pass them 'as is' */ - if (argv[i][1] == 'l' || argv[i][1] == 'I' || argv[i][1] == 'L') - raw_linker_arg = 0; - if (argv[i][1] == 'c' || argv[i][1] == 'L') - raw_compiler_arg = 0; - if (argv[i][1] == 'o' || argv[i][1] == 'b' || argv[i][1] == 'V') - raw_compiler_arg = raw_linker_arg = 0; + raw_linker_arg = is_linker_arg(argv[i]); + raw_compiler_arg = !raw_linker_arg; /* do a bit of semantic analysis */ switch (argv[i][1]) @@ -1552,14 +1497,18 @@ int main(int argc, char **argv) if (strendswith(str, "/")) str[strlen(str) - 1] = 0; if (!opts.prefix) opts.prefix = strarray_alloc(); strarray_add(opts.prefix, str); + raw_linker_arg = 1; break; case 'b': parse_target_option( &opts, option_arg ); + raw_compiler_arg = 0; break; case 'V': opts.version = xstrdup( option_arg ); + raw_compiler_arg = 0; break; case 'c': /* compile or assemble */ + raw_compiler_arg = 0; if (argv[i][2] == 0) opts.compile_only = 1; /* fall through */ case 'S': /* generate assembler code */ @@ -1583,24 +1532,45 @@ int main(int argc, char **argv) break; case 'l': strarray_add(opts.files, strmake("-l%s", option_arg)); + raw_compiler_arg = 0; break; case 'L': strarray_add(opts.lib_dirs, option_arg); + raw_compiler_arg = 0; break; case 'M': /* map file generation */ linking = 0; break; case 'm': if (strcmp("-mno-cygwin", argv[i]) == 0) + { opts.use_msvcrt = 1; + raw_compiler_arg = 0; + } else if (strcmp("-mwindows", argv[i]) == 0) + { opts.gui_app = 1; + raw_compiler_arg = 0; + } else if (strcmp("-mconsole", argv[i]) == 0) + { opts.gui_app = 0; + raw_compiler_arg = 0; + } else if (strcmp("-municode", argv[i]) == 0) + { opts.unicode_app = 1; + raw_compiler_arg = 0; + } + else if (strcmp("-mthreads", argv[i]) == 0) + { + raw_compiler_arg = 0; + } else if (strcmp("-m16", argv[i]) == 0) + { opts.win16_app = 1; + raw_compiler_arg = 0; + } else if (strcmp("-m32", argv[i]) == 0) { if (opts.target_cpu == CPU_x86_64) @@ -1642,12 +1612,15 @@ int main(int argc, char **argv) break; case 'o': opts.output_name = option_arg; + raw_compiler_arg = 0; break; case 's': if (strcmp("-static", argv[i]) == 0) linking = -1; else if(strcmp("-save-temps", argv[i]) == 0) keep_generated = 1; + else if (strncmp("-specs=", argv[i], 7) == 0) + raw_linker_arg = 1; else if(strcmp("-shared", argv[i]) == 0) { opts.shared = 1; @@ -1723,7 +1696,10 @@ int main(int argc, char **argv) if (strcmp("-static", argv[i]+1) == 0) linking = -1; else if (is_option( argv, i, "--sysroot", &option_arg )) + { opts.sysroot = option_arg; + raw_linker_arg = 1; + } else if (is_option( argv, i, "--target", &option_arg )) { parse_target_option( &opts, option_arg );

1 0

Alexandre Julliard : kernelbase: Use set_ntstatus() in more places.
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: 31d522ea53e5cfeb426e88e5badd3864be5b047a URL: https://source.winehq.org/git/wine.git/?a=commit;h=31d522ea53e5cfeb426e88e5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 27 12:42:16 2019 +0200 kernelbase: Use set_ntstatus() in more places. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/loader.c | 30 +++++-------------- dlls/kernelbase/main.c | 14 ++------- dlls/kernelbase/memory.c | 9 ++---- dlls/kernelbase/sync.c | 78 ++++++++---------------------------------------- dlls/kernelbase/thread.c | 47 +++++++++++------------------ 5 files changed, 43 insertions(+), 135 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=31d522ea53e5cfeb426e…

1 0

Zebediah Figura : strmbase: Don't execute OnStartStreaming() or OnStopStreaming() callbacks if the sink is unconnected.
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: 802ebaf9dbe781db73ac1b9373b659aad687fd45 URL: https://source.winehq.org/git/wine.git/?a=commit;h=802ebaf9dbe781db73ac1b93… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Thu Sep 26 21:40:48 2019 -0500 strmbase: Don't execute OnStartStreaming() or OnStopStreaming() callbacks if the sink is unconnected. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/strmbase/renderer.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/strmbase/renderer.c b/dlls/strmbase/renderer.c index 5634ab1905..051230186d 100644 --- a/dlls/strmbase/renderer.c +++ b/dlls/strmbase/renderer.c @@ -407,7 +407,7 @@ HRESULT WINAPI BaseRendererImpl_Stop(IBaseFilter * iface) EnterCriticalSection(&This->csRenderLock); { RendererPosPassThru_ResetMediaTime(This->pPosition); - if (This->pFuncsTable->renderer_stop_stream) + if (This->sink.pin.pConnectedTo && This->pFuncsTable->renderer_stop_stream) This->pFuncsTable->renderer_stop_stream(This); This->filter.state = State_Stopped; SetEvent(This->state_event); @@ -449,7 +449,7 @@ HRESULT WINAPI BaseRendererImpl_Run(IBaseFilter * iface, REFERENCE_TIME tStart) if (SUCCEEDED(hr)) { QualityControlRender_Start(This->qcimpl, This->filter.rtStreamStart); - if (This->pFuncsTable->renderer_start_stream) + if (This->sink.pin.pConnectedTo && This->pFuncsTable->renderer_start_stream) This->pFuncsTable->renderer_start_stream(This); if (This->filter.state == State_Stopped) BaseRendererImpl_ClearPendingSample(This); @@ -477,7 +477,7 @@ HRESULT WINAPI BaseRendererImpl_Pause(IBaseFilter * iface) ResetEvent(This->state_event); This->sink.end_of_stream = FALSE; } - else if (This->pFuncsTable->renderer_stop_stream) + else if (This->sink.pin.pConnectedTo && This->pFuncsTable->renderer_stop_stream) This->pFuncsTable->renderer_stop_stream(This); if (This->filter.state == State_Stopped)

1 0

Zebediah Figura : quartz/tests: Add some tests for unconnected filter state change.
by Alexandre Julliard 27 Sep '19

27 Sep '19

Module: wine Branch: master Commit: c08ab0b941db2cf47a9e93311bfa690f020cb32d URL: https://source.winehq.org/git/wine.git/?a=commit;h=c08ab0b941db2cf47a9e9331… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Thu Sep 26 21:40:49 2019 -0500 quartz/tests: Add some tests for unconnected filter state change. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/quartz/tests/acmwrapper.c | 58 +++++++++++++++++++++++++++++++++++++ dlls/quartz/tests/avidec.c | 58 +++++++++++++++++++++++++++++++++++++ dlls/quartz/tests/avisplit.c | 58 +++++++++++++++++++++++++++++++++++++ dlls/quartz/tests/dsoundrender.c | 58 +++++++++++++++++++++++++++++++++++++ dlls/quartz/tests/filesource.c | 61 +++++++++++++++++---------------------- dlls/quartz/tests/mpegsplit.c | 58 +++++++++++++++++++++++++++++++++++++ dlls/quartz/tests/videorenderer.c | 58 +++++++++++++++++++++++++++++++++++++ dlls/quartz/tests/vmr7.c | 58 +++++++++++++++++++++++++++++++++++++ dlls/quartz/tests/vmr9.c | 58 +++++++++++++++++++++++++++++++++++++ dlls/quartz/tests/waveparser.c | 58 +++++++++++++++++++++++++++++++++++++ 10 files changed, 549 insertions(+), 34 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=c08ab0b941db2cf47a9e…

1 0