February 2017 - Wine-Commits

Aurimas Fišeras : po: Update Lithuanian translation.
by Alexandre Julliard Feb. 20, 2017

Feb. 20, 2017

Module: wine Branch: master Commit: 176283b082e294a9618742087de07814c4859391 URL: http://source.winehq.org/git/wine.git/?a=commit;h=176283b082e294a9618742087… Author: Aurimas Fišeras <aurimas(a)members.fsf.org> Date: Sat Feb 18 08:34:25 2017 +0200 po: Update Lithuanian translation. Signed-off-by: Aurimas Fišeras <aurimas(a)members.fsf.org> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- po/lt.po | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/po/lt.po b/po/lt.po index 3bb677a..392eda3 100644 --- a/po/lt.po +++ b/po/lt.po @@ -5,7 +5,7 @@ msgstr "" "Project-Id-Version: Wine\n" "Report-Msgid-Bugs-To: http://bugs.winehq.org\n" "POT-Creation-Date: N/A\n" -"PO-Revision-Date: 2017-02-11 08:54+0300\n" +"PO-Revision-Date: 2017-02-18 08:33+0300\n" "Last-Translator: Aurimas Fišeras <aurimas(a)members.fsf.org>\n" "Language-Team: Lithuanian <komp_lt(a)konf.lt>\n" "Language: lt\n" @@ -12226,17 +12226,13 @@ msgid "Exports selected branch of the registry to a text file" msgstr "Eksportuoja pasirinktą registro šaką į tekstinį failą" #: regedit.rc:187 -#, fuzzy -#| msgid "Confirm Value Delete" msgid "Confirm Key Delete" -msgstr "Reikšmės šalinimo patvirtinimas" +msgstr "Rakto šalinimo patvirtinimas" #: regedit.rc:188 -#, fuzzy -#| msgid "Are you sure you want to delete the registry key '%1'?" msgid "" "Are you sure you want to delete the registry key '%1' and all of its subkeys?" -msgstr "Ar tikrai norite pašalinti registro raktą „%1“?" +msgstr "Ar tikrai norite pašalinti registro raktą „%1“ ir visus jo porakčius?" #: regsvr32.rc:32 msgid ""

1 0

Francois Gouget : odbccp32: Add a __WINE_ALLOC_SIZE attribute to heap_alloc().
by Alexandre Julliard Feb. 20, 2017

Feb. 20, 2017

Module: wine Branch: master Commit: 65b6cf186d3ea221db936abf504796b38bcc3037 URL: http://source.winehq.org/git/wine.git/?a=commit;h=65b6cf186d3ea221db936abf5… Author: Francois Gouget <fgouget(a)free.fr> Date: Sat Feb 18 11:21:10 2017 +0100 odbccp32: Add a __WINE_ALLOC_SIZE attribute to heap_alloc(). And standardize its parameter name. Signed-off-by: Francois Gouget <fgouget(a)free.fr> Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/odbccp32/odbccp32.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/odbccp32/odbccp32.c b/dlls/odbccp32/odbccp32.c index bc643d4..f68d98f 100644 --- a/dlls/odbccp32/odbccp32.c +++ b/dlls/odbccp32/odbccp32.c @@ -76,9 +76,9 @@ static void clear_errors(void) num_errors = 0; } -static inline void * heap_alloc(size_t len) +static inline void* __WINE_ALLOC_SIZE(1) heap_alloc(size_t size) { - return HeapAlloc(GetProcessHeap(), 0, len); + return HeapAlloc(GetProcessHeap(), 0, size); } static inline BOOL heap_free(void *mem)

1 0

Francois Gouget : testbot/reporttest: Some fixes and additions for the report template.
by Alexandre Julliard Feb. 20, 2017

Feb. 20, 2017

