October 2022 - Wine-Commits

Piotr Caban : comctl32/tests: Test passing extra data in PROPSHEETPAGE structure.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: wine Branch: master Commit: 16ac644b114ad38770fd912c8bd6513336fed2cf URL: https://gitlab.winehq.org/wine/wine/-/commit/16ac644b114ad38770fd912c8bd651… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri Aug 26 14:06:55 2022 +0200 comctl32/tests: Test passing extra data in PROPSHEETPAGE structure. --- dlls/comctl32/tests/propsheet.c | 23 +++++++++++++++++++++-- 1 file changed, 21 insertions(+), 2 deletions(-) diff --git a/dlls/comctl32/tests/propsheet.c b/dlls/comctl32/tests/propsheet.c index 2132158bd93..eee525a19d3 100644 --- a/dlls/comctl32/tests/propsheet.c +++ b/dlls/comctl32/tests/propsheet.c @@ -1035,6 +1035,7 @@ struct custom_proppage PROPSHEETPAGEA pageA; PROPSHEETPAGEW pageW; } u; + DWORD extra_data; unsigned int addref_called; unsigned int release_called; }; @@ -1050,6 +1051,12 @@ static UINT CALLBACK proppage_callback_a(HWND hwnd, UINT msg, PROPSHEETPAGEA *ps psp->lParam, psp); ok(psp_orig->pszTitle == psp->pszTitle, "Expected same page title pointer\n"); ok(!lstrcmpA(psp_orig->pszTitle, psp->pszTitle), "Expected same page title string\n"); + if (psp->dwSize >= FIELD_OFFSET(struct custom_proppage, addref_called)) + { + struct custom_proppage *extra_data = (struct custom_proppage *)psp; + todo_wine ok(extra_data->extra_data == 0x1234, "Expected extra_data to be preserved, got %lx\n", + extra_data->extra_data); + } switch (msg) { @@ -1078,6 +1085,12 @@ static UINT CALLBACK proppage_callback_w(HWND hwnd, UINT msg, PROPSHEETPAGEW *ps psp->lParam, psp); ok(psp_orig->pszTitle == psp->pszTitle, "Expected same page title pointer\n"); ok(!lstrcmpW(psp_orig->pszTitle, psp->pszTitle), "Expected same page title string\n"); + if (psp->dwSize >= FIELD_OFFSET(struct custom_proppage, addref_called)) + { + struct custom_proppage *extra_data = (struct custom_proppage *)psp; + todo_wine ok(extra_data->extra_data == 0x4321, "Expected extra_data to be preserved, got %lx\n", + extra_data->extra_data); + } switch (msg) { @@ -1110,8 +1123,11 @@ static void test_CreatePropertySheetPage(void) page.u.pageA.pszTitle = "Title"; /* Only minimal size validation is performed */ - for (page.u.pageA.dwSize = PROPSHEETPAGEA_V1_SIZE - 1; page.u.pageA.dwSize <= PROPSHEETPAGEA_V4_SIZE + 1; page.u.pageA.dwSize++) + for (page.u.pageA.dwSize = PROPSHEETPAGEA_V1_SIZE - 1; + page.u.pageA.dwSize <= FIELD_OFFSET(struct custom_proppage, addref_called); + page.u.pageA.dwSize++) { + page.extra_data = 0x1234; page.addref_called = 0; hpsp = pCreatePropertySheetPageA(&page.u.pageA); @@ -1139,8 +1155,11 @@ static void test_CreatePropertySheetPage(void) page.u.pageW.lParam = (LPARAM)&page; page.u.pageW.pszTitle = L"Title"; - for (page.u.pageW.dwSize = PROPSHEETPAGEW_V1_SIZE - 1; page.u.pageW.dwSize <= PROPSHEETPAGEW_V4_SIZE + 1; page.u.pageW.dwSize++) + for (page.u.pageW.dwSize = PROPSHEETPAGEW_V1_SIZE - 1; + page.u.pageW.dwSize <= FIELD_OFFSET(struct custom_proppage, addref_called); + page.u.pageW.dwSize++) { + page.extra_data = 0x4321; page.addref_called = 0; hpsp = pCreatePropertySheetPageW(&page.u.pageW);

1 0

