September 2023 - wine-gitlab

[PATCH v9 0/1] MR3959: uxtheme: Add ShouldSystemUseDarkMode.
by Helix Graziani (＠TornadoCookie) 01 Oct '23

01 Oct '23

Used in some .NET apps for theming purposes. -- v9: uxtheme: Add ShouldSystemUseDarkMode. https://gitlab.winehq.org/wine/wine/-/merge_requests/3959

4 8

[PATCH v2 0/1] MR3988: winetts: Add stub dll.
by Shaun Ren (＠shaunren) 30 Sep '23

30 Sep '23

-- v2: winetts: Add stub dll. https://gitlab.winehq.org/wine/wine/-/merge_requests/3988

4 3

[PATCH 0/1] MR3989: user32/tests: Print regions in test_hvredraw failure case.
by Esme Povirk (＠madewokherd) 30 Sep '23

30 Sep '23

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/3989

2 1

Re: [PATCH v8 0/1] MR3959: uxtheme: Add ShouldSystemUseDarkMode.
by Zhiyi Zhang (＠zhiyi) 30 Sep '23

30 Sep '23

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/tests/system.c: > + /* Call ShouldSystemUseDarkMode() and check for errors */ > + SetLastError(ERROR_SUCCESS); /* Clear error buffer so that we can detect if an error occurred. */ > + function_result = pShouldSystemUseDarkMode(); > + last_error = GetLastError(); > + ok(last_error == ERROR_SUCCESS, "ShouldSystemUseDarkMode failed (?) with error %ld.\n", > + last_error); > + > + /* Expect same value as key */ > + ok(function_result == !system_uses_light_theme, "Expected value %d, got %d.\n", > + !system_uses_light_theme, function_result); > + > + ls = RegDeleteValueA(hk, "AppsUseLightTheme"); > + ok(ls == 0, "RegDeleteValueA failed: %ld.\n", ls); > + > + /* Call ShouldSystemUseDarkMode() and check for errors with deleted value */ > + SetLastError(ERROR_SUCCESS); /* Clear error buffer so that we can detect if an error occurred. */ For the last error code. You can SetLastError(0xdeadbeef) before the call and check that the last error is set to ERROR_SUCCESS after the call. And then you set the last error code in the ShouldSystemUseDarkMode() implementation. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3959#note_47274

1 0

Re: [PATCH v8 0/1] MR3959: uxtheme: Add ShouldSystemUseDarkMode.
by Zhiyi Zhang (＠zhiyi) 30 Sep '23

30 Sep '23

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/system.c: > UnregisterUserApiHook(); > return TRUE; > } > + > +/********************************************************************** > + * ShouldSystemUseDarkMode (UXTHEME.138) > + * > + * RETURNS > + * Whether or not the system/app should use dark mode. Shouldn't `system/app` be `system`? because there is a ShouldAppsUseDarkMode() as you said previously. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3959#note_47273

1 0

Re: [PATCH v8 0/1] MR3959: uxtheme: Add ShouldSystemUseDarkMode.
by Zhiyi Zhang (＠zhiyi) 30 Sep '23

30 Sep '23

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/system.c: > UnregisterUserApiHook(); > return TRUE; > } > + > +/********************************************************************** > + * ShouldSystemUseDarkMode (UXTHEME.138) > + * > + * RETURNS > + * Whether or not the system/app should use dark mode. > + */ > +BOOL WINAPI ShouldSystemUseDarkMode(void) > +{ > + DWORD system_uses_light_theme_size = sizeof(DWORD); You can remove the system_uses_ prefix. These codes are in ShouldSystemUseDarkMode() so it's already known that it's for querying system settings. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3959#note_47272

1 0

Re: [PATCH v8 0/1] MR3959: uxtheme: Add ShouldSystemUseDarkMode.
by Zhiyi Zhang (＠zhiyi) 30 Sep '23

30 Sep '23

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/system.c: > +/********************************************************************** > + * ShouldSystemUseDarkMode (UXTHEME.138) > + * > + * RETURNS > + * Whether or not the system/app should use dark mode. > + */ > +BOOL WINAPI ShouldSystemUseDarkMode(void) > +{ > + DWORD system_uses_light_theme_size = sizeof(DWORD); > + /* Persists between calls, in windows it might look up some internal table. */ > + static DWORD system_uses_light_theme = TRUE; > + > + /* We don't necessarily care that this might fail because it doesn't affect > + * system_uses_light_theme if it does. > + */ > + RegGetValueA(HKEY_CURRENT_USER, Please use RegGetValueW() instead so that we don't have to do ASCII/Unicode conversion every time this gets called. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3959#note_47271

1 0

Re: [PATCH v8 0/1] MR3959: uxtheme: Add ShouldSystemUseDarkMode.
by Zhiyi Zhang (＠zhiyi) 30 Sep '23

30 Sep '23

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/tests/system.c: > DestroyWindow(hwnd); > } > > +static void test_ShouldSystemUseDarkMode(void) I think these tests might be a bit too much. I like the initial version that uses RegGetValueA() and confirms the registry value is the opposite of the ShouldSystemUseDarkMode. With some checks for the last error code and I think that's enough. These newly added tests confirm caching for ShouldSystemUseDarkMode() and that's good. However, ShouldSystemUseDarkMode() still need to query the registry key so it's not that useful. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3959#note_47270

1 0

Re: [PATCH v8 0/1] MR3959: uxtheme: Add ShouldSystemUseDarkMode.
by Zhiyi Zhang (＠zhiyi) 30 Sep '23

30 Sep '23

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/system.c: > + > +/********************************************************************** > + * ShouldSystemUseDarkMode (UXTHEME.138) > + * > + * RETURNS > + * Whether or not the system/app should use dark mode. > + */ > +BOOL WINAPI ShouldSystemUseDarkMode(void) > +{ > + DWORD system_uses_light_theme_size = sizeof(DWORD); > + /* Persists between calls, in windows it might look up some internal table. */ > + static DWORD system_uses_light_theme = TRUE; > + > + /* We don't necessarily care that this might fail because it doesn't affect > + * system_uses_light_theme if it does. > + */ While comments are nice to have, these are pretty obvious so I think you can ignore it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3959#note_47269

1 0

Re: [PATCH v8 0/1] MR3959: uxtheme: Add ShouldSystemUseDarkMode.
by Zhiyi Zhang (＠zhiyi) 30 Sep '23

30 Sep '23

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/system.c: > UnregisterUserApiHook(); > return TRUE; > } > + > +/********************************************************************** > + * ShouldSystemUseDarkMode (UXTHEME.138) > + * > + * RETURNS > + * Whether or not the system/app should use dark mode. > + */ > +BOOL WINAPI ShouldSystemUseDarkMode(void) > +{ > + DWORD system_uses_light_theme_size = sizeof(DWORD); > + /* Persists between calls, in windows it might look up some internal table. */ > + static DWORD system_uses_light_theme = TRUE; You can remove the static keyword and always initialize system_uses_light_theme to TRUE. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3959#note_47268

1 0

2025

2024

2023

2022

wine-gitlab September 2023