Module: tools Branch: master Commit: 1239052d91cdfcb770964b007ea5ae8d89a61dea URL: http://source.winehq.org/git/tools.git/?a=commit;h=1239052d91cdfcb770964b00… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sat Feb 18 11:03:59 2017 +0100 testbot/reporttest: Some fixes and additions for the report template. Some tests had copy/paste errors which were caught by an actual parser. Added a few more tests for misplaced lines, exceptions in subprocesses, and races in the test output. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/src/reporttest/report.template | 121 ++++++++++++++++++++++++++------- 1 file changed, 98 insertions(+), 23 deletions(-) diff --git a/testbot/src/reporttest/report.template b/testbot/src/reporttest/report.template index 003d41e..cb83ad7 100644 --- a/testbot/src/reporttest/report.template +++ b/testbot/src/reporttest/report.template @@ -296,6 +296,7 @@ service.c:20: Test failed: A failure in the subprocess advpack:advpack start dlls/advpack/tests/advpack.c - ----- A standard successful unit test ----- Expected assessement: Success, unless the previous test interferes +0da0:advpack: 99 tests executed (0 marked as todo, 0 failures), 0 skipped. advpack:advpack:0da0 done (0) in 0s advpack:files start dlls/advapi32/tests/crypt.c - @@ -309,7 +310,7 @@ advpack:files:0880 done (3) in 0s advpack:install start dlls/advpack/tests/install.c - ----- A unit test with a crash ----- Expected assessement: Crash -install: unhandled exception c0000005 at 0040167C +07b0:install: unhandled exception c0000005 at 0040167C advpack:install:07b0 done (3221225477) in 0s amstream:amstream start dlls/amstream/tests/amstream.c - @@ -339,13 +340,14 @@ atl:module:0920 done (0) in 0s atl:registrar start dlls/atl/tests/registrar.c - ----- A unit test with a foreign test summary line ----- Expected assessement: "Misplaced test summary line" error -9876:service: 10 tests executed (0 marked as todo, 0 failures), 0 skipped. +9876:service: 10 tests executed (0 marked as todo, 1 failures), 0 skipped. 0922:registrar: 17 tests executed (0 marked as todo, 0 failures), 0 skipped. atl:registrar:0922 done (0) in 0s ------ Skipping all tests for a missing dll (see header) ------ Expected assessement: Skipped (FIXME: or Not Run?) -avifil32:api skipped dlls/avifil32/tests/api.c - +----- Skipping all tests for a missing dll (see the "Dll info:" header) +----- Expected assessement: Not available +avifil32 is missing and thus no test will be run for it and no entry will be +found in the test report. browseui:autocomplete start dlls/browseui/tests/autocomplete.c - ----- A unit test with a skipped test @@ -408,17 +410,16 @@ This line says nothing is wrong because it's from the subprocess comctl32:comboex:0c9c done (1) in 0s comctl32:datetime start dlls/comctl32/tests/datetime.c - ------ A unit test with a missing test failure line +----- A unit test reporting a failure but with no test failed line ----- Expected assessement: "Missing test failed line" error -datetime.c:1: Test failed: Something wrong -0998:datetime: 651 tests executed (0 marked as todo, 0 failures), 0 skipped. -comctl32:datetime:0998 done (0) in 0s +0998:datetime: 651 tests executed (0 marked as todo, 1 failures), 0 skipped. +comctl32:datetime:0998 done (1) in 0s comctl32:dpa start dlls/comctl32/tests/dpa.c - ----- A unit test with a non-zero exit code ----- Expected assessement: "Non-zero exit code" error This would cause things like 'make test' to consider the unit test failed. -0b14:dpa: 16 tests executed (0 marked as todo, 0 failures), 0 skipped. +0998:dpa: 16 tests executed (0 marked as todo, 0 failures), 0 skipped. comctl32:dpa:0998 done (1) in 0s comctl32:header start dlls/comctl32/tests/header.c - @@ -426,7 +427,7 @@ comctl32:header start dlls/comctl32/tests/header.c - ----- Expected assessement: "Zero exit code" error A non-zero exit code is necessary for 'make test'. header.c:1: Test failed: Something wrong -0b14:header: 16 tests executed (0 marked as todo, 1 failure), 0 skipped. +0998:header: 16 tests executed (0 marked as todo, 1 failure), 0 skipped. comctl32:header:0998 done (0) in 0s comctl32:imagelist start - - @@ -449,16 +450,90 @@ stub comctl32:listview 0998:misc: 16 tests executed (0 marked as todo, 0 failure), 0 skipped. comctl32:misc:0998 done (0) in 0s -stub comctl32:monthcal -stub comctl32:mru -stub comctl32:pager -stub comctl32:progress -stub comctl32:propsheet -stub comctl32:rebar -stub comctl32:status -stub comctl32:subclass -stub comctl32:syslink -stub comctl32:tab +comctl32:monthcal start dlls/comctl32/tests/monthcal.c - +----- A unit test with a misplaced todo subprocess test summary line +----- Expected assessement: "Misplaced summary line" error +See the previous misplaced test summary line scenarios. +0148:monthcal: 2251 tests executed (0 marked as todo, 0 failures), 0 skipped. +comctl32:monthcal:0148 done (0) in 9s +security.c:41: Test marked todo: Hello Wine! +0955:security: 18 tests executed (1 marked as todo, 0 failures), 0 skipped. + +comctl32:mru start dlls/comctl32/tests/mru.c - +----- A standard successful unit test +----- Expected assessement: Success, unless the previous test interferes +0148:mru: 2251 tests executed (0 marked as todo, 0 failures), 0 skipped. +comctl32:mru:0148 done (0) in 9s + +comctl32:pager start dlls/comctl32/tests/pager.c - +----- A unit test with a misplaced subprocess skip test summary line +----- Expected assessement: Success +See the previous misplaced test summary line scenarios. +0148:pager: 2251 tests executed (0 marked as todo, 0 failures), 0 skipped. +comctl32:pager:0148 done (0) in 9s +security.c:42: Tests skipped: Another time +0956:security: 18 tests executed (0 marked as todo, 0 failures), 1 skipped. + +comctl32:progress start dlls/comctl32/tests/progress.c - +----- A standard successful unit test +----- Expected assessement: Success +0148:progress: 2251 tests executed (0 marked as todo, 0 failures), 0 skipped. +comctl32:progress:0148 done (0) in 9s + +comctl32:propsheet start dlls/comctl32/tests/propsheet.c - +----- A unit test with no main process test summary line and no pid anywhere +----- Expected assessement: "Missing test summary" error +comctl32:propsheet done (0) in 9s + +comctl32:rebar start dlls/comctl32/tests/rebar.c - +----- A unit test with an unreported crash in a subprocess +----- Expected assessement: "Zero exit code" error +Even in subprocesses crashes are equivalent to 'Test failed' errors and as +such the main process cannot report success, i.e. it must have a non-zero exit +code. Note also that since we don't require the main process to account for the +subprocess failures in its test summary line, it's ok for it not to account for +subprocess crashes either. This is why the error is "Zero exit code" and not +"Unreported failure". +1234:rebar: unhandled exception c0000005 at 0040167C +07b0:rebar: 2251 tests executed (0 marked as todo, 0 failures), 0 skipped. +comctl32:rebar:07b0 done (0) in 0s + +comctl32:status start dlls/comctl32/tests/status.c - +----- A unit test with a subprocess crash but no pid +----- Expected assessement: "Zero exit code" error +See the comctl32:rebar comment. +status: unhandled exception c0000005 at 0040167C +status: 2251 tests executed (0 marked as todo, 0 failures), 0 skipped. +comctl32:status:07b0 done (0) in 0s + +comctl32:subclass start dlls/comctl32/tests/subclass.c - +----- A unit test with a subprocess crash, no pid and a non-zero exit code +----- Expected assessement: 1 failure +See the comctl32:rebar comment. Then note that here the main process does not +claim success. Also note that the exit code does not correspond to that of a +crashed process which makes sense since it did not crash. +subclass: unhandled exception c0000005 at 0040167C +subclass: 2251 tests executed (0 marked as todo, 0 failures), 0 skipped. +comctl32:subclass:07b0 done (1) in 0s + +comctl32:syslink start dlls/comctl32/tests/syslink.c - +----- A unit test with a garbled test failed line +----- Expected assessement: 1 failure +Multi-threaded and multi-process tests should really try to avoid races in +the report. But analysis tools should also try to be resilient. +Do not cut syslink.c:40: Test failed: Something wrong +me off +07b0:syslink: 2251 tests executed (0 marked as todo, 1 failures), 0 skipped. +comctl32:syslink:07b0 done (1) in 0s + +comctl32:tab start dlls/comctl32/tests/tab.c - +----- A unit test with a garbled test summary line +----- Expected assessement: Success +See previous comment. +Do not cut 07b0:tab: 2251 tests executed (0 marked as todo, 0 failures), 0 skipped. +me off +comctl32:tab:07b0 done (0) in 0s + stub comctl32:toolbar stub comctl32:tooltips stub comctl32:trackbar @@ -946,8 +1021,8 @@ stub xmllite:reader stub xmllite:writer xmllite:writer start fake/source/writer.c - ------ A unit test with a misplaced and failed subprocess test summary line ------ Expected assessement: "Misplaced test summary line" error +----- A unit test with a misplaced test failed line +----- Expected assessement: "Misplaced test failed line" error See the similar scenario before but note the lack of the extra test summary line here. Also beware of the end-of-file. 037c:writer: 1930 tests executed (0 marked as todo, 0 failures), 0 skipped.