Piotr Caban : comctl32/tests: Test PROPERTYSHEETPAGE passed in WM_INITDIALOG.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: wine Branch: master Commit: 88bc45a7ec3eb4da1d5ace213003cbaca2393449 URL: https://gitlab.winehq.org/wine/wine/-/commit/88bc45a7ec3eb4da1d5ace213003cb… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri Aug 26 14:13:12 2022 +0200 comctl32/tests: Test PROPERTYSHEETPAGE passed in WM_INITDIALOG. --- dlls/comctl32/tests/propsheet.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/dlls/comctl32/tests/propsheet.c b/dlls/comctl32/tests/propsheet.c index 8b2c282c72e..2132158bd93 100644 --- a/dlls/comctl32/tests/propsheet.c +++ b/dlls/comctl32/tests/propsheet.c @@ -112,10 +112,18 @@ static INT_PTR CALLBACK page_dlg_proc(HWND hwnd, UINT msg, WPARAM wparam, { case WM_INITDIALOG: { + PROPSHEETPAGEA *psp = (PROPSHEETPAGEA *)lparam; HWND sheet = GetParent(hwnd); char caption[256]; + GetWindowTextA(sheet, caption, sizeof(caption)); ok(!strcmp(caption,"test caption"), "caption: %s\n", caption); + + if (psp->dwFlags & PSP_USETITLE) + { + todo_wine ok(!strcmp(psp->pszTitle, "page title"), "psp->pszTitle = %s\n", + wine_dbgstr_a(psp->pszTitle)); + } return TRUE; } @@ -149,8 +157,9 @@ static void test_title(void) memset(&psp, 0, sizeof(psp)); psp.dwSize = sizeof(psp); - psp.dwFlags = 0; + psp.dwFlags = PSP_USETITLE; psp.hInstance = GetModuleHandleA(NULL); + psp.pszTitle = "page title"; U(psp).pszTemplate = "prop_page1"; U2(psp).pszIcon = NULL; psp.pfnDlgProc = page_dlg_proc;

1 0

Francois Gouget : testbot: Only test complete patchsets.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: tools Branch: master Commit: ca13ddef375ff236d3914770311df1698eeb51df URL: https://gitlab.winehq.org/winehq/tools/-/commit/ca13ddef375ff236d3914770311… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sun Oct 2 18:02:53 2022 +0200 testbot: Only test complete patchsets. This avoids having multiple comments in case an early GitLab merge request commit introduces new failures. --- testbot/lib/WineTestBot/Config.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testbot/lib/WineTestBot/Config.pm b/testbot/lib/WineTestBot/Config.pm index d1a24288..fccfba9c 100644 --- a/testbot/lib/WineTestBot/Config.pm +++ b/testbot/lib/WineTestBot/Config.pm @@ -116,7 +116,7 @@ $MaxTaskTries = 3; # Set to true if the combined patchset should be tested all at once # instead of testing the individual parts one by one. -$OnlyCompletePatchSets = undef; +$OnlyCompletePatchSets = 1; # Build configure timeout (in seconds) $ReconfigBuildTimeout = 60;

1 0

Francois Gouget : testbot/LogUtils: Fix a Perldoc entry.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: tools Branch: master Commit: 7b86cb5a3be0af4c00eb200b8f7eb9e43ee45413 URL: https://gitlab.winehq.org/winehq/tools/-/commit/7b86cb5a3be0af4c00eb200b8f7… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Fri Sep 23 19:01:55 2022 +0200 testbot/LogUtils: Fix a Perldoc entry. --- testbot/lib/WineTestBot/LogUtils.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm index 7780d1c8..fd850f8a 100644 --- a/testbot/lib/WineTestBot/LogUtils.pm +++ b/testbot/lib/WineTestBot/LogUtils.pm @@ -218,7 +218,7 @@ The number of errors if any. =item ErrGroupNames An array containing the names of all the error groups. -= ErrGroups +=item ErrGroups A hashtable indexed by the error group name. Each entry contains: =over

1 0

