- Wine-Commits - list.winehq.org

Alexandre Julliard : ws2_32/tests: Fix a couple of failures on Windows.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: 66f940a2eccbf00c08d3bc08467bc18d321868f7 URL: https://gitlab.winehq.org/wine/wine/-/commit/66f940a2eccbf00c08d3bc08467bc1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Mar 12 08:42:51 2024 +0100 ws2_32/tests: Fix a couple of failures on Windows. --- dlls/ws2_32/tests/protocol.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/dlls/ws2_32/tests/protocol.c b/dlls/ws2_32/tests/protocol.c index 8a1efb6b5d0..e1a431a3e32 100644 --- a/dlls/ws2_32/tests/protocol.c +++ b/dlls/ws2_32/tests/protocol.c @@ -1917,8 +1917,8 @@ static void test_GetAddrInfoW(void) SetLastError(0xdeadbeef); result2 = NULL; ret = GetAddrInfoW(idn_domain, NULL, &hint, &result2); - ok(ret == WSAHOST_NOT_FOUND, "got %d expected WSAHOST_NOT_FOUND\n", ret); - ok(WSAGetLastError() == WSAHOST_NOT_FOUND, "expected 11001, got %d\n", WSAGetLastError()); + ok(ret == WSAHOST_NOT_FOUND || ret == WSATRY_AGAIN, "got %d\n", ret); + ok(WSAGetLastError() == ret, "got %d\n", WSAGetLastError()); ok(result2 == NULL, "got %p\n", result2); } @@ -1926,6 +1926,7 @@ static struct completion_routine_test { WSAOVERLAPPED *overlapped; DWORD error; + DWORD error2; ADDRINFOEXW **result; HANDLE event; DWORD called; @@ -1935,7 +1936,7 @@ static void CALLBACK completion_routine(DWORD error, DWORD byte_count, WSAOVERLA { struct completion_routine_test *test = &completion_routine_test; - ok(error == test->error, "got %lu\n", error); + ok(error == test->error || (test->error2 && error == test->error2), "got %lu\n", error); ok(!byte_count, "got %lu\n", byte_count); ok(overlapped == test->overlapped, "got %p\n", overlapped); ok(overlapped->Internal == test->error, "got %Iu\n", overlapped->Internal); @@ -2073,6 +2074,7 @@ static void test_GetAddrInfoExW(void) overlapped.hEvent = NULL; completion_routine_test.overlapped = &overlapped; completion_routine_test.error = ERROR_SUCCESS; + completion_routine_test.error2 = ERROR_SUCCESS; completion_routine_test.result = &result; completion_routine_test.event = event; completion_routine_test.called = 0; @@ -2093,6 +2095,7 @@ static void test_GetAddrInfoExW(void) result = (void *)0xdeadbeef; completion_routine_test.overlapped = &overlapped; completion_routine_test.error = WSAHOST_NOT_FOUND; + completion_routine_test.error2 = WSANO_DATA; completion_routine_test.called = 0; ResetEvent(event); ret = pGetAddrInfoExW(nxdomain, NULL, NS_DNS, NULL, NULL, &result, NULL, &overlapped, completion_routine, NULL);

1 0