1 0

Francois Gouget : testbot/WineRunTask: Better check the task report.
by Alexandre Julliard Feb. 20, 2017

Feb. 20, 2017

Module: tools Branch: master Commit: 75f60f8068e640281ff4e624ebafb11bc604767a URL: http://source.winehq.org/git/tools.git/?a=commit;h=75f60f8068e640281ff4e624… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sat Feb 18 11:09:18 2017 +0100 testbot/WineRunTask: Better check the task report. This makes the WineRunTask analysis match the expected results on the report test. In particular it now leverages the pid traces to detect when a test has no test summary line for its main process. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/bin/WineRunTask.pl | 155 ++++++++++++++++++++++++++++++++++++++++----- 1 file changed, 139 insertions(+), 16 deletions(-) diff --git a/testbot/bin/WineRunTask.pl b/testbot/bin/WineRunTask.pl index e109874..c21aec0 100755 --- a/testbot/bin/WineRunTask.pl +++ b/testbot/bin/WineRunTask.pl @@ -459,34 +459,157 @@ if ($TA->GetFile($RptFileName, $FullLogFileName)) chmod 0664, $FullLogFileName; if (open(my $LogFile, "<", $FullLogFileName)) { - my $LogFailures; + # Note that for the TestBot we don't really care about the todos and skips + my ($CurrentDll, $CurrentUnit, $LineFailures, $SummaryFailures) = ("", "", 0, 0); + my ($CurrentIsPolluted, %CurrentPids, $LogFailures); foreach my $Line (<$LogFile>) { - # There may be more than one result line due to child processes - if ($Line =~ /: \d+ tests? executed $\d+ marked as todo, (\d+) failures?$, \d+ skipped\./) + # There may be more than one summary line due to child processes + if ($Line =~ m%([_.a-z0-9]+):([_a-z0-9]+) start (?:-|[/_.a-z0-9]+) (?:-|[.0-9a-f]+)\r?$%) { - $LogFailures += $1; + my ($Dll, $Unit) = ($1, $2); + if ($CurrentDll ne "") + { + LogTaskError("The done line is missing for $CurrentDll:$CurrentUnit\n"); + $LogFailures++; + } + ($CurrentDll, $CurrentUnit) = ($Dll, $Unit); } - elsif ($Line =~ / done $258$/ or - $Line =~ /: unhandled exception [0-9a-fA-F]{8} at /) + elsif ($Line =~ /^([_a-z0-9]+)\.c:\d+: Test failed: / or + ($CurrentUnit ne "" and + $Line =~ /($CurrentUnit)\.c:\d+: Test failed: /)) { - $LogFailures++; + # If the failure is not for the current test unit we'll let its + # developer hash it out with the polluter ;-) + $CurrentIsPolluted = 1 if ($1 ne $CurrentUnit); + $LineFailures++; + } + elsif ($Line =~ /^(?:([0-9a-f]+):)?([_.a-z0-9]+): unhandled exception [0-9a-fA-F]{8} at / or + ($CurrentUnit ne "" and + $Line =~ /(?:([0-9a-f]+):)?($CurrentUnit): unhandled exception [0-9a-fA-F]{8} at /)) + { + my ($Pid, $Unit) = ($1, $2); + + if ($Unit eq $CurrentUnit) + { + # This also replaces a test summary line. + $CurrentPids{$Pid || 0} = 1; + $SummaryFailures++; + } + else + { + $CurrentIsPolluted = 1; + } + $LineFailures++; + } + elsif ($Line =~ /^(?:([0-9a-f]+):)?([_a-z0-9]+): \d+ tests? executed $(\d+) marked as todo, (\d+) failures?$, \d+ skipped\./ or + ($CurrentUnit ne "" and + $Line =~ /(?:([0-9a-f]+):)?($CurrentUnit): \d+ tests? executed $(\d+) marked as todo, (\d+) failures?$, \d+ skipped\./)) + { + my ($Pid, $Unit, $Todo, $Failures) = ($1, $2, $3, $4); + + if ($Unit eq $CurrentUnit) + { + $CurrentPids{$Pid || 0} = 1; + $SummaryFailures += $Failures; + } + else + { + $CurrentIsPolluted = 1; + if ($Todo or $Failures) + { + LogTaskError("Found a misplaced '$Unit' test summary line.\n"); + $LogFailures++; + } + } + } + elsif ($Line =~ /^([_.a-z0-9]+):([_a-z0-9]+)(?::([0-9a-f]+))? done $(-?\d+)$(?:\r?$| in)/) + { + my ($Dll, $Unit, $Pid, $Rc) = ($1, $2, $3, $4); + + if ($Dll ne $CurrentDll or $Unit ne $CurrentUnit) + { + LogTaskError("The start line is missing for $Dll:$Unit\n"); + $LogFailures++; + $CurrentIsPolluted = 1; + } + + # Verify the summary lines + if (!$CurrentIsPolluted) + { + if ($LineFailures != 0 and $SummaryFailures == 0) + { + LogTaskError("$Dll:$Unit has unreported failures\n"); + $LogFailures++; + } + elsif ($LineFailures == 0 and $SummaryFailures != 0) + { + LogTaskError("Some test failed messages are missing for $Dll:$Unit\n"); + $LogFailures++; + } + } + # Note that $SummaryFailures may count some failures twice so only use + # it as a fallback for $LineFailures. + $LineFailures ||= $SummaryFailures; + + if (!$CurrentIsPolluted) + { + # Verify the exit code + if ($LineFailures != 0 and $Rc == 0) + { + LogTaskError("$Dll:$Unit returned success despite having failures\n"); + $LogFailures++; + } + elsif ($Rc == 258) + { + # This is a timeout + $LogFailures++; + } + elsif ($LineFailures == 0 and $Rc != 0) + { + LogTaskError("$Dll:$Unit returned an error ($Rc) despite having no failures\n"); + $LogFailures++; + } + } + + if ($Rc != 258 and + ((!$Pid and !%CurrentPids) or + ($Pid and !$CurrentPids{$Pid} and !$CurrentPids{0}))) + { + LogTaskError("$Dll:$Unit has no test summary line (early exit of the main process?)\n"); + $LogFailures++; + } + + $LogFailures += $LineFailures; + + $CurrentDll = $CurrentUnit = ""; + %CurrentPids = (); + $CurrentIsPolluted = $LineFailures = $SummaryFailures = 0; } } close($LogFile); - if (defined $LogFailures) + + if ($TimedOut) + { + # The report got truncated due to the timeout so ignore the other checks + } + elsif (!defined $LogFailures) + { + LogTaskError("Found no trace of a test summary line or of a crash\n"); + $LogFailures = 1; + } + elsif ($CurrentDll ne "") { - $TestFailures += $LogFailures; - # The log file looks good so ignore the TestAgent error - # but still log it in case the log was in fact truncated. - LogTaskError($TAError) if (defined $TAError); - $TAError = undef; + LogTaskError("The report seems to have been truncated\n"); + $LogFailures++; } - elsif (!defined $LogFailures and !defined $TestFailures) + elsif ($LineFailures or $SummaryFailures) { - LogTaskError("Found no trace of the test summary line or of a crash\n"); - $TestFailures = 1; + LogTaskError("Found a trailing test summary or test failed line.\n"); + $LogFailures++; } + # $LogFailures can legitimately be undefined in case of a timeout + $TestFailures += $LogFailures || 0; } else {

