wine-commits February 2021

wine-commits@winehq.org

2 participants
915 discussions

Ziqing Hui : windowscodecs/tests: Fix an argument typo in ok().
by Alexandre Julliard 01 Feb '21

01 Feb '21

Module: wine Branch: master Commit: 33608a2c9ce1779dd4fe1280e6200ecbc3975906 URL: https://source.winehq.org/git/wine.git/?a=commit;h=33608a2c9ce1779dd4fe1280… Author: Ziqing Hui <zhui(a)codeweavers.com> Date: Fri Jan 29 13:02:40 2021 +0800 windowscodecs/tests: Fix an argument typo in ok(). Signed-off-by: Ziqing Hui <zhui(a)codeweavers.com> Signed-off-by: Esme Povirk <esme(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/windowscodecs/tests/ddsformat.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/windowscodecs/tests/ddsformat.c b/dlls/windowscodecs/tests/ddsformat.c index b7b002d380d..28554e77c9b 100644 --- a/dlls/windowscodecs/tests/ddsformat.c +++ b/dlls/windowscodecs/tests/ddsformat.c @@ -495,7 +495,7 @@ static IWICBitmapDecoder *create_decoder(void) hr = IWICBitmapDecoder_GetContainerFormat(decoder, &guidresult); ok(hr == S_OK, "GetContainerFormat failed, hr %#x\n", hr); ok(IsEqualGUID(&guidresult, &GUID_ContainerFormatDds), - "Got unexpected container format %s\n", debugstr_guid(&GUID_ContainerFormatDds)); + "Got unexpected container format %s\n", debugstr_guid(&guidresult)); return decoder; }

1 0

Haoyang Chen : gdiplus: Avoid negative integer overflow due to implicit type conversion.
by Alexandre Julliard 01 Feb '21

01 Feb '21

Module: wine Branch: master Commit: 4c4a5cb799f54326941f5d38eafef908867271dd URL: https://source.winehq.org/git/wine.git/?a=commit;h=4c4a5cb799f54326941f5d38… Author: Haoyang Chen <chenhaoyang(a)uniontech.com> Date: Fri Jan 29 18:08:20 2021 +0800 gdiplus: Avoid negative integer overflow due to implicit type conversion. Signed-off-by: Haoyang Chen <chenhaoyang(a)uniontech.com> Signed-off-by: Esme Povirk <esme(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/graphics.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 2e6c0d7c9c8..7b4794e3620 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -972,9 +972,9 @@ static ARGB sample_bitmap_pixel(GDIPCONST GpRect *src_rect, LPBYTE bits, UINT wi { /* Tiling. Make sure co-ordinates are positive as it simplifies the math. */ if (x < 0) - x = width*2 + x % (width * 2); + x = width*2 + x % (INT)(width * 2); if (y < 0) - y = height*2 + y % (height * 2); + y = height*2 + y % (INT)(height * 2); if (attributes->wrap & WrapModeTileFlipX) {

1 0

Francois Gouget : winetest/dissect: Don't issue multiple errors in check_unit().
by Alexandre Julliard 01 Feb '21

01 Feb '21

Module: tools Branch: master Commit: e202221249991184a8bbf974bde85acaee33d56f URL: https://source.winehq.org/git/tools.git/?a=commit;h=e202221249991184a8bbf97… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Feb 1 12:58:55 2021 +0100 winetest/dissect: Don't issue multiple errors in check_unit(). Don't complain about misplaced lines after the first one has been reported (or if the test has already been tagged as broken for some other reason, e.g. missing start line). Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- winetest/dissect | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/winetest/dissect b/winetest/dissect index 27bcf17..12a939d 100755 --- a/winetest/dissect +++ b/winetest/dissect @@ -483,7 +483,7 @@ sub add_test_line($$) sub check_unit($$) { my ($l_unit, $l_type) = @_; - if (!$units{$l_unit}) + if (!$units{$l_unit} and !$broken) { add_test_line("end", "Misplaced $l_type message\n"); $extra_failures++;

1 0

Francois Gouget : testbot/LogUtils: Always set IsWineTest in _CheckUnit().
by Alexandre Julliard 01 Feb '21

01 Feb '21

Module: tools Branch: master Commit: c9aa4dbf830563f13b53a1c7561b89569cc92412 URL: https://source.winehq.org/git/tools.git/?a=commit;h=c9aa4dbf830563f13b53a1c… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Feb 1 12:58:32 2021 +0100 testbot/LogUtils: Always set IsWineTest in _CheckUnit(). _CheckUnit() is only called on Wine test lines. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/lib/WineTestBot/LogUtils.pm | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm index 15ec31a..2f30493 100644 --- a/testbot/lib/WineTestBot/LogUtils.pm +++ b/testbot/lib/WineTestBot/LogUtils.pm @@ -384,13 +384,11 @@ sub _CheckUnit($$$$) { my ($LogInfo, $Cur, $Unit, $Type) = @_; - if ($Cur->{Units}->{$Unit} or $Cur->{Unit} eq "") - { - $LogInfo->{IsWineTest} = 1; - } - # To avoid issuing many duplicate errors, - # only report the first misplaced message. - elsif ($LogInfo->{IsWineTest} and !$Cur->{IsBroken}) + # _CheckUnit() is only called for Wine test lines. + $LogInfo->{IsWineTest} = 1; + + # Only report the first misplaced message to avoid duplicate errors. + if (!$Cur->{Units}->{$Unit} and $Cur->{Unit} ne "" and !$Cur->{IsBroken}) { _AddExtra($LogInfo, "contains a misplaced $Type message for $Unit", $Cur); $Cur->{IsBroken} = 1;

1 0

Francois Gouget : testbot/TestWTBS: Allow mission-specific directives, e.g. for test=module.
by Alexandre Julliard 01 Feb '21

01 Feb '21

Module: tools Branch: master Commit: 6c4b3a308721c8c21806781de0ec4af03288fb7f URL: https://source.winehq.org/git/tools.git/?a=commit;h=6c4b3a308721c8c21806781… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Feb 1 12:02:45 2021 +0100 testbot/TestWTBS: Allow mission-specific directives, e.g. for test=module. A different set of tests are run depending on whether a VM's mission is test=test (the default) or test=module. Being unable to distinguish these two cases prevents checking that the right set of test units is being run in many cases. So add support for the wine:build, wine:test, wine:module categories to allow providing specific checks for the test=build/test/module cases respectively. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/tests/TestWTBS | 184 +++++++++++++++++++++++++++++++++++-------------- 1 file changed, 131 insertions(+), 53 deletions(-) Diff: https://source.winehq.org/git/tools.git/?a=commitdiff;h=6c4b3a308721c8c2180…

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits February 2021