Francois Gouget : testbot/LogUtils: Add support for flaky lines in report comparisons.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: tools Branch: master Commit: 89b823e57ff23eaedc67a969addd560f617176ef URL: https://gitlab.winehq.org/winehq/tools/-/commit/89b823e57ff23eaedc67a969add… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu Sep 15 17:37:28 2022 +0200 testbot/LogUtils: Add support for flaky lines in report comparisons. Otherwise a simple line number change will prevent a flaky error from matching a previous instance. --- testbot/lib/WineTestBot/LogUtils.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm index ad708daf..7780d1c8 100644 --- a/testbot/lib/WineTestBot/LogUtils.pm +++ b/testbot/lib/WineTestBot/LogUtils.pm @@ -1206,7 +1206,7 @@ sub _GetLineKey($) $Line =~ s/ +$//; # Remove the line number - $Line =~ s/^([_a-z0-9]+\.c:)\d+:[0-9.]*( Test (?:failed|succeeded inside todo block):)/$1$2/ + $Line =~ s/^([_a-z0-9]+\.c:)\d+:[0-9.]*( Test (?:failed|marked flaky|succeeded inside (?:flaky )?todo block):)/$1$2/ or $Line =~ s/^([._a-zA-Z0-9-]+:)\d+(: recipe for target )/$1$2/ # Remove the crash code address: it changes whenever the test is recompiled

1 0

Francois Gouget : testbot: Add an option to test patchsets in one go.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: tools Branch: master Commit: 05f25bd2342c0811927720f1e5d14e034dae525c URL: https://gitlab.winehq.org/winehq/tools/-/commit/05f25bd2342c0811927720f1e5d… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sun Oct 2 17:54:48 2022 +0200 testbot: Add an option to test patchsets in one go. With this option a single result message is sent for the full patchset. This may mean missing issues fixed in later parts but avoids multiple reports in case an early part introduces new failures. --- testbot/lib/WineTestBot/Config.pm | 8 ++++++-- testbot/lib/WineTestBot/PendingPatchSets.pm | 14 +++++++++++++- 2 files changed, 19 insertions(+), 3 deletions(-) diff --git a/testbot/lib/WineTestBot/Config.pm b/testbot/lib/WineTestBot/Config.pm index e0947b65..d1a24288 100644 --- a/testbot/lib/WineTestBot/Config.pm +++ b/testbot/lib/WineTestBot/Config.pm @@ -30,7 +30,7 @@ use vars qw (@ISA @EXPORT @EXPORT_OK $UseSSL $LogDir $DataDir $BinDir $MaxRevertsWhileRunningVMs $MaxActiveVMs $MaxRunningVMs $MaxVMsWhenIdle $WaitForBoot $SleepAfterBoot $SleepAfterRevert $VMToolTimeout $WaitForShutdown $MaxVMErrors $MaxTaskTries - $AdminEMail $RobotEMail + $OnlyCompletePatchSets $AdminEMail $RobotEMail $WinePatchToOverride $WinePatchCc $WineBugUrl $ReconfigBuildTimeout $ExeBuildTimeout $ExeModuleTimeout $WineBuildTimeout $WineModuleTimeout $TimeoutMargin @@ -47,7 +47,7 @@ require Exporter; $MaxRevertingVMs $MaxRevertsWhileRunningVMs $MaxActiveVMs $MaxRunningVMs $MaxVMsWhenIdle $WaitForBoot $SleepAfterBoot $SleepAfterRevert $VMToolTimeout $WaitForShutdown - $MaxVMErrors $MaxTaskTries $AdminEMail + $MaxVMErrors $MaxTaskTries $OnlyCompletePatchSets $AdminEMail $RobotEMail $WinePatchToOverride $WinePatchCc $WineBugUrl $ReconfigBuildTimeout $ExeBuildTimeout $ExeModuleTimeout $WineBuildTimeout $WineModuleTimeout $TimeoutMargin @@ -114,6 +114,10 @@ $MaxVMErrors = 3; # How many times to run a test that fails before giving up. $MaxTaskTries = 3; +# Set to true if the combined patchset should be tested all at once +# instead of testing the individual parts one by one. +$OnlyCompletePatchSets = undef; + # Build configure timeout (in seconds) $ReconfigBuildTimeout = 60; diff --git a/testbot/lib/WineTestBot/PendingPatchSets.pm b/testbot/lib/WineTestBot/PendingPatchSets.pm index 51a1d134..070a71df 100644 --- a/testbot/lib/WineTestBot/PendingPatchSets.pm +++ b/testbot/lib/WineTestBot/PendingPatchSets.pm @@ -104,6 +104,13 @@ sub SubmitSubset($$$) { my ($self, $MaxPart, $FinalPatch) = @_; + if ($OnlyCompletePatchSets and $MaxPart < $self->TotalParts) + { + # Skip over the intermediate patches to instead + # test the full patchset at once it is complete. + return undef; + } + my ($CombinedFile, $CombinedFileName) = OpenNewFile("$DataDir/staging", "_patchset.diff"); return "Could not create a combined patch file: $!" if (!$CombinedFile); @@ -117,8 +124,13 @@ sub SubmitSubset($$$) last; } - if ($PartNo == $MaxPart and $PartNo > 1) + if ($OnlyCompletePatchSets) + { + ; # Intermediate parts were not tested so everything must be tested now + } + elsif ($PartNo == $MaxPart and $PartNo > 1) { + # Everything up to the last part separator has already been tested print $CombinedFile LastPartSeparator(); } if (open(my $PartFile, "<" , "$DataDir/patches/" . $Part->Patch->Id))

