31 May
2024
31 May
'24
11:51 a.m.
Nikolay Sivov (@nsivov) commented about dlls/comdlg32/colordlg.c:
if (result) { LRESULT editpos = SendMessageA(hwnd, EM_GETSEL, 0, 0); + infoPtr->updating = TRUE; SetWindowTextA(hwnd, buffer ); SendMessageA(hwnd, EM_SETSEL, 0, editpos); + infoPtr->updating = FALSE; } This does not look good. If we force content to something valid, why would it trigger same logic again?
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/5756#note_71913