[Tools] winetest: Add support for reports containing pids.

Alexandre Julliard julliard at winehq.org
Wed Feb 8 13:47:04 CST 2017


Francois Gouget <fgouget at codeweavers.com> writes:

> diff --git a/winetest/dissect b/winetest/dissect
> index d9404fe2..8ef31b01 100755
> --- a/winetest/dissect
> +++ b/winetest/dissect
> @@ -306,7 +306,7 @@ while (<IN>) {
>              $dll = undef;
>              mydie "too many tests skipped by user request\n" if ++$user_skips > $maxuserskips;
>          }
> -    } elsif (/^(.*$unit.*: (\d+) tests executed \((\d+) marked as todo, (\d+) failures?\), (\d+) skipped\.)\r?$/) {
> +    } elsif (/^($unit:(?:0x[0-9a-f]+)? (\d+) tests executed \((\d+) marked as todo, (\d+) failures?\), (\d+) skipped\.)\r?$/) {

This may be bikeshedding, but I feel that it would be nicer to use the
%04x format we use for pid in other places.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list