1 0

Francois Gouget : testbot/reporttest: Standardize on 'test summary' for consistency.
by Alexandre Julliard Feb. 20, 2017

Feb. 20, 2017

Module: tools Branch: master Commit: f15df072c11716f97b140da393dc79d3f198591c URL: http://source.winehq.org/git/tools.git/?a=commit;h=f15df072c11716f97b140da3… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Sat Feb 18 11:03:30 2017 +0100 testbot/reporttest: Standardize on 'test summary' for consistency. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- testbot/src/reporttest/report.template | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/testbot/src/reporttest/report.template b/testbot/src/reporttest/report.template index 2e891f2..003d41e 100644 --- a/testbot/src/reporttest/report.template +++ b/testbot/src/reporttest/report.template @@ -227,7 +227,7 @@ crypt_lmhash.c:20: Test failed: Second failure here advapi32:crypt_lmhash:07c0 done (1) in 0s advapi32:crypt_md4 start dlls/advapi32/tests/crypt_md4.c - ------ A unit test with two test result lines +----- A unit test with two test summary lines ----- Expected assessement: Success 0123:crypt_md4: 11 tests executed (0 marked as todo, 0 failures), 0 skipped. 0ca0:crypt_md4: 11 tests executed (0 marked as todo, 0 failures), 0 skipped. @@ -240,7 +240,7 @@ crypt_md5.c:20: Test failed: A failure here, presumably in the subprocess 0123:crypt_md5: 11 tests executed (0 marked as todo, 1 failure), 0 skipped. The parent now counts the subprocess failure as its own. Note that it means the 'Test failed' count may legitimately differ from the sum from the test -result lines. +summary lines. Note that this also applies to the todo and skip counts. 087c:crypt_md5: 5 tests executed (0 marked as todo, 1 failure), 0 skipped. advapi32:crypt_md5:087c done (3) in 0s @@ -263,18 +263,18 @@ eventlog.c:20: Test failed: Something wrong advapi32:eventlog:088c done (3) in 4s advapi32:lsa start dlls/advapi32/tests/crypt.c - ------ A unit test with no test result line ------ Expected assessement: "Missing test result" error +----- A unit test with no test summary line +----- Expected assessement: "Missing test summary" error advapi32:lsa:083c done (0) in 0s advapi32:registry start dlls/advapi32/tests/registry.c - ------ A unit test with no main process test result line ------ Expected assessement: "Missing test result" error +----- A unit test with no main process test summary line +----- Expected assessement: "Missing test summary" error 1234:registry: 765 tests executed (0 marked as todo, 0 failures), 0 skipped. advapi32:registry:0804 done (0) in 0s advapi32:security start dlls/advapi32/tests/security.c - ------ A unit test with a misplaced but successful subprocess test result line +----- A unit test with a misplaced but successful subprocess test summary line ----- Expected assessement: Success The subprocess may be a server process that exits automatically once the test is complete. That's ok as long as it's not meant to report failures. @@ -283,8 +283,8 @@ advapi32:security:0148 done (0) in 9s 0954:security: 18 tests executed (0 marked as todo, 0 failures), 0 skipped. advapi32:service start dlls/advapi32/tests/service.c - ------ A unit test with a misplaced and failed subprocess test result line ------ Expected assessement: "Misplaced test result line" error +----- A unit test with a misplaced and failed subprocess test summary line +----- Expected assessement: "Misplaced test summary line" error Same scenario as above but the subprocess can actually report failures so the main process must wait for it. Ideally the error should be reported against this unit test but reporting against the next one is also acceptable. @@ -325,20 +325,20 @@ apphelp:apphelp start dlls/apphelp/tests/apphelp.c - apphelp:apphelp:0860 done (0) atl:atl_ax start dlls/atl/tests/atl_ax.c - ------ Pid on the result line, but no pid on the done one +----- Pid on the test summary line, but no pid on the done one ----- Expected assessement: Success 0938:atl_ax: 12 tests executed (0 marked as todo, 0 failures), 0 skipped. atl:atl_ax done (0) in 0s atl:module start dlls/atl/tests/module.c - ------ Pid on the done line, but no pid on the result line +----- Pid on the done line, but no pid on the test summary line ----- Expected assessement: Success module: 544 tests executed (0 marked as todo, 0 failures), 0 skipped. atl:module:0920 done (0) in 0s atl:registrar start dlls/atl/tests/registrar.c - ------ A unit test with a foreign test result line ------ Expected assessement: "Misplaced test result line" error +----- A unit test with a foreign test summary line +----- Expected assessement: "Misplaced test summary line" error 9876:service: 10 tests executed (0 marked as todo, 0 failures), 0 skipped. 0922:registrar: 17 tests executed (0 marked as todo, 0 failures), 0 skipped. atl:registrar:0922 done (0) in 0s @@ -946,9 +946,9 @@ stub xmllite:reader stub xmllite:writer xmllite:writer start fake/source/writer.c - ------ A unit test with a misplaced and failed subprocess test result line ------ Expected assessement: "Misplaced test result line" error -See the similar scenario before but note the lack of the extra test result line +----- A unit test with a misplaced and failed subprocess test summary line +----- Expected assessement: "Misplaced test summary line" error +See the similar scenario before but note the lack of the extra test summary line here. Also beware of the end-of-file. 037c:writer: 1930 tests executed (0 marked as todo, 0 failures), 0 skipped. xmllite:writer:037c done (0) in 11s

