July 2017 - Wine-Commits - list.winehq.org

Alexandre Julliard : wordpad: New high resolution cursor generated from SVG.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: 756fdd029c6587a0be99042485d674ca0850de14 URL: http://source.winehq.org/git/wine.git/?a=commit;h=756fdd029c6587a0be9904248… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 14 13:12:27 2017 +0200 wordpad: New high resolution cursor generated from SVG. Based on a cursor set published by Ludvig Hummel under the X11 license. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/wordpad/Makefile.in | 3 +- programs/wordpad/zoom.cur | Bin 766 -> 33134 bytes programs/wordpad/zoom.svg | 246 +++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 248 insertions(+), 1 deletion(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=756fdd029c6587a0be990…

1 0

Alexandre Julliard : riched20: New high resolution cursor generated from SVG.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: 2d91663f7a86c9df69b1388c3b32023c1442ec48 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2d91663f7a86c9df69b1388c3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 14 13:12:04 2017 +0200 riched20: New high resolution cursor generated from SVG. Based on a cursor set published by Ludvig Hummel under the X11 license. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/riched20/Makefile.in | 3 + dlls/riched20/ocr_reverse.cur | Bin 326 -> 33134 bytes dlls/riched20/ocr_reverse.svg | 189 ++++++++++++++++++++++++++++++++++++++++++ dlls/riched20/version.rc | 1 + 4 files changed, 193 insertions(+) diff --git a/dlls/riched20/Makefile.in b/dlls/riched20/Makefile.in index be57b41..8f8aeee 100644 --- a/dlls/riched20/Makefile.in +++ b/dlls/riched20/Makefile.in @@ -28,3 +28,6 @@ RC_SRCS = version.rc IDL_SRCS = \ riched_tom.idl + +SVG_SRCS = \ + ocr_reverse.svg diff --git a/dlls/riched20/ocr_reverse.cur b/dlls/riched20/ocr_reverse.cur index 179774c..f6b5ca2 100644 Binary files a/dlls/riched20/ocr_reverse.cur and b/dlls/riched20/ocr_reverse.cur differ diff --git a/dlls/riched20/ocr_reverse.svg b/dlls/riched20/ocr_reverse.svg new file mode 100644 index 0000000..a6a356a --- /dev/null +++ b/dlls/riched20/ocr_reverse.svg @@ -0,0 +1,189 @@ +<?xml version="1.0" encoding="UTF-8" standalone="no"?> + + +<svg + xmlns:dc="http://purl.org/dc/elements/1.1/" + xmlns:cc="http://creativecommons.org/ns#" + xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#" + xmlns:svg="http://www.w3.org/2000/svg" + xmlns="http://www.w3.org/2000/svg" + xmlns:xlink="http://www.w3.org/1999/xlink" + xmlns:sodipodi="http://sodipodi.sourceforge.net/DTD/sodipodi-0.dtd" + xmlns:inkscape="http://www.inkscape.org/namespaces/inkscape" + id="svg2" + height="64" + width="288" + version="1.1" + sodipodi:docname="ocr_reverse.svg" + inkscape:version="0.92.1 r15371"> + <defs + id="defs5074" /> + <metadata + id="metadata5130"> + <rdf:RDF> + <cc:Work + rdf:about=""> + <dc:format>image/svg+xml</dc:format> + <dc:type + rdf:resource="http://purl.org/dc/dcmitype/StillImage" /> + <dc:title /> + </cc:Work> + </rdf:RDF> + </metadata> + <sodipodi:namedview + pagecolor="#ffffff" + bordercolor="#666666" + borderopacity="1" + objecttolerance="10" + gridtolerance="10" + guidetolerance="10" + inkscape:pageopacity="0" + inkscape:pageshadow="2" + inkscape:window-width="1777" + inkscape:window-height="1091" + id="namedview5128" + showgrid="false" + inkscape:zoom="1" + inkscape:cx="-103.00001" + inkscape:cy="32.000011" + inkscape:window-x="43" + inkscape:window-y="27" + inkscape:window-maximized="0" + inkscape:current-layer="svg2" /> + <rect + id="cursor:64-32" + height="64" + width="64" + y="0" + x="0" + style="fill:none" /> + <rect + id="cursor:48-32" + height="48" + width="48" + y="16" + x="64" + style="fill:none" /> + <rect + id="cursor:32-32" + height="32" + width="32" + y="32" + x="112" + style="fill:none" /> + <rect + id="hotspot" + height="1" + width="1" + y="0" + x="13" + style="fill:none" /> + <g + id="g4928" + transform="matrix(1.9642525,0,0,1.9642525,-194.7906,458.0998)"> + <path + sodipodi:type="inkscape:offset" + inkscape:radius="0" + inkscape:original="M 22.023438 743.13867 L 24.535156 762.16992 L 27.591797 756.31836 L 31.871094 763.69141 C 32.590378 764.93081 36.313733 762.69303 35.556641 761.47656 L 31.050781 754.23828 L 37.654297 754.2832 L 22.023438 743.13867 z " + xlink:href="#path7371" + style="display:inline;opacity:0.13600003;fill:#000000;stroke:#000000;stroke-width:1.40831447;stroke-linejoin:round;stroke-miterlimit:4;stroke-dasharray:none" + id="path6044" + inkscape:href="#path7371" + d="m 22.023438,743.13867 2.511718,19.03125 3.056641,-5.85156 4.279297,7.37305 c 0.719284,1.2394 4.442639,-0.99838 3.685547,-2.21485 l -4.50586,-7.23828 6.603516,0.0449 z" + transform="matrix(-0.91499598,0.11480473,0.1150337,0.91682096,48.848195,-913.69159)" /> + <path + style="display:inline;opacity:1;fill:#ffffff;stroke:#000000;stroke-width:1.39999998;stroke-linejoin:round;stroke-miterlimit:4;stroke-dasharray:none" + inkscape:transform-center-y="-3.2019766" + d="m 99.867659,-220.50653 6.036251,-0.79869 -3.28917,7.15307 c -0.55281,1.20222 3.11,2.8265 3.62558,1.60767 l 3.06789,-7.25104 3.47063,5.01422 0.10905,-17.73718 z" + id="path6046-3" + inkscape:connector-curvature="0" + sodipodi:nodetypes="ccsscccc" + inkscape:transform-center-x="0.59406761" /> + </g> + <g + transform="matrix(1.4731894,0,0,1.4731894,-82.09295,359.57486)" + id="g5495"> + <path + transform="matrix(-0.91499598,0.11480473,0.1150337,0.91682096,48.848195,-913.69159)" + d="m 22.023438,743.13867 2.511718,19.03125 3.056641,-5.85156 4.279297,7.37305 c 0.719284,1.2394 4.442639,-0.99838 3.685547,-2.21485 l -4.50586,-7.23828 6.603516,0.0449 z" + inkscape:href="#path7371" + id="path5491" + style="display:inline;opacity:0.13600003;fill:#000000;stroke:#000000;stroke-width:1.40831447;stroke-linejoin:round;stroke-miterlimit:4;stroke-dasharray:none" + xlink:href="#path7371" + inkscape:original="M 22.023438 743.13867 L 24.535156 762.16992 L 27.591797 756.31836 L 31.871094 763.69141 C 32.590378 764.93081 36.313733 762.69303 35.556641 761.47656 L 31.050781 754.23828 L 37.654297 754.2832 L 22.023438 743.13867 z " + inkscape:radius="0" + sodipodi:type="inkscape:offset" /> + <path + inkscape:transform-center-x="0.59406761" + sodipodi:nodetypes="ccsscccc" + inkscape:connector-curvature="0" + id="path5493" + d="m 99.867659,-220.50653 6.036251,-0.79869 -3.28917,7.15307 c -0.55281,1.20222 3.11,2.8265 3.62558,1.60767 l 3.06789,-7.25104 3.47063,5.01422 0.10905,-17.73718 z" + inkscape:transform-center-y="-3.2019766" + style="display:inline;opacity:1;fill:#ffffff;stroke:#000000;stroke-width:1.39999998;stroke-linejoin:round;stroke-miterlimit:4;stroke-dasharray:none" /> + </g> + <g + transform="matrix(0.98212626,0,0,0.98212626,14.604701,261.0499)" + id="g5501"> + <path + transform="matrix(-0.91499598,0.11480473,0.1150337,0.91682096,48.848195,-913.69159)" + d="m 22.023438,743.13867 2.511718,19.03125 3.056641,-5.85156 4.279297,7.37305 c 0.719284,1.2394 4.442639,-0.99838 3.685547,-2.21485 l -4.50586,-7.23828 6.603516,0.0449 z" + inkscape:href="#path7371" + id="path5497" + style="display:inline;opacity:0.13600003;fill:#000000;stroke:#000000;stroke-width:1.40831447;stroke-linejoin:round;stroke-miterlimit:4;stroke-dasharray:none" + xlink:href="#path7371" + inkscape:original="M 22.023438 743.13867 L 24.535156 762.16992 L 27.591797 756.31836 L 31.871094 763.69141 C 32.590378 764.93081 36.313733 762.69303 35.556641 761.47656 L 31.050781 754.23828 L 37.654297 754.2832 L 22.023438 743.13867 z " + inkscape:radius="0" + sodipodi:type="inkscape:offset" /> + <path + inkscape:transform-center-x="0.59406761" + sodipodi:nodetypes="ccsscccc" + inkscape:connector-curvature="0" + id="path5499" + d="m 99.867659,-220.50653 6.036251,-0.79869 -3.28917,7.15307 c -0.55281,1.20222 3.11,2.8265 3.62558,1.60767 l 3.06789,-7.25104 3.47063,5.01422 0.10905,-17.73718 z" + inkscape:transform-center-y="-3.2019766" + style="display:inline;opacity:1;fill:#ffffff;stroke:#000000;stroke-width:1.39999998;stroke-linejoin:round;stroke-miterlimit:4;stroke-dasharray:none" /> + </g> + <image + y="0" + x="144" + id="cursor:64-1" + xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEAAAABACAQAAAAAYLlVAAAAAmJLR0QA/4ePzL8AAAAJcEhZcwAA +DsQAAA7EAZUrDhsAAAAHdElNRQfhBwoQMzriEEEBAAAA8UlEQVRo3u2XQRLEIAgEhdr/f9m9bWJW +I1jMcGFOuaS6gZTB1s7SW1A0F38mEIg/EQjF+wWC8V6BcLxPAID3CHSMhebirQIdNwPNxVsEoPi9 +ABi/E4Dj3wUI+DcBCn4tQMKvBGj4uQARPxOg4v8FyPinAB0/CiTg7wIp+EsgCd/aZ+QKegObdmBB +EpIAv+uPEcjqLOxDN7IWkt8whCjQ55+gZHVAoApqrR+loMb65Xog/A2ZB4Ma64cpqH3+GAV11A9R +UFf9AAV11h+uoIfvCUxA9vWHRmZLKXMpkNW9gLWTyMvVjLMSVSqVSqVSqVSy8wXPKj881pbRsAAA +AABJRU5ErkJggg== +" + preserveAspectRatio="none" + height="64" + width="64" /> + <image + y="16" + x="208" + id="cursor:48-1" + xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAADAAAAAwCAQAAAD9CzEMAAAAAmJLR0QA/4ePzL8AAAAJcEhZcwAA +DsQAAA7EAZUrDhsAAAAHdElNRQfhBwoQNQx78HMeAAAAsElEQVRYw+2UOw7EMAgFedbe/8qkiKL4 +m0TroeM17mbANph9iZvbnymf8Bspsfh3wSb+TeD7jhKLfxIg+LUAwq8EGH4uAPEzAYofBTC+F+D4 +VhCArwUh+FsQhL8EYXiz300Wt0KbDmIKrzpQ/1EvowJ2UUdUxKrwBi5+0PqI7aCt/zxETvJQvNA3 +8Nm1CFwVi8sX1IGveQK/6QIpQCBsuOaVOTzCQwd06ZlMJpPJVDkAHLAuLKz26WIAAAAASUVORK5C +YII= +" + preserveAspectRatio="none" + height="48" + width="48" /> + <image + y="32" + x="256" + id="cursor:32-1" + xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAACAAAAAgCAQAAADZc7J/AAAAAmJLR0QA/4ePzL8AAAAJcEhZcwAA +DsQAAA7EAZUrDhsAAAAHdElNRQfhBwoQNjhxadRoAAAAeElEQVRIx+2Tyw6AIAwEdxv//5frgRiq +UkOXo8x9hlcBMhyOCSzVJ7FM94VAQR8FSvo7UNSfgbJ+Dwh6DEh6D4j6FZD1FljQgaO5rE5wCLBf +Y9MpDlKwqM2BB5nKM4YNcD5ho/UrW3j/RtYSFj2OU58QyRk2m59xAqonIRyk5cnGAAAAAElFTkSu +QmCC +" + preserveAspectRatio="none" + height="32" + width="32" /> +</svg> diff --git a/dlls/riched20/version.rc b/dlls/riched20/version.rc index ff2a2ba..4674371 100644 --- a/dlls/riched20/version.rc +++ b/dlls/riched20/version.rc @@ -27,4 +27,5 @@ #include "res.h" +/* @makedep: ocr_reverse.cur */ OCR_REVERSE CURSOR ocr_reverse.cur

