March 2022 - Wine-commits - list.winehq.org

Rémi Bernon : quartz: Use the right VideoRenderer pin names.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: af3fcc3f3ef14eb25781855f63ca44d1e1383aa9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=af3fcc3f3ef14eb25781855f… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri Mar 25 09:42:08 2022 +0100 quartz: Use the right VideoRenderer pin names. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/quartz/tests/videorenderer.c | 2 +- dlls/quartz/videorenderer.c | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/quartz/tests/videorenderer.c b/dlls/quartz/tests/videorenderer.c index c1a3266197b..282d1ef7b12 100644 --- a/dlls/quartz/tests/videorenderer.c +++ b/dlls/quartz/tests/videorenderer.c @@ -347,7 +347,7 @@ static void test_pin_info(void) ok(hr == S_OK, "Got hr %#lx.\n", hr); ok(info.pFilter == filter, "Expected filter %p, got %p.\n", filter, info.pFilter); ok(info.dir == PINDIR_INPUT, "Got direction %d.\n", info.dir); - todo_wine ok(!wcscmp(info.achName, L"Input"), "Got name %s.\n", debugstr_w(info.achName)); + ok(!wcscmp(info.achName, L"Input"), "Got name %s.\n", debugstr_w(info.achName)); ref = get_refcount(filter); ok(ref == 3, "Got unexpected refcount %ld.\n", ref); ref = get_refcount(pin); diff --git a/dlls/quartz/videorenderer.c b/dlls/quartz/videorenderer.c index f44ce35dc59..f725a9998ce 100644 --- a/dlls/quartz/videorenderer.c +++ b/dlls/quartz/videorenderer.c @@ -481,6 +481,8 @@ HRESULT video_renderer_create(IUnknown *outer, IUnknown **out) return E_OUTOFMEMORY; strmbase_renderer_init(&object->renderer, outer, &CLSID_VideoRenderer, L"In", &renderer_ops); + wcscpy(object->renderer.sink.pin.name, L"Input"); + object->IOverlay_iface.lpVtbl = &overlay_vtbl; video_window_init(&object->window, &IVideoWindow_VTable,

1 0

Dmitry Timoshkov : kernel32/tests: Add a test to show that services.exe is supposed to run in the session 0.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: edb5762b55f04b3322972cc400bec1e973c03ae5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=edb5762b55f04b3322972cc4… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Fri Mar 25 18:40:29 2022 +0300 kernel32/tests: Add a test to show that services.exe is supposed to run in the session 0. IsWindowsService() from Microsoft.Extensions.Hosting.WindowsServices (.Net 4.6, 4.8, 5.0) depends on this. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/process.c | 35 +++++++++++++++++++++++++++++++++++ 1 file changed, 35 insertions(+) diff --git a/dlls/kernel32/tests/process.c b/dlls/kernel32/tests/process.c index e9e8555c32e..8f18a0d37a7 100644 --- a/dlls/kernel32/tests/process.c +++ b/dlls/kernel32/tests/process.c @@ -4959,6 +4959,40 @@ static void test_job_list_attribute(HANDLE parent_job) ok(ret, "SetInformationJobObject error %lu\n", GetLastError()); } +static void test_services_exe(void) +{ + NTSTATUS status; + ULONG size, offset; + char *buf; + SYSTEM_PROCESS_INFORMATION *spi; + ULONG services_pid = 0, services_session_id = ~0; + + status = NtQuerySystemInformation(SystemProcessInformation, NULL, 0, &size); + ok(status == STATUS_INFO_LENGTH_MISMATCH, "got %#lx\n", status); + + buf = malloc(size); + status = NtQuerySystemInformation(SystemProcessInformation, buf, size, &size); + ok(status == STATUS_SUCCESS, "got %#lx\n", status); + + spi = (SYSTEM_PROCESS_INFORMATION *)buf; + offset = 0; + + do + { + spi = (SYSTEM_PROCESS_INFORMATION *)(buf + offset); + if (!wcsnicmp(spi->ProcessName.Buffer, L"services.exe", spi->ProcessName.Length/sizeof(WCHAR))) + { + services_pid = HandleToUlong(spi->UniqueProcessId); + services_session_id = spi->SessionId; + } + offset += spi->NextEntryOffset; + } while (spi->NextEntryOffset != 0); + + ok(services_pid != 0, "services.exe not found\n"); + todo_wine + ok(services_session_id == 0, "got services.exe SessionId %lu\n", services_session_id); +} + START_TEST(process) { HANDLE job, hproc, h, h2; @@ -5088,6 +5122,7 @@ START_TEST(process) test_parent_process_attribute(0, NULL); test_handle_list_attribute(FALSE, NULL, NULL); test_dead_process(); + test_services_exe(); /* things that can be tested: * lookup: check the way program to be executed is searched

1 0

Zhiyi Zhang : uxtheme: Return correct WM_ERASEBKGND result from custom dialog procedures.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: d987c277c12bc17a77c9d79b92024d449f9a85f9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d987c277c12bc17a77c9d79b… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Thu Mar 24 17:06:07 2022 +0800 uxtheme: Return correct WM_ERASEBKGND result from custom dialog procedures. Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/uxtheme/dialog.c | 3 ++- dlls/uxtheme/tests/system.c | 3 +-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/uxtheme/dialog.c b/dlls/uxtheme/dialog.c index 706d170936e..13943d753bb 100644 --- a/dlls/uxtheme/dialog.c +++ b/dlls/uxtheme/dialog.c @@ -129,9 +129,10 @@ LRESULT WINAPI UXTHEME_DefDlgProc(HWND hwnd, UINT msg, WPARAM wp, LPARAM lp, BOO case WM_ERASEBKGND: { dlgproc = (WNDPROC)GetWindowLongPtrW(hwnd, DWLP_DLGPROC); + SetWindowLongPtrW(hwnd, DWLP_MSGRESULT, 0); lr = LOWORD(CallWindowProcW(dlgproc, hwnd, msg, wp, lp)); if (lr) - return lr; + return GetWindowLongPtrW(hwnd, DWLP_MSGRESULT); brush = get_dialog_background_brush(hwnd, TRUE); if (!brush) diff --git a/dlls/uxtheme/tests/system.c b/dlls/uxtheme/tests/system.c index 1308d54264b..1f04ac879ec 100644 --- a/dlls/uxtheme/tests/system.c +++ b/dlls/uxtheme/tests/system.c @@ -1889,8 +1889,7 @@ static void test_EnableThemeDialogTexture(void) /* Test that the dialog procedure should take precedence over DefDlgProc() for WM_ERASEBKGND */ handle_WM_ERASEBKGND = TRUE; lr = SendMessageW(dialog, WM_ERASEBKGND, (WPARAM)child_hdc, 0); - todo_wine - ok(lr == 0, "Expected 0, got %#lx.\n", lr); + ok(lr == 0, "Expected 0, got %#Ix.\n", lr); handle_WM_ERASEBKGND = FALSE; /* Test that dialog doesn't have theme handle opened for itself */

1 0

Zhiyi Zhang : uxtheme: Remove a redundant condition check.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: f9923d6bb9b800dbf9a3bec93b7aa7cbebd92e99 URL: https://source.winehq.org/git/wine.git/?a=commit;h=f9923d6bb9b800dbf9a3bec9… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Thu Mar 24 17:06:14 2022 +0800 uxtheme: Remove a redundant condition check. The sizingtype check is redundant because it can only be ST_TRUESIZE, ST_STRETCH or ST_TILE and the check is already in the not ST_TRUESIZE branch. Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/uxtheme/draw.c | 111 ++++++++++++++++++++++++++-------------------------- 1 file changed, 55 insertions(+), 56 deletions(-) diff --git a/dlls/uxtheme/draw.c b/dlls/uxtheme/draw.c index d503fad3fea..3ddb9b3bf4d 100644 --- a/dlls/uxtheme/draw.c +++ b/dlls/uxtheme/draw.c @@ -710,6 +710,7 @@ static HRESULT UXTHEME_DrawImageBackground(HTHEME hTheme, HDC hdc, int iPartId, int iStateId, RECT *pRect, const DTBGOPTS *pOptions) { + int destCenterWidth, srcCenterWidth, destCenterHeight, srcCenterHeight; HRESULT hr = S_OK; HBITMAP bmpSrc; HGDIOBJ oldSrc; @@ -825,68 +826,66 @@ static HRESULT UXTHEME_DrawImageBackground(HTHEME hTheme, HDC hdc, int iPartId, goto draw_error; } - if ((sizingtype == ST_STRETCH) || (sizingtype == ST_TILE)) { - int destCenterWidth = dstSize.x - (sm.cxLeftWidth + sm.cxRightWidth); - int srcCenterWidth = srcSize.x - (sm.cxLeftWidth + sm.cxRightWidth); - int destCenterHeight = dstSize.y - (sm.cyTopHeight + sm.cyBottomHeight); - int srcCenterHeight = srcSize.y - (sm.cyTopHeight + sm.cyBottomHeight); - - if(destCenterWidth > 0) { - /* Center top */ - if(!UXTHEME_SizedBlt (hdcDst, sm.cxLeftWidth, 0, - destCenterWidth, sm.cyTopHeight, - hdcSrc, rcSrc.left+sm.cxLeftWidth, rcSrc.top, - srcCenterWidth, sm.cyTopHeight, - sizingtype, transparent, transparentcolor)) { - hr = HRESULT_FROM_WIN32(GetLastError()); - goto draw_error; - } - /* Center bottom */ - if(!UXTHEME_SizedBlt (hdcDst, sm.cxLeftWidth, dstSize.y-sm.cyBottomHeight, - destCenterWidth, sm.cyBottomHeight, - hdcSrc, rcSrc.left+sm.cxLeftWidth, rcSrc.bottom-sm.cyBottomHeight, - srcCenterWidth, sm.cyBottomHeight, - sizingtype, transparent, transparentcolor)) { - hr = HRESULT_FROM_WIN32(GetLastError()); - goto draw_error; - } + destCenterWidth = dstSize.x - (sm.cxLeftWidth + sm.cxRightWidth); + srcCenterWidth = srcSize.x - (sm.cxLeftWidth + sm.cxRightWidth); + destCenterHeight = dstSize.y - (sm.cyTopHeight + sm.cyBottomHeight); + srcCenterHeight = srcSize.y - (sm.cyTopHeight + sm.cyBottomHeight); + + if(destCenterWidth > 0) { + /* Center top */ + if(!UXTHEME_SizedBlt (hdcDst, sm.cxLeftWidth, 0, + destCenterWidth, sm.cyTopHeight, + hdcSrc, rcSrc.left+sm.cxLeftWidth, rcSrc.top, + srcCenterWidth, sm.cyTopHeight, + sizingtype, transparent, transparentcolor)) { + hr = HRESULT_FROM_WIN32(GetLastError()); + goto draw_error; } - if(destCenterHeight > 0) { - /* Left center */ - if(!UXTHEME_SizedBlt (hdcDst, 0, sm.cyTopHeight, - sm.cxLeftWidth, destCenterHeight, - hdcSrc, rcSrc.left, rcSrc.top+sm.cyTopHeight, - sm.cxLeftWidth, srcCenterHeight, - sizingtype, - transparent, transparentcolor)) { - hr = HRESULT_FROM_WIN32(GetLastError()); - goto draw_error; - } - /* Right center */ - if(!UXTHEME_SizedBlt (hdcDst, dstSize.x-sm.cxRightWidth, sm.cyTopHeight, - sm.cxRightWidth, destCenterHeight, - hdcSrc, rcSrc.right-sm.cxRightWidth, rcSrc.top+sm.cyTopHeight, - sm.cxRightWidth, srcCenterHeight, + /* Center bottom */ + if(!UXTHEME_SizedBlt (hdcDst, sm.cxLeftWidth, dstSize.y-sm.cyBottomHeight, + destCenterWidth, sm.cyBottomHeight, + hdcSrc, rcSrc.left+sm.cxLeftWidth, rcSrc.bottom-sm.cyBottomHeight, + srcCenterWidth, sm.cyBottomHeight, + sizingtype, transparent, transparentcolor)) { + hr = HRESULT_FROM_WIN32(GetLastError()); + goto draw_error; + } + } + if(destCenterHeight > 0) { + /* Left center */ + if(!UXTHEME_SizedBlt (hdcDst, 0, sm.cyTopHeight, + sm.cxLeftWidth, destCenterHeight, + hdcSrc, rcSrc.left, rcSrc.top+sm.cyTopHeight, + sm.cxLeftWidth, srcCenterHeight, + sizingtype, + transparent, transparentcolor)) { + hr = HRESULT_FROM_WIN32(GetLastError()); + goto draw_error; + } + /* Right center */ + if(!UXTHEME_SizedBlt (hdcDst, dstSize.x-sm.cxRightWidth, sm.cyTopHeight, + sm.cxRightWidth, destCenterHeight, + hdcSrc, rcSrc.right-sm.cxRightWidth, rcSrc.top+sm.cyTopHeight, + sm.cxRightWidth, srcCenterHeight, + sizingtype, transparent, transparentcolor)) { + hr = HRESULT_FROM_WIN32(GetLastError()); + goto draw_error; + } + } + if(destCenterHeight > 0 && destCenterWidth > 0) { + BOOL borderonly = FALSE; + GetThemeBool(hTheme, iPartId, iStateId, TMT_BORDERONLY, &borderonly); + if(!borderonly) { + /* Center */ + if(!UXTHEME_SizedBlt (hdcDst, sm.cxLeftWidth, sm.cyTopHeight, + destCenterWidth, destCenterHeight, + hdcSrc, rcSrc.left+sm.cxLeftWidth, rcSrc.top+sm.cyTopHeight, + srcCenterWidth, srcCenterHeight, sizingtype, transparent, transparentcolor)) { hr = HRESULT_FROM_WIN32(GetLastError()); goto draw_error; } } - if(destCenterHeight > 0 && destCenterWidth > 0) { - BOOL borderonly = FALSE; - GetThemeBool(hTheme, iPartId, iStateId, TMT_BORDERONLY, &borderonly); - if(!borderonly) { - /* Center */ - if(!UXTHEME_SizedBlt (hdcDst, sm.cxLeftWidth, sm.cyTopHeight, - destCenterWidth, destCenterHeight, - hdcSrc, rcSrc.left+sm.cxLeftWidth, rcSrc.top+sm.cyTopHeight, - srcCenterWidth, srcCenterHeight, - sizingtype, transparent, transparentcolor)) { - hr = HRESULT_FROM_WIN32(GetLastError()); - goto draw_error; - } - } - } } draw_error:

1 0

Eric Pouech : shell32: Always allocate a whole ITEMIDLIST structure.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: e62dd2a5b62a6a75d9bcaec29e80b31eb257c41d URL: https://source.winehq.org/git/wine.git/?a=commit;h=e62dd2a5b62a6a75d9bcaec2… Author: Eric Pouech <eric.pouech(a)gmail.com> Date: Thu Mar 24 10:13:26 2022 +0100 shell32: Always allocate a whole ITEMIDLIST structure. Partially allocating a structure generates -Warray-bounds warnings with GCC11. In this specific case, given the rounding in memory allocators, would even not increase the effective allocated size. Signed-off-by: Eric Pouech <eric.pouech(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/shell32/pidl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/shell32/pidl.c b/dlls/shell32/pidl.c index 4c66a14c700..f9c91a79132 100644 --- a/dlls/shell32/pidl.c +++ b/dlls/shell32/pidl.c @@ -1574,7 +1574,7 @@ LPITEMIDLIST _ILCreateDesktop(void) LPITEMIDLIST ret; TRACE("()\n"); - ret = SHAlloc(2); + ret = SHAlloc(sizeof(*ret)); if (ret) ret->mkid.cb = 0; return ret;

1 0

Alexandre Julliard : kernelbase: Reimplement LOCALE_SNAME in GetLocaleInfoW/Ex using the locale.nls data.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: d370618a5a505cbaf8cb5b1c441d3335aa8b2c1e URL: https://source.winehq.org/git/wine.git/?a=commit;h=d370618a5a505cbaf8cb5b1c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 28 12:09:51 2022 +0200 kernelbase: Reimplement LOCALE_SNAME in GetLocaleInfoW/Ex using the locale.nls data. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/locale.c | 435 +++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 423 insertions(+), 12 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=d370618a5a505cbaf8cb…

1 0

Alexandre Julliard : kernelbase: Reimplement LOCALE_SPARENT in GetLocaleInfoW/Ex using the locale.nls data.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: 0f7cd3d685b2b81a3d6802c0b32080bfa5dc4b90 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0f7cd3d685b2b81a3d6802c0… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 28 12:13:04 2022 +0200 kernelbase: Reimplement LOCALE_SPARENT in GetLocaleInfoW/Ex using the locale.nls data. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/locale.c | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/dlls/kernelbase/locale.c b/dlls/kernelbase/locale.c index 258893cada6..39d7fe1ccd9 100644 --- a/dlls/kernelbase/locale.c +++ b/dlls/kernelbase/locale.c @@ -1103,7 +1103,7 @@ static int get_locale_info( const NLS_LOCALE_DATA *locale, LCID lcid, LCTYPE typ return -1; case LOCALE_SPARENT: - return -1; + return locale_return_string( locale->sparent, type, buffer, len ); case LOCALE_SCONSOLEFALLBACKNAME: return -1; @@ -5017,17 +5017,6 @@ INT WINAPI DECLSPEC_HOTPATCH GetLocaleInfoEx( const WCHAR *name, LCTYPE info, WC lcid = LocaleNameToLCID( name, 0 ); if (!lcid) return 0; - - /* special handling for neutral locale names */ - if (name && lstrlenW( name ) == 2) - { - switch (LOWORD( info )) - { - case LOCALE_SPARENT: - if (len) buffer[0] = 0; - return 1; - } - } return GetLocaleInfoW( lcid, info, buffer, len ); }

1 0

Alexandre Julliard : kernelbase: Reimplement LCIDToLocaleName() using the locale.nls data.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: 412e79181fc15aec030a584ca484a6b6fedee868 URL: https://source.winehq.org/git/wine.git/?a=commit;h=412e79181fc15aec030a584c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 28 12:21:15 2022 +0200 kernelbase: Reimplement LCIDToLocaleName() using the locale.nls data. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernel32/tests/locale.c | 8 ++++++++ dlls/kernelbase/locale.c | 16 ++++++++++------ 2 files changed, 18 insertions(+), 6 deletions(-) diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c index d30a771c2c2..9475df22cbe 100644 --- a/dlls/kernel32/tests/locale.c +++ b/dlls/kernel32/tests/locale.c @@ -2790,6 +2790,14 @@ static void test_LocaleNameToLCID(void) ok(ret > 0, "%s: got %d\n", wine_dbgstr_w(L"zh-hans"), ret); ok(!lstrcmpW(L"zh-CN", buffer), "%s: got wrong locale name %s\n", wine_dbgstr_w(L"zh-hans"), wine_dbgstr_w(buffer)); + + /* de-DE_phoneb */ + lcid = pLocaleNameToLCID(L"de-DE_phoneb", 0); + ok(lcid == MAKELCID(MAKELANGID(LANG_GERMAN, SUBLANG_DEFAULT), SORT_GERMAN_PHONE_BOOK), + "%s: got wrong lcid 0x%04lx\n", wine_dbgstr_w(L"zh-hans"), lcid); + ret = pLCIDToLocaleName(lcid, buffer, ARRAY_SIZE(buffer), 0); + ok(!lstrcmpW(L"de-DE_phoneb", buffer), "got wrong locale name %s\n", + wine_dbgstr_w(buffer)); } if (pRtlLocaleNameToLcid) diff --git a/dlls/kernelbase/locale.c b/dlls/kernelbase/locale.c index 39d7fe1ccd9..8b137cdf90a 100644 --- a/dlls/kernelbase/locale.c +++ b/dlls/kernelbase/locale.c @@ -5180,9 +5180,9 @@ LANGID WINAPI DECLSPEC_HOTPATCH GetSystemDefaultLangID(void) /*********************************************************************** * GetSystemDefaultLocaleName (kernelbase.@) */ -INT WINAPI DECLSPEC_HOTPATCH GetSystemDefaultLocaleName( LPWSTR name, INT len ) +INT WINAPI DECLSPEC_HOTPATCH GetSystemDefaultLocaleName( LPWSTR name, INT count ) { - return LCIDToLocaleName( GetSystemDefaultLCID(), name, len, 0 ); + return get_locale_info( system_locale, system_lcid, LOCALE_SNAME, name, count ); } @@ -5329,7 +5329,7 @@ LANGID WINAPI DECLSPEC_HOTPATCH GetUserDefaultLangID(void) */ INT WINAPI DECLSPEC_HOTPATCH GetUserDefaultLocaleName( LPWSTR name, INT len ) { - return LCIDToLocaleName( GetUserDefaultLCID(), name, len, 0 ); + return get_locale_info( user_locale, user_lcid, LOCALE_SNAME, name, len ); } @@ -5692,10 +5692,14 @@ DWORD WINAPI DECLSPEC_HOTPATCH IsValidNLSVersion( NLS_FUNCTION func, const WCHAR */ INT WINAPI DECLSPEC_HOTPATCH LCIDToLocaleName( LCID lcid, WCHAR *name, INT count, DWORD flags ) { - static int once; - if (flags && !once++) FIXME( "unsupported flags %lx\n", flags ); + const NLS_LOCALE_DATA *locale = get_locale_by_id( &lcid, flags ); - return GetLocaleInfoW( lcid, LOCALE_SNAME | LOCALE_NOUSEROVERRIDE, name, count ); + if (!locale) + { + SetLastError( ERROR_INVALID_PARAMETER ); + return 0; + } + return get_locale_info( locale, lcid, LOCALE_SNAME, name, count ); }

1 0

Alexandre Julliard : kernelbase: Reimplement LOCALE_ILANGUAGE/IDEFAULTLANGUAGE in GetLocaleInfoW/Ex using the locale.nls data.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: 956c88db183402caf5f1bc4c45af5e667eef41e4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=956c88db183402caf5f1bc4c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 28 12:15:30 2022 +0200 kernelbase: Reimplement LOCALE_ILANGUAGE/IDEFAULTLANGUAGE in GetLocaleInfoW/Ex using the locale.nls data. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/locale.c | 45 +++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 43 insertions(+), 2 deletions(-) diff --git a/dlls/kernelbase/locale.c b/dlls/kernelbase/locale.c index 8b137cdf90a..7cb16e5b387 100644 --- a/dlls/kernelbase/locale.c +++ b/dlls/kernelbase/locale.c @@ -850,16 +850,57 @@ static int locale_return_string( DWORD pos, LCTYPE type, WCHAR *buffer, int len } +static int locale_return_number( UINT val, LCTYPE type, WCHAR *buffer, int len ) +{ + int ret; + WCHAR tmp[80]; + + if (!(type & LOCALE_RETURN_NUMBER)) + { + switch (LOWORD(type)) + { + case LOCALE_ILANGUAGE: + case LOCALE_IDEFAULTLANGUAGE: + ret = swprintf( tmp, ARRAY_SIZE(tmp), L"%04x", val ) + 1; + break; + case LOCALE_IDEFAULTEBCDICCODEPAGE: + ret = swprintf( tmp, ARRAY_SIZE(tmp), L"%03u", val ) + 1; + break; + default: + ret = swprintf( tmp, ARRAY_SIZE(tmp), L"%u", val ) + 1; + break; + } + } + else ret = sizeof(UINT) / sizeof(WCHAR); + + if (!len) return ret; + if (ret > len) + { + SetLastError( ERROR_INSUFFICIENT_BUFFER ); + return 0; + } + + if (type & LOCALE_RETURN_NUMBER) memcpy( buffer, &val, sizeof(val) ); + else wcscpy( buffer, tmp ); + + return ret; +} + + /* get locale information from the locale.nls file */ static int get_locale_info( const NLS_LOCALE_DATA *locale, LCID lcid, LCTYPE type, WCHAR *buffer, int len ) { + UINT val; + if (locale != user_locale) type |= LOCALE_NOUSEROVERRIDE; switch (LOWORD(type)) { case LOCALE_ILANGUAGE: - return -1; + /* return default language for neutral locales */ + val = locale->inotneutral ? locale->ilanguage : locale->idefaultlanguage; + return locale_return_number( val, type, buffer, len ); case LOCALE_SLOCALIZEDDISPLAYNAME: return -1; @@ -883,7 +924,7 @@ static int get_locale_info( const NLS_LOCALE_DATA *locale, LCID lcid, LCTYPE typ return -1; case LOCALE_IDEFAULTLANGUAGE: - return -1; + return locale_return_number( locale->idefaultlanguage, type, buffer, len ); case LOCALE_IDEFAULTCOUNTRY: return -1;

1 0

Alexandre Julliard : kernelbase: Reimplement LOCALE_*LANGNAME in GetLocaleInfoW/Ex using the locale.nls data.
by Alexandre Julliard 28 Mar '22

28 Mar '22

Module: wine Branch: master Commit: 85a39a977347e037bcd910efd388faa6f04d870a URL: https://source.winehq.org/git/wine.git/?a=commit;h=85a39a977347e037bcd910ef… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Mar 28 12:26:34 2022 +0200 kernelbase: Reimplement LOCALE_*LANGNAME in GetLocaleInfoW/Ex using the locale.nls data. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/locale.c | 13 +++++++------ dlls/msvcrt/locale.c | 9 +++++++-- 2 files changed, 14 insertions(+), 8 deletions(-) diff --git a/dlls/kernelbase/locale.c b/dlls/kernelbase/locale.c index 7cb16e5b387..0caf790384c 100644 --- a/dlls/kernelbase/locale.c +++ b/dlls/kernelbase/locale.c @@ -906,10 +906,10 @@ static int get_locale_info( const NLS_LOCALE_DATA *locale, LCID lcid, LCTYPE typ return -1; case LOCALE_SABBREVLANGNAME: - return -1; + return locale_return_string( locale->sabbrevlangname, type, buffer, len ); case LOCALE_SNATIVELANGNAME: - return -1; + return locale_return_string( locale->snativelangname, type, buffer, len ); case LOCALE_ICOUNTRY: return -1; @@ -1094,7 +1094,7 @@ static int get_locale_info( const NLS_LOCALE_DATA *locale, LCID lcid, LCTYPE typ return -1; case LOCALE_SISO639LANGNAME: - return -1; + return locale_return_string( locale->siso639langname, type, buffer, len ); case LOCALE_SISO3166CTRYNAME: return -1; @@ -1126,7 +1126,7 @@ static int get_locale_info( const NLS_LOCALE_DATA *locale, LCID lcid, LCTYPE typ return -1; case LOCALE_SISO639LANGNAME2: - return -1; + return locale_return_string( locale->siso639langname2, type, buffer, len ); case LOCALE_SISO3166CTRYNAME2: return -1; @@ -1150,7 +1150,8 @@ static int get_locale_info( const NLS_LOCALE_DATA *locale, LCID lcid, LCTYPE typ return -1; case LOCALE_SLOCALIZEDLANGUAGENAME: - return -1; + /* FIXME: localization */ + return locale_return_string( locale->senglanguage, type, buffer, len ); case LOCALE_IREADINGLAYOUT: return -1; @@ -1201,7 +1202,7 @@ static int get_locale_info( const NLS_LOCALE_DATA *locale, LCID lcid, LCTYPE typ return -1; case LOCALE_SENGLANGUAGE: - return -1; + return locale_return_string( locale->senglanguage, type, buffer, len ); case LOCALE_SENGCOUNTRY: return -1; diff --git a/dlls/msvcrt/locale.c b/dlls/msvcrt/locale.c index 924671e7047..e800e7142c8 100644 --- a/dlls/msvcrt/locale.c +++ b/dlls/msvcrt/locale.c @@ -525,8 +525,13 @@ static BOOL update_threadlocinfo_category(LCID lcid, unsigned short cp, if(!locinfo->lc_category[category].locale) { int len = 0; - len += GetLocaleInfoA(lcid, LOCALE_SENGLANGUAGE - |LOCALE_NOUSEROVERRIDE, buf, 256); + if (lcid == MAKELANGID( LANG_NORWEGIAN, SUBLANG_NORWEGIAN_NYNORSK )) + { + /* locale.nls contains "Norwegian Nynorsk" instead for LOCALE_SENGLANGUAGE */ + strcpy( buf, "Norwegian-Nynorsk" ); + len = strlen( buf ) + 1; + } + else len += GetLocaleInfoA(lcid, LOCALE_SENGLANGUAGE|LOCALE_NOUSEROVERRIDE, buf, 256); buf[len-1] = '_'; len += GetLocaleInfoA(lcid, LOCALE_SENGCOUNTRY |LOCALE_NOUSEROVERRIDE, &buf[len], 256-len);

1 0