1 0

Alexandre Julliard : Wine release 2.2
by Alexandre Julliard Feb. 17, 2017

Feb. 17, 2017

Module: website Branch: master Commit: 7bf5ec42847de99e60a5f588c42e8043d50ca56b URL: http://source.winehq.org/git/website.git/?a=commit;h=7bf5ec42847de99e60a5f5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Feb 17 21:37:22 2017 +0100 Wine release 2.2 Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- news/en/2017021701.xml | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/news/en/2017021701.xml b/news/en/2017021701.xml new file mode 100644 index 0000000..491ee8d --- /dev/null +++ b/news/en/2017021701.xml @@ -0,0 +1,16 @@ +<news> +<date>February 17, 2017</date> +<title>Wine 2.2 Released</title> +<body> + The Wine development release 2.2 is now available. + <a href="{$root}/announce/2.2">What's new</a> in this release: +<ul> + <li>Windows version set to Windows 7 for new prefixes.</li> + <li>More steps towards the Direct3D command stream.</li> + <li>Still more Shader Model 5 instructions.</li> + <li>Initial support for double-buffered theme painting.</li> + <li>Various bug fixes.</li> +</ul> +The source is <a href="//dl.winehq.org/wine/source/2.x/wine-2.2.tar.xz">available now</a>. +Binary packages are in the process of being built, and will appear soon at their respective <a href="{$root}/download">download locations</a>. +</body></news>

1 0

Jacek Caban : mshtml: Don't use attr_name in attr_to_eid.
by Alexandre Julliard Feb. 17, 2017

Feb. 17, 2017

Module: wine Branch: master Commit: e89c279d1ce7d047756e70e1013ccd930f8405fb URL: http://source.winehq.org/git/wine.git/?a=commit;h=e89c279d1ce7d047756e70e10… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Feb 17 13:17:09 2017 +0100 mshtml: Don't use attr_name in attr_to_eid. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index 7c25ad9..c776ee3 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -268,12 +268,15 @@ eventid_t str_to_eid(LPCWSTR str) return EVENTID_LAST; } -static eventid_t attr_to_eid(LPCWSTR str) +static eventid_t attr_to_eid(const WCHAR *str) { int i; + if((str[0] != 'o' && str[0] != 'O') || (str[1] != 'n' && str[1] != 'N')) + return EVENTID_LAST; + for(i=0; i < sizeof(event_info)/sizeof(event_info[0]); i++) { - if(!strcmpW(event_info[i].attr_name, str)) + if(!strcmpW(event_info[i].name, str+2)) return i; }