1 0

Francois Gouget : testbot: Allow two VMs to be active at the same time.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: tools Branch: master Commit: 562784083371bb2ef3e59ccfb98aba43c8afbd35 URL: https://gitlab.winehq.org/winehq/tools/-/commit/562784083371bb2ef3e59ccfb98… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sun Oct 2 17:15:27 2022 +0200 testbot: Allow two VMs to be active at the same time. An active VM is one that is either idle, running tests or reverting. --- testbot/lib/WineTestBot/Config.pm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/testbot/lib/WineTestBot/Config.pm b/testbot/lib/WineTestBot/Config.pm index 6f57995e..e0947b65 100644 --- a/testbot/lib/WineTestBot/Config.pm +++ b/testbot/lib/WineTestBot/Config.pm @@ -88,10 +88,10 @@ sub PrependPaths(@) $WineBugUrl = "https://bugs.winehq.org/show_bug.cgi?id="; # See the ScheduleOnHost() documentation in lib/WineTestBot/Jobs.pm -$MaxRevertingVMs = 1; -$MaxRevertsWhileRunningVMs = 0; +$MaxRevertingVMs = 2; +$MaxRevertsWhileRunningVMs = 1; $MaxActiveVMs = 2; -$MaxRunningVMs = 1; +$MaxRunningVMs = 2; $MaxVMsWhenIdle = undef; # How long to attempt to connect to the TestAgent while the VM is booting.

1 0

Francois Gouget : testbot/LogUtils: Fix the summary flaky count check.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: tools Branch: master Commit: cd760081220709d18cfe5924a596c38987d52da6 URL: https://gitlab.winehq.org/winehq/tools/-/commit/cd760081220709d18cfe5924a59… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sun Oct 2 18:15:39 2022 +0200 testbot/LogUtils: Fix the summary flaky count check. --- testbot/lib/WineTestBot/LogUtils.pm | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm index 98fed6aa..ad708daf 100644 --- a/testbot/lib/WineTestBot/LogUtils.pm +++ b/testbot/lib/WineTestBot/LogUtils.pm @@ -365,9 +365,11 @@ sub _NewCurrentUnit($$) UnitsRE => $Unit, UnitSize => 0, LineFailures => 0, + LineFlaky => 0, LineTodos => 0, LineSkips => 0, SummaryFailures => 0, + SummaryFlaky => 0, SummaryTodos => 0, SummarySkips => 0, IsBroken => 0, @@ -424,6 +426,7 @@ sub _CloseTestUnit($$$) if ($Cur->{Dll} ne "" and !$Cur->{IsBroken}) { _CheckSummaryCounter($LogInfo, $Cur, "Failures", "failure"); + _CheckSummaryCounter($LogInfo, $Cur, "Flaky", "flaky"); _CheckSummaryCounter($LogInfo, $Cur, "Todos", "todo"); _CheckSummaryCounter($LogInfo, $Cur, "Skips", "skip"); } @@ -619,7 +622,7 @@ sub ParseWineTestReport($$$) { _CheckUnit($LogInfo, $Cur, $1, "flaky"); _AddReportError($LogInfo, $Cur, $LineNo, "$&$'"); - $Cur->{LineFailures}++; + $Cur->{LineFlaky}++; } elsif (($Cur->{Unit} ne "" and $Line =~ /($Cur->{UnitsRE})\.c:\d+:[0-9.]* Test marked todo: /) or @@ -714,6 +717,7 @@ sub ParseWineTestReport($$$) # There may be more than one summary line due to child processes $Cur->{Pids}->{$Pid || 0} = 1; $Cur->{SummaryFailures} += $Failures; + $Cur->{SummaryFlaky} += $Flaky; $Cur->{SummaryTodos} += $Todos; $Cur->{SummarySkips} += $Skips; $LogInfo->{IsWineTest} = 1;

