NtUserMessageCall(..., NtUserSendDriverMessage) does not ensure the
validity of the hwnd, so this check will incorrectly fail if the
last error was ERROR_INVALID_WINDOW_HANDLE by coincidence. The
notify_owner call immediately following this code will catch a
destroyed owner hwnd.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3147
On Fri Jun 23 19:02:05 2023 +0000, Davide Beatrici wrote:
> I would expect the same behavior for `PFD_DRAW_TO_BITMAP` though. Aren't
> both the same flag "class"?
I guess we need more tests to be sure.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3111#note_36674
On Fri Jun 23 18:54:52 2023 +0000, Damjan Jovanovic wrote:
> I don't think it works like that. If the flag is set, it only returns a
> PIXELFORMATDESCRIPTOR with the flag set, but if it is clear, then the
> returned PIXELFORMATDESCRIPTOR can have the flag set or cleared.
I would expect the same behavior for `PFD_DRAW_TO_BITMAP` though. Aren't both the same flag "class"?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3111#note_36673
On Fri Jun 23 18:48:33 2023 +0000, Davide Beatrici wrote:
> For consistency:
> ```suggestion:-0+0
> if ((ppfd->dwFlags & PFD_SUPPORT_GDI) != (format.dwFlags & PFD_SUPPORT_GDI))
> ```
I don't think it works like that. If the flag is set, it only returns a PIXELFORMATDESCRIPTOR with the flag set, but if it is clear, then the returned PIXELFORMATDESCRIPTOR can have the flag set or cleared.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3111#note_36672
And use that instead of duplicating those helpers across the tree.
Besides removing existing duplicates, I am planning to add yet another WinRT dll (gamepad) and we will likely add more in the future, each of those can use that, as well as potentially more common helpers.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1045
This MR supercedes !1895 and !3073 as a solution to bug [#50771](https://bugs.winehq.org/show_bug.cgi?id=50771). Rather than fixing the problem of Wine's inability to modify the attributes of read-only files, this patch set instead implements `FileDispositionInformationEx` which the Msys2 and Cygwin runtime libriries can use to avoid needing to modify attributes.
All the flags of `FileDispositionInformationEx` are implemented with the exception of `FILE_DISPOSITION_FORCE_IMAGE_SECTION_CHECK`.
This MR improves upon !3073 by implementing `FILE_DISPOSITION_POSIX_SEMANTICS` and `FILE_DISPOSITION_ON_CLOSE` where the previous MR only implemented `FILE_DISPOSITION_DELETE` and `FILE_DISPOSITION_IGNORE_READONLY_ATTRIBUTE`.
--
v2: ntdll: Implement FILE_DISPOSITION_POSIX_SEMANTICS
server: Replace unlink member of closed_fd with disp_flags
ntdll/tests: Add tests for FILE_DISPOSITION_POSIX_SEMANTICS
https://gitlab.winehq.org/wine/wine/-/merge_requests/3122