1 0

Henri Verbeet : wined3d: Introduce a helper function to enable lights.
by Alexandre Julliard Feb. 17, 2017

Feb. 17, 2017

Module: wine Branch: master Commit: 03a046f96713cff235f3c8583e0547955070266c URL: http://source.winehq.org/git/wine.git/?a=commit;h=03a046f96713cff235f3c8583… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Feb 17 14:08:18 2017 +0100 wined3d: Introduce a helper function to enable lights. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/device.c | 65 ++++-------------------------------------- dlls/wined3d/stateblock.c | 45 +++++++++++++++++++++++++++++ dlls/wined3d/wined3d_private.h | 2 ++ 3 files changed, 53 insertions(+), 59 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index e68827c..2c8da30 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -1678,6 +1678,7 @@ HRESULT CDECL wined3d_device_get_light(const struct wined3d_device *device, HRESULT CDECL wined3d_device_set_light_enable(struct wined3d_device *device, UINT light_idx, BOOL enable) { struct wined3d_light_info *light_info; + int prev_idx; TRACE("device %p, light_idx %u, enable %#x.\n", device, light_idx, enable); @@ -1694,66 +1695,12 @@ HRESULT CDECL wined3d_device_set_light_enable(struct wined3d_device *device, UIN } } - if (!enable) + prev_idx = light_info->glIndex; + wined3d_state_enable_light(device->update_state, &device->adapter->d3d_info, light_info, enable); + if (!device->recording && light_info->glIndex != prev_idx) { - if (light_info->glIndex != -1) - { - if (!device->recording) - { - device_invalidate_state(device, STATE_LIGHT_TYPE); - device_invalidate_state(device, STATE_ACTIVELIGHT(light_info->glIndex)); - } - - device->update_state->lights[light_info->glIndex] = NULL; - light_info->glIndex = -1; - } - else - { - TRACE("Light already disabled, nothing to do\n"); - } - light_info->enabled = FALSE; - } - else - { - light_info->enabled = TRUE; - if (light_info->glIndex != -1) - { - TRACE("Nothing to do as light was enabled\n"); - } - else - { - unsigned int light_count = device->adapter->d3d_info.limits.active_light_count; - unsigned int i; - - /* Find a free light. */ - for (i = 0; i < light_count; ++i) - { - if (!device->update_state->lights[i]) - { - device->update_state->lights[i] = light_info; - light_info->glIndex = i; - break; - } - } - if (light_info->glIndex == -1) - { - /* Our tests show that Windows returns D3D_OK in this situation, even with - * D3DCREATE_HARDWARE_VERTEXPROCESSING | D3DCREATE_PUREDEVICE devices. This - * is consistent among ddraw, d3d8 and d3d9. GetLightEnable returns TRUE - * as well for those lights. - * - * TODO: Test how this affects rendering. */ - WARN("Too many concurrently active lights\n"); - return WINED3D_OK; - } - - /* i == light_info->glIndex */ - if (!device->recording) - { - device_invalidate_state(device, STATE_LIGHT_TYPE); - device_invalidate_state(device, STATE_ACTIVELIGHT(i)); - } - } + device_invalidate_state(device, STATE_LIGHT_TYPE); + device_invalidate_state(device, STATE_ACTIVELIGHT(enable ? light_info->glIndex : prev_idx)); } return WINED3D_OK; diff --git a/dlls/wined3d/stateblock.c b/dlls/wined3d/stateblock.c index 70e31a2..22e511f 100644 --- a/dlls/wined3d/stateblock.c +++ b/dlls/wined3d/stateblock.c @@ -575,6 +575,51 @@ struct wined3d_light_info *wined3d_state_get_light(const struct wined3d_state *s return NULL; } +void wined3d_state_enable_light(struct wined3d_state *state, const struct wined3d_d3d_info *d3d_info, + struct wined3d_light_info *light_info, BOOL enable) +{ + unsigned int light_count, i; + + if (!(light_info->enabled = enable)) + { + if (light_info->glIndex == -1) + { + TRACE("Light already disabled, nothing to do.\n"); + return; + } + + state->lights[light_info->glIndex] = NULL; + light_info->glIndex = -1; + return; + } + + if (light_info->glIndex != -1) + { + TRACE("Light already enabled, nothing to do.\n"); + return; + } + + /* Find a free light. */ + light_count = d3d_info->limits.active_light_count; + for (i = 0; i < light_count; ++i) + { + if (state->lights[i]) + continue; + + state->lights[i] = light_info; + light_info->glIndex = i; + return; + } + + /* Our tests show that Windows returns D3D_OK in this situation, even with + * D3DCREATE_HARDWARE_VERTEXPROCESSING | D3DCREATE_PUREDEVICE devices. + * This is consistent among ddraw, d3d8 and d3d9. GetLightEnable returns + * TRUE * as well for those lights. + * + * TODO: Test how this affects rendering. */ + WARN("Too many concurrently active lights.\n"); +} + static void wined3d_state_record_lights(struct wined3d_state *dst_state, const struct wined3d_state *src_state) { UINT i; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 585055c..4c70c97 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -3140,6 +3140,8 @@ struct wined3d_stateblock void stateblock_init_contained_states(struct wined3d_stateblock *stateblock) DECLSPEC_HIDDEN; void state_cleanup(struct wined3d_state *state) DECLSPEC_HIDDEN; +void wined3d_state_enable_light(struct wined3d_state *state, const struct wined3d_d3d_info *d3d_info, + struct wined3d_light_info *light_info, BOOL enable) DECLSPEC_HIDDEN; struct wined3d_light_info *wined3d_state_get_light(const struct wined3d_state *state, unsigned int idx) DECLSPEC_HIDDEN; void state_init(struct wined3d_state *state, struct wined3d_fb_state *fb,

