This is a new try of mciqtz32's video window patch.
Comparing to the previous !370, I added a lot of tests regarding the video window behavior.
Some portion of the previous patch are dropped to avoid complexity, I'll add that part later.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/557
--
v4: mciqtz32: Show the default video window when switching from another one.
mciqtz32: Hide the default video window when switching to another one.
mciqtz32: Reset the video size when switching video destination.
mciqtz32: Correct initial video window dimensions.
mciqtz32: Fix MCI_DGV_WHERE_WINDOW behavior.
mciqtz32: Correct video window behavior by creating default window.
winmm/tests: Add tests for destination of video window.
winmm/tests: Add tests for dimensions of video window.
winmm/tests: Add tests for window style of video window.
mciqtz32: Don't hide video window when stopping.
winmm/tests: Add tests for visibility of video window.
https://gitlab.winehq.org/wine/wine/-/merge_requests/370
1. Default to zero on IDOK if the edit control is empty.
2. Limit text input length to 8 or 16 hexadecimal characters, or 10 or 20 decimal characters, depending on registry value type and format specifier chosen.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/554
On Thu Jul 28 03:42:04 2022 +0000, **** wrote:
> Marvin replied on the mailing list:
> ```
> Hi,
> It looks like your patch introduced the new failures shown below.
> Please investigate and fix them before resubmitting your patch.
> If they are not new, fixing them anyway would help a lot. Otherwise
> please ask for the known failures list to be updated.
> The full results can be found at:
> https://testbot.winehq.org/JobDetails.pl?Key=119994
> Your paranoid android.
> === debian11 (32 bit report) ===
> comctl32:
> listview.c:3578: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3595: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3578: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3595: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> === debian11 (32 bit Chinese:China report) ===
> comctl32:
> listview.c:3578: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3595: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3578: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3595: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> === debian11 (32 bit WoW report) ===
> comctl32:
> listview.c:3578: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3595: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3578: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3595: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> === debian11 (64 bit WoW report) ===
> comctl32:
> listview.c:3578: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3595: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3578: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> listview.c:3595: Test succeeded inside todo block: ownerdata select
> multiple notification: marked "todo_wine" but succeeds
> ```
Should be fixed.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/550#note_5203