[PATCH] gdi32: Remove no-op abs(3) call.
Huw Davies
huw at codeweavers.com
Wed Feb 17 02:50:43 CST 2016
On Tue, Feb 16, 2016 at 01:55:34PM -0700, Charles Davis wrote:
> Signed-off-by: Charles Davis <cdavis5x at gmail.com>
> ---
> dlls/gdi32/pen.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/gdi32/pen.c b/dlls/gdi32/pen.c
> index 2ae2f2b..6793934 100644
> --- a/dlls/gdi32/pen.c
> +++ b/dlls/gdi32/pen.c
> @@ -195,7 +195,7 @@ HPEN WINAPI ExtCreatePen( DWORD style, DWORD width,
> if (logbrush.lbStyle == BS_DIBPATTERN) logbrush.lbStyle = BS_DIBPATTERNPT;
>
> penPtr->logpen.elpPenStyle = style;
> - penPtr->logpen.elpWidth = abs(width);
> + penPtr->logpen.elpWidth = width;
> penPtr->logpen.elpBrushStyle = logbrush.lbStyle;
> penPtr->logpen.elpColor = logbrush.lbColor;
> penPtr->logpen.elpHatch = brush->lbHatch;
This breaks a test:
../../../tools/runtest -q -P wine -T ../../.. -M gdi32.dll -p gdi32_test.exe.so pen && touch pen.ok
pen.c:440: Test failed: expected 123, got ffffff85
width is actually treated as signed.
Huw.
More information about the wine-devel
mailing list