1 0

Jacek Caban : mshtml: Added get_event_dispex_ref helper that avoids using attr_name.
by Alexandre Julliard Feb. 17, 2017

Feb. 17, 2017

Module: wine Branch: master Commit: 607a8d0121dcdee783f808eeb9dc7e32ac47f936 URL: http://source.winehq.org/git/wine.git/?a=commit;h=607a8d0121dcdee783f808eeb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Feb 17 13:17:02 2017 +0100 mshtml: Added get_event_dispex_ref helper that avoids using attr_name. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/htmlevent.c | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/dlls/mshtml/htmlevent.c b/dlls/mshtml/htmlevent.c index 13fda24..7c25ad9 100644 --- a/dlls/mshtml/htmlevent.c +++ b/dlls/mshtml/htmlevent.c @@ -1396,13 +1396,22 @@ void detach_events(HTMLDocumentNode *doc) release_nsevents(doc); } +static HRESULT get_event_dispex_ref(EventTarget *event_target, eventid_t eid, BOOL alloc, VARIANT **ret) +{ + WCHAR buf[64]; + buf[0] = 'o'; + buf[1] = 'n'; + strcpyW(buf+2, event_info[eid].name); + return dispex_get_dprop_ref(&event_target->dispex, buf, alloc, ret); +} + static void remove_event_handler(EventTarget *event_target, eventid_t eid) { handler_vector_t *handler_vector; VARIANT *store; HRESULT hres; - hres = dispex_get_dprop_ref(&event_target->dispex, event_info[eid].attr_name, FALSE, &store); + hres = get_event_dispex_ref(event_target, eid, FALSE, &store); if(SUCCEEDED(hres)) VariantClear(store); @@ -1458,7 +1467,7 @@ HRESULT set_event_handler(EventTarget *event_target, eventid_t eid, VARIANT *var */ remove_event_handler(event_target, eid); - hres = dispex_get_dprop_ref(&event_target->dispex, event_info[eid].attr_name, TRUE, &v); + hres = get_event_dispex_ref(event_target, eid, TRUE, &v); if(FAILED(hres)) return hres; @@ -1485,7 +1494,7 @@ HRESULT get_event_handler(EventTarget *event_target, eventid_t eid, VARIANT *var VARIANT *v; HRESULT hres; - hres = dispex_get_dprop_ref(&event_target->dispex, event_info[eid].attr_name, FALSE, &v); + hres = get_event_dispex_ref(event_target, eid, FALSE, &v); if(SUCCEEDED(hres) && V_VT(v) != VT_EMPTY) { V_VT(var) = VT_EMPTY; return VariantCopy(var, v);

1 0

Piotr Caban : msvcr120: Add _except1 implementation.
by Alexandre Julliard Feb. 17, 2017

Feb. 17, 2017