1 0

Alexandre Julliard : tools: Add support for generating cursor files from SVG.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: 8e45a6ecbfbb54c4b0cedd1dc34694d26dd5f488 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8e45a6ecbfbb54c4b0cedd1dc… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 14 11:58:54 2017 +0200 tools: Add support for generating cursor files from SVG. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/buildimage | 70 ++++++++++++++++++++++++++------------------------------ tools/makedep.c | 15 ++++++++---- 2 files changed, 43 insertions(+), 42 deletions(-) diff --git a/tools/buildimage b/tools/buildimage index 0a7e466..fd6661e 100755 --- a/tools/buildimage +++ b/tools/buildimage @@ -34,23 +34,22 @@ die "Cannot open output file" unless defined($outFileName); $outFileName =~ m/(.*)\.(.*)/; my $outName = $1; my $ext = lc($2); -die "Only BMP and ICO outputs are supported" unless $ext eq "bmp" or $ext eq "ico"; +die "Only BMP, ICO and CUR outputs are supported" unless $ext eq "bmp" or $ext eq "ico" or $ext eq "cur"; my $renderedSVGFileName = "$svgFileName.png"; my @pngFiles; -my @pngFilesRaw; # Get the programs from the environment variables my $convert = $ENV{"CONVERT"} || "convert"; my $rsvg = $ENV{"RSVG"} || "rsvg"; -my $icotool = $ENV{"ICOTOOL"} || "icotool"; +my @icotool_args = ($ENV{"ICOTOOL"} || "icotool", "--create", + $ext eq "cur" ? "--cursor" : "--icon", "-o", $outFileName); # Be ready to abort sub cleanup() { unlink $renderedSVGFileName; unlink $_ foreach(@pngFiles); - unlink $_ foreach(@pngFilesRaw); } $SIG{"INT"} = "cleanup"; @@ -78,10 +77,20 @@ sub svg_element_start my $width = 0; my $height = 0; + if ($id eq "hotspot") + { + push @icotool_args, "--hotspot-x=$attr{x}", "--hotspot-y=$attr{y}"; + return; + } + if($ext eq "ico") { return unless $id =~ /icon:(\d*)-(\d*)/; $size = $1; $depth = $2; + } elsif($ext eq "cur") { + return unless $id =~ /cursor:(\d*)-(\d*)/; + $size = $1; + $depth = $2; } elsif($ext eq "bmp") { return unless $id =~ /bitmap:(\d*)-(\d*)/; $size = $1; @@ -90,14 +99,24 @@ sub svg_element_start return unless defined($size) and defined($depth); - warn "Unexpected icon depth" unless - $depth == 4 or $depth == 8 or $depth == 24 or $depth == 32; + warn "Unexpected depth" unless + $depth == 1 or $depth == 4 or $depth == 8 or $depth == 24 or $depth == 32; my $pngFileName = "$outName-$size-$depth.png"; if($element eq "svg") { + if ($ext eq "bmp") { + if ($depth == 24) { + shell $convert, $renderedSVGFileName, "+matte", $outFileName; + } else { + shell $convert, $renderedSVGFileName, $outFileName; + } + cleanup(); + exit(0); + } + # The whole file is tagged - copy($renderedSVGFileName, $pngFileName) or die "File could not be copied"; + $pngFileName = $renderedSVGFileName; } elsif($element eq "rect") { @@ -109,10 +128,9 @@ sub svg_element_start if(defined($x) and defined($y)) { if($x =~ /\d*/ and $y =~ /\d*/) { - shell $convert, $renderedSVGFileName, "-crop", "${width}x${height}+$x+$y", $pngFileName; + shell $convert, $renderedSVGFileName, "-crop", "${width}x${height}+$x+$y", "-depth", $depth, $pngFileName; } } - } elsif($element eq "image" ) { # Extract Base64 encoded PNG data to files @@ -132,12 +150,13 @@ sub svg_element_start if ($width >= 128 && $height >= 128) { - push(@pngFilesRaw, $pngFileName); + push @icotool_args, "--raw=$pngFileName"; } else { - push(@pngFiles, $pngFileName); + push @icotool_args, $pngFileName; } + push @pngFiles, $pngFileName; } # Render the SVG image @@ -153,34 +172,9 @@ my $parser = new XML::Parser( Handlers => {Start => \&svg_element_start}); $parser->parsefile("$svgFileName"); -# If no render directives were found, take the full image as-is -unless (@pngFiles || @pngFilesRaw) { - my $pngFileName = "bmp$renderedSVGFileName"; - copy($renderedSVGFileName, $pngFileName) or die "File could not be copied"; - push(@pngFiles, $pngFileName); -} - -# Combine the renderings into the output file -if($ext eq "ico") { - - # Place images into the ICO - shell $icotool, "-c", "-o", $outFileName, @pngFiles, map { "--raw=$_"; } @pngFilesRaw; - -} elsif($ext eq "bmp") { - - # Only the first image becomes the final BMP - my $pngFile = $pngFiles[0]; - $pngFile =~ /.*-\d*-(\d*)\.png/; - my $depth = $1; +die "no render directive found in $svgFileName" unless @pngFiles; - # Convert it into a bmp - if($depth == 24) { - shell $convert, "png:$pngFile", "+matte", $outFileName; - } else { - shell $convert, "png:$pngFile", $outFileName; - } - -} +shell @icotool_args; # Delete the intermediate images cleanup(); diff --git a/tools/makedep.c b/tools/makedep.c index fe670af..296356b 100644 --- a/tools/makedep.c +++ b/tools/makedep.c @@ -2600,10 +2600,17 @@ static struct strarray output_sources( const struct makefile *make ) { if (convert && rsvg && icotool && !make->src_dir) { - output( "%s.ico %s.bmp: %s\n", - src_dir_path( make, obj ), src_dir_path( make, obj ), source->filename ); - output( "\tCONVERT=\"%s\" ICOTOOL=\"%s\" RSVG=\"%s\" %s %s $@\n", convert, icotool, rsvg, - top_src_dir_path( make, "tools/buildimage" ), source->filename ); + static const char * const images[] = { "bmp", "cur", "ico", NULL }; + + for (i = 0; images[i]; i++) + if (find_include_file( make, strmake( "%s.%s", obj, images[i] ))) break; + + if (images[i]) + { + output( "%s.%s: %s\n", src_dir_path( make, obj ), images[i], source->filename ); + output( "\tCONVERT=\"%s\" ICOTOOL=\"%s\" RSVG=\"%s\" %s %s $@\n", convert, icotool, rsvg, + top_src_dir_path( make, "tools/buildimage" ), source->filename ); + } } } else if (!strcmp( ext, "po" )) /* po file */

