Re: [PATCH 0/3] MR5756: comdlg32: Fix a few issues with color dialog

[Nikolay Sivov (＠nsivov)]

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?