1 0

Giovanni Mascellani : demos/gears: Recompile shaders with our compiler.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: vkd3d Branch: master Commit: 3d85d77cedfd9e8aece1c1feb622d82425f7e9ee URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/3d85d77cedfd9e8aece1c1feb622d… Author: Giovanni Mascellani <gmascellani(a)codeweavers.com> Date: Wed Sep 28 13:29:19 2022 +0200 demos/gears: Recompile shaders with our compiler. --- demos/gears.c | 28 +++-- demos/gears.hlsl | 55 ---------- demos/gears_hlsl.h | 56 ++++++++++ demos/gears_ps_flat.h | 73 ------------- demos/gears_ps_smooth.h | 73 ------------- demos/gears_vs.h | 272 ------------------------------------------------ 6 files changed, 75 insertions(+), 482 deletions(-)

1 0

Giovanni Mascellani : vkd3d-shader/hlsl: Write SM4 rsq instructions.
by Alexandre Julliard Oct. 10, 2022

Oct. 10, 2022

Module: vkd3d Branch: master Commit: 1655d309bd4be05f10dac6da66c64e9c3078e86b URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/1655d309bd4be05f10dac6da66c64… Author: Giovanni Mascellani <gmascellani(a)codeweavers.com> Date: Wed Sep 28 13:49:03 2022 +0200 vkd3d-shader/hlsl: Write SM4 rsq instructions. --- libs/vkd3d-shader/hlsl_sm4.c | 5 +++++ tests/hlsl-normalize.shader_test | 10 +++++----- 2 files changed, 10 insertions(+), 5 deletions(-) diff --git a/libs/vkd3d-shader/hlsl_sm4.c b/libs/vkd3d-shader/hlsl_sm4.c index 5d763c56..3ce6fc5e 100644 --- a/libs/vkd3d-shader/hlsl_sm4.c +++ b/libs/vkd3d-shader/hlsl_sm4.c @@ -1669,6 +1669,11 @@ static void write_sm4_expr(struct hlsl_ctx *ctx, write_sm4_unary_op(buffer, VKD3D_SM4_OP_ROUND_NE, &expr->node, arg1, 0); break; + case HLSL_OP1_RSQ: + assert(type_is_float(dst_type)); + write_sm4_unary_op(buffer, VKD3D_SM4_OP_RSQ, &expr->node, arg1, 0); + break; + case HLSL_OP1_SAT: assert(type_is_float(dst_type)); write_sm4_unary_op(buffer, VKD3D_SM4_OP_MOV diff --git a/tests/hlsl-normalize.shader_test b/tests/hlsl-normalize.shader_test index 3e4bb83e..71b900e8 100644 --- a/tests/hlsl-normalize.shader_test +++ b/tests/hlsl-normalize.shader_test @@ -8,7 +8,7 @@ float4 main() : SV_TARGET [test] uniform 0 float4 2.0 3.0 4.0 5.0 -todo draw quad +draw quad probe all rgba (0.272165537, 0.408248305, 0.544331074, 0.680413842) 1 [pixel shader] @@ -21,7 +21,7 @@ float4 main() : SV_TARGET [test] uniform 0 float4 2.0 3.0 4.0 0.0 -todo draw quad +draw quad probe all rgba (0.371390700, 0.557086051, 0.742781401, 0.0) 1 [pixel shader] @@ -34,7 +34,7 @@ float4 main() : SV_TARGET [test] uniform 0 float4 2.0 3.0 0.0 0.0 -todo draw quad +draw quad probe all rgba (0.554700196, 0.832050323, 0.0, 0.0) 1 [pixel shader] @@ -47,7 +47,7 @@ float4 main() : SV_TARGET [test] uniform 0 float4 2.0 0.0 0.0 0.0 -todo draw quad +draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) [pixel shader] @@ -60,7 +60,7 @@ float4 main() : SV_TARGET [test] uniform 0 float4 2.0 0.0 0.0 0.0 -todo draw quad +draw quad probe all rgba (1.0, 1.0, 1.0, 1.0) [pixel shader fail]

1 0