1 0

Vincent Povirk : gdiplus: Fix leak in widen_dashed_figure.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: 9f22041b31e6656a38aa92dcc1757fd7f5b07847 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9f22041b31e6656a38aa92dcc… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Jul 13 13:56:14 2017 -0500 gdiplus: Fix leak in widen_dashed_figure. Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/graphicspath.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c index 458019f..e190532 100644 --- a/dlls/gdiplus/graphicspath.c +++ b/dlls/gdiplus/graphicspath.c @@ -2187,6 +2187,7 @@ static void widen_dashed_figure(GpPath *path, GpPen *pen, int start, int end, closed ? LineCapFlat : pen->endcap, pen->customend, last_point); } + heap_free(dash_pattern_scaled); heap_free(tmp_points); }

1 0

Henri Verbeet : wined3d: Fix the WINED3DFMT_D24_UNORM_S8_UINT entry in wined3d_format_convert_from_float ().
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: 7578e09801b7a2a651be258322bd4574bcd8e7c5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7578e09801b7a2a651be25832… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Jul 14 07:54:11 2017 +0200 wined3d: Fix the WINED3DFMT_D24_UNORM_S8_UINT entry in wined3d_format_convert_from_float(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index 79c322f..1c64606 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -5057,7 +5057,7 @@ DWORD wined3d_format_convert_from_float(const struct wined3d_format *format, con } double_conv[] = { - {WINED3DFMT_D24_UNORM_S8_UINT, { 16777215.0, 0.0, 0.0, 0.0}, {0, 0, 0, 0}}, + {WINED3DFMT_D24_UNORM_S8_UINT, { 16777215.0, 1.0, 0.0, 0.0}, {8, 0, 0, 0}}, {WINED3DFMT_X8D24_UNORM, { 16777215.0, 0.0, 0.0, 0.0}, {0, 0, 0, 0}}, {WINED3DFMT_D32_UNORM, {4294967295.0, 0.0, 0.0, 0.0}, {0, 0, 0, 0}}, };

1 0

Jacek Caban : winhttp: Use notification flags to explicitly mark notifications that should signal an event.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: 8c9df5b774beb58f1bb4b44fc43ddb1cb405a000 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8c9df5b774beb58f1bb4b44fc… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Jul 14 15:43:45 2017 +0200 winhttp: Use notification flags to explicitly mark notifications that should signal an event. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winhttp/tests/notification.c | 53 ++++++++++++++++++++------------------- 1 file changed, 27 insertions(+), 26 deletions(-) diff --git a/dlls/winhttp/tests/notification.c b/dlls/winhttp/tests/notification.c index 3af35cf..dc36b88 100644 --- a/dlls/winhttp/tests/notification.c +++ b/dlls/winhttp/tests/notification.c @@ -52,6 +52,7 @@ struct notification #define NF_ALLOW 0x0001 /* notification may or may not happen */ #define NF_WINE_ALLOW 0x0002 /* wine sends notification when it should not */ +#define NF_SIGNAL 0x0004 /* signal wait handle when notified */ struct info { @@ -88,9 +89,8 @@ static void CALLBACK check_notification( HINTERNET handle, DWORD_PTR context, DW function_ok = (info->test[info->index].function == info->function); ok(status_ok, "%u: expected status 0x%08x got 0x%08x\n", info->line, info->test[info->index].status, status); ok(function_ok, "%u: expected function %u got %u\n", info->line, info->test[info->index].function, info->function); - if (status_ok && function_ok) info->index++; - if (status & (WINHTTP_CALLBACK_FLAG_ALL_COMPLETIONS | WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING)) + if (status_ok && function_ok && info->test[info->index++].flags & NF_SIGNAL) { SetEvent( info->wait ); } @@ -110,7 +110,7 @@ static const struct notification cache_test[] = { winhttp_receive_response, WINHTTP_CALLBACK_STATUS_RESPONSE_RECEIVED }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CLOSING_CONNECTION, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CONNECTION_CLOSED, NF_WINE_ALLOW }, - { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, + { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING, NF_SIGNAL }, { winhttp_open_request, WINHTTP_CALLBACK_STATUS_HANDLE_CREATED }, { winhttp_send_request, WINHTTP_CALLBACK_STATUS_CONNECTING_TO_SERVER, NF_WINE_ALLOW }, { winhttp_send_request, WINHTTP_CALLBACK_STATUS_CONNECTED_TO_SERVER, NF_WINE_ALLOW }, @@ -121,8 +121,8 @@ static const struct notification cache_test[] = { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CLOSING_CONNECTION, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CONNECTION_CLOSED, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, - { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, - { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, + { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING, NF_SIGNAL }, + { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING, NF_SIGNAL }, { winhttp_connect, WINHTTP_CALLBACK_STATUS_HANDLE_CREATED }, { winhttp_open_request, WINHTTP_CALLBACK_STATUS_HANDLE_CREATED }, { winhttp_send_request, WINHTTP_CALLBACK_STATUS_RESOLVING_NAME, NF_WINE_ALLOW }, @@ -135,7 +135,7 @@ static const struct notification cache_test[] = { winhttp_receive_response, WINHTTP_CALLBACK_STATUS_RESPONSE_RECEIVED }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CLOSING_CONNECTION, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CONNECTION_CLOSED, NF_WINE_ALLOW }, - { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, + { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING, NF_SIGNAL }, { winhttp_open_request, WINHTTP_CALLBACK_STATUS_HANDLE_CREATED }, { winhttp_send_request, WINHTTP_CALLBACK_STATUS_CONNECTING_TO_SERVER, NF_WINE_ALLOW }, { winhttp_send_request, WINHTTP_CALLBACK_STATUS_CONNECTED_TO_SERVER, NF_WINE_ALLOW }, @@ -146,12 +146,13 @@ static const struct notification cache_test[] = { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CLOSING_CONNECTION, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CONNECTION_CLOSED, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, - { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, - { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING } + { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING, NF_SIGNAL }, + { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING, NF_SIGNAL } }; static void setup_test( struct info *info, enum api function, unsigned int line ) { + if (info->wait) ResetEvent( info->wait ); info->function = function; info->line = line; while (info->index < info->count && info->test[info->index].function != function @@ -251,11 +252,9 @@ static void test_connection_cache( void ) ResetEvent( info.wait ); setup_test( &info, winhttp_close_handle, __LINE__ ); WinHttpCloseHandle( req ); - WaitForSingleObject( info.wait, INFINITE ); - - setup_test( &info, winhttp_close_handle, __LINE__ ); WinHttpCloseHandle( req ); WinHttpCloseHandle( con ); + WaitForSingleObject( info.wait, INFINITE ); if (unload) { @@ -263,12 +262,13 @@ static void test_connection_cache( void ) ok(status == WAIT_TIMEOUT, "got %08x\n", status); } + setup_test( &info, winhttp_close_handle, __LINE__ ); WinHttpCloseHandle( ses ); + WaitForSingleObject( info.wait, INFINITE ); - Sleep(2000); /* make sure connection is evicted from cache */ if (unload) { - status = WaitForSingleObject( event, 0 ); + status = WaitForSingleObject( event, 100 ); ok(status == WAIT_OBJECT_0, "got %08x\n", status); } @@ -351,9 +351,8 @@ static void test_connection_cache( void ) setup_test( &info, winhttp_close_handle, __LINE__ ); done: WinHttpCloseHandle( req ); - WaitForSingleObject( info.wait, INFINITE ); WinHttpCloseHandle( con ); - CloseHandle( info.wait ); + WaitForSingleObject( info.wait, INFINITE ); if (unload) { @@ -361,12 +360,14 @@ done: ok(status == WAIT_TIMEOUT, "got %08x\n", status); } + setup_test( &info, winhttp_close_handle, __LINE__ ); WinHttpCloseHandle( ses ); + WaitForSingleObject( info.wait, INFINITE ); + CloseHandle( info.wait ); - Sleep(2000); /* make sure connection is evicted from cache */ if (unload) { - status = WaitForSingleObject( event, 0 ); + status = WaitForSingleObject( event, 100 ); ok(status == WAIT_OBJECT_0, "got %08x\n", status); } @@ -398,7 +399,7 @@ static const struct notification redirect_test[] = { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CONNECTION_CLOSED, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, - { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING } + { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING, NF_SIGNAL } }; static void test_redirect( void ) @@ -411,7 +412,7 @@ static void test_redirect( void ) info.test = redirect_test; info.count = sizeof(redirect_test) / sizeof(redirect_test[0]); info.index = 0; - info.wait = NULL; + info.wait = CreateEventW( NULL, FALSE, FALSE, NULL ); ses = WinHttpOpen( user_agent, 0, NULL, NULL, 0 ); ok(ses != NULL, "failed to open session %u\n", GetLastError()); @@ -453,6 +454,8 @@ done: WinHttpCloseHandle( req ); WinHttpCloseHandle( con ); WinHttpCloseHandle( ses ); + WaitForSingleObject( info.wait, INFINITE ); + CloseHandle( info.wait ); } static const struct notification async_test[] = @@ -465,19 +468,19 @@ static const struct notification async_test[] = { winhttp_send_request, WINHTTP_CALLBACK_STATUS_CONNECTED_TO_SERVER, NF_WINE_ALLOW }, { winhttp_send_request, WINHTTP_CALLBACK_STATUS_SENDING_REQUEST }, { winhttp_send_request, WINHTTP_CALLBACK_STATUS_REQUEST_SENT }, - { winhttp_send_request, WINHTTP_CALLBACK_STATUS_SENDREQUEST_COMPLETE }, + { winhttp_send_request, WINHTTP_CALLBACK_STATUS_SENDREQUEST_COMPLETE, NF_SIGNAL }, { winhttp_receive_response, WINHTTP_CALLBACK_STATUS_RECEIVING_RESPONSE }, { winhttp_receive_response, WINHTTP_CALLBACK_STATUS_RESPONSE_RECEIVED }, - { winhttp_receive_response, WINHTTP_CALLBACK_STATUS_HEADERS_AVAILABLE }, - { winhttp_query_data, WINHTTP_CALLBACK_STATUS_DATA_AVAILABLE }, + { winhttp_receive_response, WINHTTP_CALLBACK_STATUS_HEADERS_AVAILABLE, NF_SIGNAL }, + { winhttp_query_data, WINHTTP_CALLBACK_STATUS_DATA_AVAILABLE, NF_SIGNAL }, { winhttp_read_data, WINHTTP_CALLBACK_STATUS_RECEIVING_RESPONSE, NF_ALLOW }, { winhttp_read_data, WINHTTP_CALLBACK_STATUS_RESPONSE_RECEIVED, NF_ALLOW }, - { winhttp_read_data, WINHTTP_CALLBACK_STATUS_READ_COMPLETE }, + { winhttp_read_data, WINHTTP_CALLBACK_STATUS_READ_COMPLETE, NF_SIGNAL }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CLOSING_CONNECTION, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_CONNECTION_CLOSED, NF_WINE_ALLOW }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING }, - { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING } + { winhttp_close_handle, WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING, NF_SIGNAL } }; static void test_async( void ) @@ -581,7 +584,6 @@ static void test_async( void ) setup_test( &info, winhttp_close_handle, __LINE__ ); WinHttpCloseHandle( req ); - WaitForSingleObject( info.wait, INFINITE ); WinHttpCloseHandle( con ); if (unload) @@ -590,7 +592,6 @@ static void test_async( void ) ok(status == WAIT_TIMEOUT, "got %08x\n", status); } WinHttpCloseHandle( ses ); - WaitForSingleObject( info.wait, INFINITE ); if (unload)

1 0

Alexandre Julliard : user32: New high resolution cursors generated from SVG.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: 32ccc0b9e0f1536655b7fca6e68823e1bf05374a URL: http://source.winehq.org/git/wine.git/?a=commit;h=32ccc0b9e0f1536655b7fca6e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 14 12:46:45 2017 +0200 user32: New high resolution cursors generated from SVG. Based on a cursor set published by Ludvig Hummel under the X11 license. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/Makefile.in | 16 ++ dlls/user32/resources/ocr_appstarting.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_appstarting.svg | 411 ++++++++++++++++++++++++++++++ dlls/user32/resources/ocr_cross.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_cross.svg | 184 +++++++++++++ dlls/user32/resources/ocr_hand.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_hand.svg | 190 ++++++++++++++ dlls/user32/resources/ocr_help.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_help.svg | 293 +++++++++++++++++++++ dlls/user32/resources/ocr_ibeam.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_ibeam.svg | 172 +++++++++++++ dlls/user32/resources/ocr_no.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_no.svg | 182 +++++++++++++ dlls/user32/resources/ocr_normal.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_normal.svg | 199 +++++++++++++++ dlls/user32/resources/ocr_pen.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_pen.svg | 260 +++++++++++++++++++ dlls/user32/resources/ocr_size.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_size.svg | 165 ++++++++++++ dlls/user32/resources/ocr_sizeall.cur | Bin 326 -> 35374 bytes dlls/user32/resources/ocr_sizeall.svg | 204 +++++++++++++++ dlls/user32/resources/ocr_sizenesw.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_sizenesw.svg | 189 ++++++++++++++ dlls/user32/resources/ocr_sizens.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_sizens.svg | 182 +++++++++++++ dlls/user32/resources/ocr_sizenwse.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_sizenwse.svg | 193 ++++++++++++++ dlls/user32/resources/ocr_sizewe.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_sizewe.svg | 179 +++++++++++++ dlls/user32/resources/ocr_up.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_up.svg | 181 +++++++++++++ dlls/user32/resources/ocr_wait.cur | Bin 326 -> 33134 bytes dlls/user32/resources/ocr_wait.svg | 327 ++++++++++++++++++++++++ 33 files changed, 3527 insertions(+) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=32ccc0b9e0f1536655b7f…

1 0

Jacek Caban : winhttp/tests: Make sure that all notifications were called before the end of a test.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: 4620bcf8062545306adfba8f564c4c9a172ac2b8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4620bcf8062545306adfba8f5… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Jul 14 15:43:52 2017 +0200 winhttp/tests: Make sure that all notifications were called before the end of a test. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winhttp/tests/notification.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/dlls/winhttp/tests/notification.c b/dlls/winhttp/tests/notification.c index dc36b88..e9de152 100644 --- a/dlls/winhttp/tests/notification.c +++ b/dlls/winhttp/tests/notification.c @@ -163,6 +163,12 @@ static void setup_test( struct info *info, enum api function, unsigned int line info->test[info->index].function, function); } +static void end_test( struct info *info, unsigned int line ) +{ + ok_(__FILE__,line)(info->index == info->count, "some notifications were missing: %x\n", + info->test[info->index].status); +} + static void test_connection_cache( void ) { HANDLE ses, con, req, event; @@ -364,6 +370,7 @@ done: WinHttpCloseHandle( ses ); WaitForSingleObject( info.wait, INFINITE ); CloseHandle( info.wait ); + end_test( &info, __LINE__ ); if (unload) { @@ -456,6 +463,7 @@ done: WinHttpCloseHandle( ses ); WaitForSingleObject( info.wait, INFINITE ); CloseHandle( info.wait ); + end_test( &info, __LINE__ ); } static const struct notification async_test[] = @@ -593,6 +601,7 @@ static void test_async( void ) } WinHttpCloseHandle( ses ); WaitForSingleObject( info.wait, INFINITE ); + end_test( &info, __LINE__ ); if (unload) {

1 0

Alexandre Julliard : comctl32: New high resolution cursors generated from SVG.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: d6fdbae1bc5e425edb949e0d7d096244824f9ad2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d6fdbae1bc5e425edb949e0d7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Jul 14 13:11:23 2017 +0200 comctl32: New high resolution cursors generated from SVG. Based on a cursor set published by Ludvig Hummel under the X11 license. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/comctl32/Makefile.in | 6 +- dlls/comctl32/idc_copy.cur | Bin 326 -> 33134 bytes dlls/comctl32/idc_copy.svg | 212 +++++++++++++++++++++++++++++++ dlls/comctl32/idc_divider.cur | Bin 326 -> 33134 bytes dlls/comctl32/idc_divider.svg | 186 +++++++++++++++++++++++++++ dlls/comctl32/idc_divideropen.cur | Bin 326 -> 33134 bytes dlls/comctl32/idc_divideropen.svg | 210 +++++++++++++++++++++++++++++++ dlls/comctl32/idc_movebutton.cur | Bin 326 -> 33134 bytes dlls/comctl32/idc_movebutton.svg | 259 ++++++++++++++++++++++++++++++++++++++ 9 files changed, 872 insertions(+), 1 deletion(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d6fdbae1bc5e425edb949…

1 0

Nikolay Sivov : regedit: Simplify double-click listview handler.
by Alexandre Julliard July 14, 2017

July 14, 2017

Module: wine Branch: master Commit: cbd6e05e681ffc08cdaf89d00d041aae59c40d5a URL: http://source.winehq.org/git/wine.git/?a=commit;h=cbd6e05e681ffc08cdaf89d00… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Jul 14 00:27:11 2017 +0300 regedit: Simplify double-click listview handler. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/regedit/childwnd.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/programs/regedit/childwnd.c b/programs/regedit/childwnd.c index 1797b52..f2359c4 100644 --- a/programs/regedit/childwnd.c +++ b/programs/regedit/childwnd.c @@ -347,12 +347,8 @@ static int listview_notify(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam case NM_DBLCLK: { NMITEMACTIVATE *nmitem = (NMITEMACTIVATE *)lParam; - LVHITTESTINFO info; - info.pt.x = nmitem->ptAction.x; - info.pt.y = nmitem->ptAction.y; - - if (SendMessageW(g_pChildWnd->hListWnd, LVM_HITTEST, 0, (LPARAM)&info) != -1) + if (nmitem->iItem != -1) { LVITEMW item; @@ -362,7 +358,7 @@ static int listview_notify(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam item.state = LVIS_FOCUSED | LVIS_SELECTED; item.stateMask = LVIS_FOCUSED | LVIS_SELECTED; - SendMessageW(g_pChildWnd->hListWnd, LVM_SETITEMSTATE, info.iItem, (LPARAM)&item); + SendMessageW(g_pChildWnd->hListWnd, LVM_SETITEMSTATE, nmitem->iItem, (LPARAM)&item); SendMessageW(hFrameWnd, WM_COMMAND, ID_EDIT_MODIFY, 0); }

1 0