Brendan McGrath : comdlg32: Don't treat cmb1 as the printer list unless in PRINT_SETUP.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: bbdc50c5320dc1c51c080e24bf7ec712c7b847c0 URL: https://gitlab.winehq.org/wine/wine/-/commit/bbdc50c5320dc1c51c080e24bf7ec7… Author: Brendan McGrath <bmcgrath(a)codeweavers.com> Date: Tue Jan 30 06:37:13 2024 +1100 comdlg32: Don't treat cmb1 as the printer list unless in PRINT_SETUP. This changes WMCommandW to match WMCommandA. --- dlls/comdlg32/printdlg.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/dlls/comdlg32/printdlg.c b/dlls/comdlg32/printdlg.c index d300b46a5b6..aed486a9b09 100644 --- a/dlls/comdlg32/printdlg.c +++ b/dlls/comdlg32/printdlg.c @@ -1988,7 +1988,10 @@ static LRESULT PRINTDLG_WMCommandW(HWND hDlg, WPARAM wParam, } break; - case cmb1: /* Printer Combobox in PRINT SETUP */ + case cmb1: /* Printer Combobox in PRINT SETUP, quality combobox in PRINT16 */ + if (PrinterComboID != LOWORD(wParam)) { + break; + } /* FALLTHROUGH */ case cmb4: /* Printer combobox */ if (HIWORD(wParam)==CBN_SELCHANGE) {

1 0

Alexandre Julliard : ntdll/tests: Fix a backtrace test failure on Windows ARM64.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: ffade87df1f851867d0a33a80d8d7fec9e68e6ea URL: https://gitlab.winehq.org/wine/wine/-/commit/ffade87df1f851867d0a33a80d8d7f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Mar 12 20:49:54 2024 +0100 ntdll/tests: Fix a backtrace test failure on Windows ARM64. --- dlls/ntdll/tests/exception.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/ntdll/tests/exception.c b/dlls/ntdll/tests/exception.c index 01177a2c5d7..9ce0646cd47 100644 --- a/dlls/ntdll/tests/exception.c +++ b/dlls/ntdll/tests/exception.c @@ -10294,7 +10294,7 @@ static void test_backtrace(void) ULONG hash, hash_expect; int i, count = RtlCaptureStackBackTrace( 0, 1024, buffer, &hash ); - ok( count > 1, "got %u entries\n", count ); + ok( count > 0, "got %u entries\n", count ); for (i = hash_expect = 0; i < count; i++) hash_expect += (ULONG_PTR)buffer[i]; ok( hash == hash_expect, "hash mismatch %lx / %lx\n", hash, hash_expect ); RtlPcToFileHeader( buffer[0], &module ); @@ -10302,6 +10302,7 @@ static void test_backtrace(void) module, GetModuleHandleA(0), buffer[0]); if (count && !buffer[count - 1]) count--; /* win11 32-bit */ + if (count <= 1) return; RtlPcToFileHeader( buffer[count - 1], &module ); GetModuleFileNameW( module, name, ARRAY_SIZE(name) ); if ((p = wcsrchr( name, '\\' ))) p++;

1 0

Dmitry Timoshkov : d2d1: Make some strings const.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: 354ddcb382f7dffe515fa72a961b5910839b60dc URL: https://gitlab.winehq.org/wine/wine/-/commit/354ddcb382f7dffe515fa72a961b59… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Thu Feb 29 11:01:24 2024 +0300 d2d1: Make some strings const. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> --- dlls/d2d1/effect.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/d2d1/effect.c b/dlls/d2d1/effect.c index 755b2821df4..8dc006c1293 100644 --- a/dlls/d2d1/effect.c +++ b/dlls/d2d1/effect.c @@ -211,7 +211,7 @@ static HRESULT STDMETHODCALLTYPE builtin_factory_stub(IUnknown **effect_impl) return S_OK; } -static const WCHAR * const _2d_affine_transform_description = +static const WCHAR _2d_affine_transform_description[] = L"<?xml version='1.0'?> \ <Effect> \ <Property name='DisplayName' type='string' value='2D Affine Transform'/> \ @@ -223,7 +223,7 @@ L"<?xml version='1.0'?> \ </Inputs> \ </Effect>"; -static const WCHAR * const _3d_perspective_transform_description = +static const WCHAR _3d_perspective_transform_description[] = L"<?xml version='1.0'?> \ <Effect> \ <Property name='DisplayName' type='string' value='3D Perspective Transform'/> \ @@ -235,7 +235,7 @@ L"<?xml version='1.0'?> </Inputs> \ </Effect>"; -static const WCHAR * const composite_description = +static const WCHAR composite_description[] = L"<?xml version='1.0'?> \ <Effect> \ <Property name='DisplayName' type='string' value='Composite'/> \ @@ -248,7 +248,7 @@ L"<?xml version='1.0'?> \ </Inputs> \ </Effect>"; -static const WCHAR * const crop_description = +static const WCHAR crop_description[] = L"<?xml version='1.0'?> \ <Effect> \ <Property name='DisplayName' type='string' value='Crop'/> \ @@ -260,7 +260,7 @@ L"<?xml version='1.0'?> \ </Inputs> \ </Effect>"; -static const WCHAR * const shadow_description = +static const WCHAR shadow_description[] = L"<?xml version='1.0'?> \ <Effect> \ <Property name='DisplayName' type='string' value='Shadow'/> \ @@ -272,7 +272,7 @@ L"<?xml version='1.0'?> \ </Inputs> \ </Effect>"; -static const WCHAR * const grayscale_description = +static const WCHAR grayscale_description[] = L"<?xml version='1.0'?> \ <Effect> \ <Property name='DisplayName' type='string' value='Grayscale'/> \

1 0

Alexandre Julliard : include: Define setjmpex prototype even when it's a builtin.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: fd865a838695157affe5a5d18c63d416b0d0149f URL: https://gitlab.winehq.org/wine/wine/-/commit/fd865a838695157affe5a5d18c63d4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Mar 12 13:42:00 2024 +0100 include: Define setjmpex prototype even when it's a builtin. --- include/msvcrt/setjmp.h | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/include/msvcrt/setjmp.h b/include/msvcrt/setjmp.h index 38f9ba682ad..fca5543a04b 100644 --- a/include/msvcrt/setjmp.h +++ b/include/msvcrt/setjmp.h @@ -164,10 +164,11 @@ _ACRTIMP void __cdecl longjmp(jmp_buf,int); _ACRTIMP int __cdecl __attribute__((__nothrow__,__returns_twice__)) _setjmp(jmp_buf); # endif # define setjmp(buf) _setjmp((buf)) -#else /* __i386__ */ -# if defined(_setjmpex) || !__has_builtin(_setjmpex) +#elif !defined(_setjmpex) && __has_builtin(_setjmpex) +_ACRTIMP int __cdecl __attribute__((__nothrow__,__returns_twice__)) _setjmpex(jmp_buf); +# define setjmp(buf) _setjmpex(buf) +#else _ACRTIMP int __cdecl __attribute__((__nothrow__,__returns_twice__)) _setjmpex(jmp_buf,void*); -# endif # if __has_builtin(__builtin_sponentry) # define setjmp(buf) _setjmpex((buf), __builtin_sponentry()) # elif __has_builtin(__builtin_frame_address)

1 0

Brendan McGrath : comdlg32: Populate printer name on the print dialogs.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: 91207991f2c64ed9f5cc564926c5b0e528712e44 URL: https://gitlab.winehq.org/wine/wine/-/commit/91207991f2c64ed9f5cc564926c5b0… Author: Brendan McGrath <bmcgrath(a)codeweavers.com> Date: Tue Jan 23 13:24:01 2024 +1100 comdlg32: Populate printer name on the print dialogs. This will display the name of the selected printer on the Print dialog and the default printer on the Print Setup dialog --- dlls/comdlg32/printdlg.c | 34 +++++++++++++++++++++++++++++----- 1 file changed, 29 insertions(+), 5 deletions(-) diff --git a/dlls/comdlg32/printdlg.c b/dlls/comdlg32/printdlg.c index b777a83825f..e90a9e00efb 100644 --- a/dlls/comdlg32/printdlg.c +++ b/dlls/comdlg32/printdlg.c @@ -978,10 +978,11 @@ static BOOL PRINTDLG_SetUpPaperComboBoxW(HWND hDlg, /*********************************************************************** * PRINTDLG_UpdatePrinterInfoTexts [internal] */ -static void PRINTDLG_UpdatePrinterInfoTextsA(HWND hDlg, const PRINTER_INFO_2A *pi) +static void PRINTDLG_UpdatePrinterInfoTextsA(HWND hDlg, DWORD flags, const PRINTER_INFO_2A *pi) { char StatusMsg[256]; char ResourceString[256]; + char printer_name[256]; int i; /* Status Message */ @@ -1004,6 +1005,17 @@ static void PRINTDLG_UpdatePrinterInfoTextsA(HWND hDlg, const PRINTER_INFO_2A *p SetDlgItemTextA(hDlg, stc12, StatusMsg); /* set all other printer info texts */ + if (flags & PD_PRINTSETUP) + { + DWORD dwBufLen = ARRAY_SIZE(printer_name); + GetDefaultPrinterA(printer_name, &dwBufLen); + } + else + { + /* FIXME: Windows decorates the printer name with text like 'System Printer' or 'on <port>'. */ + lstrcpynA(printer_name, pi->pPrinterName, ARRAY_SIZE(printer_name)); + } + SetDlgItemTextA(hDlg, stc1, printer_name); SetDlgItemTextA(hDlg, stc11, pi->pDriverName); if (pi->pLocation != NULL && pi->pLocation[0] != '\0') @@ -1014,11 +1026,12 @@ static void PRINTDLG_UpdatePrinterInfoTextsA(HWND hDlg, const PRINTER_INFO_2A *p return; } -static void PRINTDLG_UpdatePrinterInfoTextsW(HWND hDlg, const PRINTER_INFO_2W *pi) +static void PRINTDLG_UpdatePrinterInfoTextsW(HWND hDlg, DWORD flags, const PRINTER_INFO_2W *pi) { WCHAR StatusMsg[256]; WCHAR ResourceString[256]; - int i; + WCHAR printer_name[256]; + int i; /* Status Message */ StatusMsg[0]='\0'; @@ -1040,6 +1053,17 @@ static void PRINTDLG_UpdatePrinterInfoTextsW(HWND hDlg, const PRINTER_INFO_2W *p SetDlgItemTextW(hDlg, stc12, StatusMsg); /* set all other printer info texts */ + if (flags & PD_PRINTSETUP) + { + DWORD dwBufLen = ARRAY_SIZE(printer_name); + GetDefaultPrinterW(printer_name, &dwBufLen); + } + else + { + /* FIXME: Windows decorates the printer name with text like 'System Printer' or 'on <port>'. */ + lstrcpynW(printer_name, pi->pPrinterName, ARRAY_SIZE(printer_name)); + } + SetDlgItemTextW(hDlg, stc1, printer_name); SetDlgItemTextW(hDlg, stc11, pi->pDriverName); if (pi->pLocation != NULL && pi->pLocation[0] != '\0') SetDlgItemTextW(hDlg, stc14, pi->pLocation); @@ -1081,7 +1105,7 @@ static BOOL PRINTDLG_ChangePrinterA(HWND hDlg, char *name, PRINT_PTRA *PrintStru } ClosePrinter(hprn); - PRINTDLG_UpdatePrinterInfoTextsA(hDlg, PrintStructures->lpPrinterInfo); + PRINTDLG_UpdatePrinterInfoTextsA(hDlg, lppd->Flags, PrintStructures->lpPrinterInfo); free(PrintStructures->lpDevMode); PrintStructures->lpDevMode = NULL; @@ -1288,7 +1312,7 @@ static BOOL PRINTDLG_ChangePrinterW(HWND hDlg, WCHAR *name, } ClosePrinter(hprn); - PRINTDLG_UpdatePrinterInfoTextsW(hDlg, PrintStructures->lpPrinterInfo); + PRINTDLG_UpdatePrinterInfoTextsW(hDlg, lppd->Flags, PrintStructures->lpPrinterInfo); free(PrintStructures->lpDevMode); PrintStructures->lpDevMode = NULL;

1 0

Brendan McGrath : comdlg32: Use ANSI functions in ANSI WMCommandA.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: 39bd338a626accc8c1302b3bf04c4da8700d6dd4 URL: https://gitlab.winehq.org/wine/wine/-/commit/39bd338a626accc8c1302b3bf04c4d… Author: Brendan McGrath <bmcgrath(a)codeweavers.com> Date: Tue Jan 30 06:34:47 2024 +1100 comdlg32: Use ANSI functions in ANSI WMCommandA. --- dlls/comdlg32/printdlg.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/comdlg32/printdlg.c b/dlls/comdlg32/printdlg.c index e90a9e00efb..d300b46a5b6 100644 --- a/dlls/comdlg32/printdlg.c +++ b/dlls/comdlg32/printdlg.c @@ -1840,8 +1840,8 @@ static LRESULT PRINTDLG_WMCommandA(HWND hDlg, WPARAM wParam, case cmb4: /* Printer combobox */ if (HIWORD(wParam)==CBN_SELCHANGE) { char *PrinterName; - INT index = SendDlgItemMessageW(hDlg, LOWORD(wParam), CB_GETCURSEL, 0, 0); - INT length = SendDlgItemMessageW(hDlg, LOWORD(wParam), CB_GETLBTEXTLEN, index, 0); + INT index = SendDlgItemMessageA(hDlg, LOWORD(wParam), CB_GETCURSEL, 0, 0); + INT length = SendDlgItemMessageA(hDlg, LOWORD(wParam), CB_GETLBTEXTLEN, index, 0); PrinterName = malloc(length + 1); SendDlgItemMessageA(hDlg, LOWORD(wParam), CB_GETLBTEXT, index, (LPARAM)PrinterName); PRINTDLG_ChangePrinterA(hDlg, PrinterName, PrintStructures);

1 0

Brendan McGrath : comdlg32: Use values from DeviceCapabilities in combobox.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: 4ec559743d254e3547c0cabdcb1014854404e108 URL: https://gitlab.winehq.org/wine/wine/-/commit/4ec559743d254e3547c0cabdcb1014… Author: Brendan McGrath <bmcgrath(a)codeweavers.com> Date: Tue Jan 30 06:05:23 2024 +1100 comdlg32: Use values from DeviceCapabilities in combobox. dpiX,dpiY are the default printer values from the GetDeviceCaps calls. This causes the item data to be the same for every option in the combobox. The item data should reflect the selected values. --- dlls/comdlg32/printdlg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/comdlg32/printdlg.c b/dlls/comdlg32/printdlg.c index 700119bc9ff..2a2b7e9d1b2 100644 --- a/dlls/comdlg32/printdlg.c +++ b/dlls/comdlg32/printdlg.c @@ -1229,7 +1229,7 @@ static BOOL PRINTDLG_ChangePrinterA(HWND hDlg, char *name, PRINT_PTRA *PrintStru if(IsDefault) SendMessageA(hQuality, CB_SETCURSEL, Index, 0); - SendMessageA(hQuality, CB_SETITEMDATA, Index, MAKELONG(dpiX,dpiY)); + SendMessageA(hQuality, CB_SETITEMDATA, Index, MAKELONG(Resolutions[i], Resolutions[i+1])); } free(Resolutions); }

1 0

Gabriel Ivăncescu : shell32: Construct the proper path target for UnixFolder.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: a9b3fdd3aebe0ef2f7f7905864e7c4fe01f5c4fd URL: https://gitlab.winehq.org/wine/wine/-/commit/a9b3fdd3aebe0ef2f7f7905864e7c4… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Mon Mar 4 21:52:06 2024 +0200 shell32: Construct the proper path target for UnixFolder. Worse than just no unix integration is that it can actually crash if you parse e.g. "/". Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- dlls/shell32/shfldr_fs.c | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/dlls/shell32/shfldr_fs.c b/dlls/shell32/shfldr_fs.c index 808b20c9cae..928cadfc77c 100644 --- a/dlls/shell32/shfldr_fs.c +++ b/dlls/shell32/shfldr_fs.c @@ -1797,7 +1797,7 @@ static const IDropTargetVtbl dtvt = { ISFDropTarget_Drop }; -static HRESULT create_fs( IUnknown *outer_unk, REFIID riid, void **ppv, const CLSID *clsid) +static HRESULT create_fs( IUnknown *outer_unk, REFIID riid, void **ppv, const CLSID *clsid, const WCHAR *path_target) { IGenericSFImpl *sf; HRESULT hr; @@ -1820,6 +1820,16 @@ static HRESULT create_fs( IUnknown *outer_unk, REFIID riid, void **ppv, const CL sf->ISFHelper_iface.lpVtbl = &shvt; sf->pclsid = clsid; sf->outer_unk = outer_unk ? outer_unk : &sf->IUnknown_inner; + if (path_target) + { + SIZE_T size = (wcslen(path_target) + 1) * sizeof(WCHAR); + if (!(sf->sPathTarget = SHAlloc(size))) + { + LocalFree(sf); + return E_OUTOFMEMORY; + } + memcpy(sf->sPathTarget, path_target, size); + } hr = IUnknown_QueryInterface(&sf->IUnknown_inner, riid, ppv); IUnknown_Release(&sf->IUnknown_inner); @@ -1830,25 +1840,25 @@ static HRESULT create_fs( IUnknown *outer_unk, REFIID riid, void **ppv, const CL HRESULT WINAPI IFSFolder_Constructor(IUnknown *outer_unk, REFIID riid, void **ppv) { - return create_fs( outer_unk, riid, ppv, &CLSID_ShellFSFolder ); + return create_fs( outer_unk, riid, ppv, &CLSID_ShellFSFolder, NULL ); } HRESULT WINAPI UnixFolder_Constructor(IUnknown *outer_unk, REFIID riid, void **ppv) { - return create_fs( outer_unk, riid, ppv, &CLSID_UnixFolder ); + return create_fs( outer_unk, riid, ppv, &CLSID_UnixFolder, L"\\\\?\\unix\\" ); } HRESULT WINAPI UnixDosFolder_Constructor(IUnknown *outer_unk, REFIID riid, void **ppv) { - return create_fs( outer_unk, riid, ppv, &CLSID_UnixDosFolder ); + return create_fs( outer_unk, riid, ppv, &CLSID_UnixDosFolder, NULL ); } HRESULT WINAPI FolderShortcut_Constructor(IUnknown *outer_unk, REFIID riid, void **ppv) { - return create_fs( outer_unk, riid, ppv, &CLSID_FolderShortcut ); + return create_fs( outer_unk, riid, ppv, &CLSID_FolderShortcut, NULL ); } HRESULT WINAPI MyDocuments_Constructor(IUnknown *outer_unk, REFIID riid, void **ppv) { - return create_fs( outer_unk, riid, ppv, &CLSID_MyDocuments ); + return create_fs( outer_unk, riid, ppv, &CLSID_MyDocuments, NULL ); }

1 0

Nikolay Sivov : d3d10/tests: Add a small effect compilation test.
by Alexandre Julliard March 12, 2024

March 12, 2024

Module: wine Branch: master Commit: 40afa8d937e034bf4183a9b6773c5ce449e12ec9 URL: https://gitlab.winehq.org/wine/wine/-/commit/40afa8d937e034bf4183a9b6773c5c… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Mar 10 11:37:59 2024 +0100 d3d10/tests: Add a small effect compilation test. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/d3d10/tests/effect.c | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/dlls/d3d10/tests/effect.c b/dlls/d3d10/tests/effect.c index a9f4c1e47e5..0935541ced0 100644 --- a/dlls/d3d10/tests/effect.c +++ b/dlls/d3d10/tests/effect.c @@ -9875,6 +9875,36 @@ static void test_effect_fx_4_1_blend_state(void) ok(!refcount, "Device has %lu references left.\n", refcount); } +static void test_effect_compiler(void) +{ + static char empty_effect[] = "technique10 {};"; + D3D10_EFFECT_DESC desc; + ID3D10Device *device; + ID3D10Effect *effect; + ID3D10Blob *blob; + HRESULT hr; + + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } + + hr = D3D10CompileEffectFromMemory(empty_effect, sizeof(empty_effect), NULL, NULL, NULL, 0, 0, + &blob, NULL); + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); + + hr = create_effect(ID3D10Blob_GetBufferPointer(blob), 0, device, NULL, &effect); + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); + hr = ID3D10Effect_GetDesc(effect, &desc); + ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); + ok(desc.Techniques == 1, "Unexpected technique count %u.\n", desc.Techniques); + ID3D10Effect_Release(effect); + + ID3D10Device_Release(device); + ID3D10Blob_Release(blob); +} + START_TEST(effect) { test_effect_constant_buffer_type(); @@ -9903,4 +9933,5 @@ START_TEST(effect) test_effect_value_expression(); test_effect_fx_4_1(); test_effect_fx_4_1_blend_state(); + test_effect_compiler(); }

1 0