Module: wine Branch: master Commit: 10200af74142c786ca8c7594bfc1fb3ad9cb0ab3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=10200af74142c786ca8c7594b… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri Feb 17 15:54:25 2017 +0100 msvcr120: Add _except1 implementation. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- .../api-ms-win-crt-math-l1-1-0.spec | 2 +- dlls/msvcr120/msvcr120.spec | 2 +- dlls/msvcr120_app/msvcr120_app.spec | 2 +- dlls/msvcrt/math.c | 92 ++++++++++++++++++++++ dlls/ucrtbase/ucrtbase.spec | 2 +- 5 files changed, 96 insertions(+), 4 deletions(-) diff --git a/dlls/api-ms-win-crt-math-l1-1-0/api-ms-win-crt-math-l1-1-0.spec b/dlls/api-ms-win-crt-math-l1-1-0/api-ms-win-crt-math-l1-1-0.spec index 8ff4c58..d5ade42 100644 --- a/dlls/api-ms-win-crt-math-l1-1-0/api-ms-win-crt-math-l1-1-0.spec +++ b/dlls/api-ms-win-crt-math-l1-1-0/api-ms-win-crt-math-l1-1-0.spec @@ -62,7 +62,7 @@ @ stub _dsin @ cdecl _dtest(ptr) ucrtbase._dtest @ stub _dunscale -@ stub _except1 +@ cdecl _except1(long long double double long ptr) ucrtbase._except1 @ stub _fd_int @ cdecl _fdclass(float) ucrtbase._fdclass @ stub _fdexp diff --git a/dlls/msvcr120/msvcr120.spec b/dlls/msvcr120/msvcr120.spec index 62481c0..db2ae9c 100644 --- a/dlls/msvcr120/msvcr120.spec +++ b/dlls/msvcr120/msvcr120.spec @@ -1124,7 +1124,7 @@ @ extern _environ MSVCRT__environ @ cdecl _eof(long) MSVCRT__eof @ cdecl _errno() MSVCRT__errno -@ stub _except1 +@ cdecl _except1(long long double double long ptr) @ cdecl -arch=i386 _except_handler2(ptr ptr ptr ptr) @ cdecl -arch=i386 _except_handler3(ptr ptr ptr ptr) @ cdecl -arch=i386 _except_handler4_common(ptr ptr ptr ptr ptr ptr) diff --git a/dlls/msvcr120_app/msvcr120_app.spec b/dlls/msvcr120_app/msvcr120_app.spec index a74a140..849e135 100644 --- a/dlls/msvcr120_app/msvcr120_app.spec +++ b/dlls/msvcr120_app/msvcr120_app.spec @@ -1060,7 +1060,7 @@ @ extern _environ msvcr120._environ @ cdecl _eof(long) msvcr120._eof @ cdecl _errno() msvcr120._errno -@ stub _except1 +@ cdecl _except1(long long double double long ptr) msvcr120._except1 @ cdecl -arch=i386 _except_handler2(ptr ptr ptr ptr) msvcr120._except_handler2 @ cdecl -arch=i386 _except_handler3(ptr ptr ptr ptr) msvcr120._except_handler3 @ cdecl -arch=i386 _except_handler4_common(ptr ptr ptr ptr ptr ptr) msvcr120._except_handler4_common diff --git a/dlls/msvcrt/math.c b/dlls/msvcrt/math.c index 74685d4..857561b 100644 --- a/dlls/msvcrt/math.c +++ b/dlls/msvcrt/math.c @@ -2986,3 +2986,95 @@ float CDECL MSVCR120_nanf(const char *tagp) { return NAN; } + +/********************************************************************* + * _except1 (MSVCR120.@) + * TODO: + * - find meaning of ignored cw and operation bits + * - unk parameter + */ +double CDECL _except1(DWORD fpe, _FP_OPERATION_CODE op, double arg, double res, DWORD cw, void *unk) +{ + ULONG_PTR exception_arg; + DWORD exception = 0; + MSVCRT_fenv_t env; + DWORD fpword = 0; + WORD operation; + + TRACE("(%x %x %lf %lf %x %p)\n", fpe, op, arg, res, cw, unk); + +#ifdef _WIN64 + cw = ((cw >> 7) & 0x3f) | ((cw >> 3) & 0xc00); +#endif + operation = op << 5; + exception_arg = (ULONG_PTR)&operation; + + MSVCRT_fegetenv(&env); + + if (fpe & 0x1) { /* overflow */ + if ((fpe == 0x1 && (cw & 0x8)) || (fpe==0x11 && (cw & 0x28))) { + /* 32-bit version also sets SW_INEXACT here */ + env.status |= MSVCRT__SW_OVERFLOW; + if (fpe & 0x10) env.status |= MSVCRT__SW_INEXACT; + res = signbit(res) ? -INFINITY : INFINITY; + } else { + exception = EXCEPTION_FLT_OVERFLOW; + } + } else if (fpe & 0x2) { /* underflow */ + if ((fpe == 0x2 && (cw & 0x10)) || (fpe==0x12 && (cw & 0x30))) { + env.status |= MSVCRT__SW_UNDERFLOW; + if (fpe & 0x10) env.status |= MSVCRT__SW_INEXACT; + res = signbit(res) ? -0.0 : 0.0; + } else { + exception = EXCEPTION_FLT_UNDERFLOW; + } + } else if (fpe & 0x4) { /* zerodivide */ + if ((fpe == 0x4 && (cw & 0x4)) || (fpe==0x14 && (cw & 0x24))) { + env.status |= MSVCRT__SW_ZERODIVIDE; + if (fpe & 0x10) env.status |= MSVCRT__SW_INEXACT; + } else { + exception = EXCEPTION_FLT_DIVIDE_BY_ZERO; + } + } else if (fpe & 0x8) { /* invalid */ + if (fpe == 0x8 && (cw & 0x1)) { + env.status |= MSVCRT__SW_INVALID; + } else { + exception = EXCEPTION_FLT_INVALID_OPERATION; + } + } else if (fpe & 0x10) { /* inexact */ + if (fpe == 0x10 && (cw & 0x20)) { + env.status |= MSVCRT__SW_INEXACT; + } else { + exception = EXCEPTION_FLT_INEXACT_RESULT; + } + } + + if (exception) + env.status = 0; + MSVCRT_fesetenv(&env); + if (exception) + RaiseException(exception, 0, 1, &exception_arg); + + if (cw & 0x1) fpword |= MSVCRT__EM_INVALID; + if (cw & 0x2) fpword |= MSVCRT__EM_DENORMAL; + if (cw & 0x4) fpword |= MSVCRT__EM_ZERODIVIDE; + if (cw & 0x8) fpword |= MSVCRT__EM_OVERFLOW; + if (cw & 0x10) fpword |= MSVCRT__EM_UNDERFLOW; + if (cw & 0x20) fpword |= MSVCRT__EM_INEXACT; + switch (cw & 0xc00) + { + case 0xc00: fpword |= MSVCRT__RC_UP|MSVCRT__RC_DOWN; break; + case 0x800: fpword |= MSVCRT__RC_UP; break; + case 0x400: fpword |= MSVCRT__RC_DOWN; break; + } + switch (cw & 0x300) + { + case 0x0: fpword |= MSVCRT__PC_24; break; + case 0x200: fpword |= MSVCRT__PC_53; break; + case 0x300: fpword |= MSVCRT__PC_64; break; + } + if (cw & 0x1000) fpword |= MSVCRT__IC_AFFINE; + _control87(fpword, 0xffffffff); + + return res; +} diff --git a/dlls/ucrtbase/ucrtbase.spec b/dlls/ucrtbase/ucrtbase.spec index 7602698..5c97d8b 100644 --- a/dlls/ucrtbase/ucrtbase.spec +++ b/dlls/ucrtbase/ucrtbase.spec @@ -271,7 +271,7 @@ @ cdecl _endthreadex(long) @ cdecl _eof(long) MSVCRT__eof @ cdecl _errno() MSVCRT__errno -@ stub _except1 +@ cdecl _except1(long long double double long ptr) @ cdecl -arch=i386 _except_handler2(ptr ptr ptr ptr) @ cdecl -arch=i386 _except_handler3(ptr ptr ptr ptr) @ cdecl -arch=i386 _except_handler4_common(ptr ptr ptr ptr ptr ptr)

1 0