Skip to content

W
wine-winehq
Commit beefe76c authored by Nikolay Sivov's avatar Nikolay Sivov Committed by Alexandre Julliard
Browse files
Options

gdiplus: Add a helper to initialize RectF.

Signed-off-by: 's avatarNikolay Sivov <nsivov@codeweavers.com> Signed-off-by: 's avatarEsme Povirk <esme@codeweavers.com> Signed-off-by: 's avatarAlexandre Julliard <julliard@winehq.org>
parent 41651d2e
Hide whitespace changes
Inline Side-by-side
Showing with 40 additions and 104 deletions
dlls/gdiplus/brush.c
View file @ beefe76c
......@@ -471,11 +471,7 @@ GpStatus WINGDIPAPI GdipCreateLineBrushFromRectI(GDIPCONST GpRect* rect,
TRACE("(%p, %x, %x, %d, %d, %p)\n", rect, startcolor, endcolor, mode,
wrap, line);
rectF.X = (REAL) rect->X;
rectF.Y = (REAL) rect->Y;
rectF.Width = (REAL) rect->Width;
rectF.Height = (REAL) rect->Height;
set_rect(&rectF, rect->X, rect->Y, rect->Width, rect->Height);
return GdipCreateLineBrushFromRect(&rectF, startcolor, endcolor, mode, wrap, line);
}
......
dlls/gdiplus/gdiplus_private.h
View file @ beefe76c
......@@ -618,4 +618,12 @@ static inline void image_unlock(GpImage *image, BOOL unlock)
if (unlock) image->busy = 0;
}
static inline void set_rect(GpRectF *rect, REAL x, REAL y, REAL width, REAL height)
{
rect->X = x;
rect->Y = y;
rect->Width = width;
rect->Height = height;
}
#endif
dlls/gdiplus/graphics.c
View file @ beefe76c
......@@ -2923,10 +2923,7 @@ GpStatus WINGDIPAPI GdipDrawEllipse(GpGraphics *graphics, GpPen *pen, REAL x,
if (is_metafile_graphics(graphics))
{
rect.X = x;
rect.Y = y;
rect.Width = width;
rect.Height = height;
set_rect(&rect, x, y, width, height);
return METAFILE_DrawEllipse((GpMetafile *)graphics->image, pen, &rect);
}
......@@ -3411,11 +3408,7 @@ GpStatus WINGDIPAPI GdipDrawImagePointsRect(GpGraphics *graphics, GpImage *image
{
GpRectF rc;
rc.X = srcx;
rc.Y = srcy;
rc.Width = srcwidth;
rc.Height = srcheight;
set_rect(&rc, srcx, srcy, srcwidth, srcheight);
return GdipEnumerateMetafileSrcRectDestPoints(graphics, (GpMetafile*)image,
points, count, &rc, srcUnit, play_metafile_proc, image, imageAttributes);
}
......@@ -4092,10 +4085,7 @@ GpStatus WINGDIPAPI GdipDrawRectangle(GpGraphics *graphics, GpPen *pen, REAL x,
TRACE("(%p, %p, %.2f, %.2f, %.2f, %.2f)\n", graphics, pen, x, y, width, height);
rect.X = x;
rect.Y = y,
rect.Width = width;
rect.Height = height;
set_rect(&rect, x, y, width, height);
return GdipDrawRectangles(graphics, pen, &rect, 1);
}
......@@ -4151,12 +4141,8 @@ GpStatus WINGDIPAPI GdipDrawRectanglesI(GpGraphics *graphics, GpPen *pen,
if(!rectsF)
return OutOfMemory;
for(i = 0;i < count;i++){
rectsF[i].X = (REAL)rects[i].X;
rectsF[i].Y = (REAL)rects[i].Y;
rectsF[i].Width = (REAL)rects[i].Width;
rectsF[i].Height = (REAL)rects[i].Height;
}
for(i = 0;i < count;i++)
set_rect(&rectsF[i], rects[i].X, rects[i].Y, rects[i].Width, rects[i].Height);
ret = GdipDrawRectangles(graphics, pen, rectsF, count);
heap_free(rectsF);
......@@ -4246,6 +4232,7 @@ GpStatus WINGDIPAPI GdipFillEllipse(GpGraphics *graphics, GpBrush *brush, REAL x
{
GpStatus stat;
GpPath *path;
GpRectF rect;
TRACE("(%p, %p, %.2f, %.2f, %.2f, %.2f)\n", graphics, brush, x, y, width, height);
......@@ -4257,12 +4244,7 @@ GpStatus WINGDIPAPI GdipFillEllipse(GpGraphics *graphics, GpBrush *brush, REAL x
if (is_metafile_graphics(graphics))
{
GpRectF rect;
rect.X = x;
rect.Y = y;
rect.Width = width;
rect.Height = height;
set_rect(&rect, x, y, width, height);
return METAFILE_FillEllipse((GpMetafile *)graphics->image, brush, &rect);
}
......@@ -4404,10 +4386,7 @@ GpStatus WINGDIPAPI GdipFillPie(GpGraphics *graphics, GpBrush *brush, REAL x,
if (is_metafile_graphics(graphics))
{
rect.X = x;
rect.Y = y;
rect.Width = width;
rect.Height = height;
set_rect(&rect, x, y, width, height);
return METAFILE_FillPie((GpMetafile *)graphics->image, brush, &rect, startAngle, sweepAngle);
}
......@@ -4516,11 +4495,7 @@ GpStatus WINGDIPAPI GdipFillRectangle(GpGraphics *graphics, GpBrush *brush,
TRACE("(%p, %p, %.2f, %.2f, %.2f, %.2f)\n", graphics, brush, x, y, width, height);
rect.X = x;
rect.Y = y;
rect.Width = width;
rect.Height = height;
set_rect(&rect, x, y, width, height);
return GdipFillRectangles(graphics, brush, &rect, 1);
}
......@@ -4531,11 +4506,7 @@ GpStatus WINGDIPAPI GdipFillRectangleI(GpGraphics *graphics, GpBrush *brush,
TRACE("(%p, %p, %d, %d, %d, %d)\n", graphics, brush, x, y, width, height);
rect.X = (REAL)x;
rect.Y = (REAL)y;
rect.Width = (REAL)width;
rect.Height = (REAL)height;
set_rect(&rect, x, y, width, height);
return GdipFillRectangles(graphics, brush, &rect, 1);
}
......@@ -4584,12 +4555,8 @@ GpStatus WINGDIPAPI GdipFillRectanglesI(GpGraphics *graphics, GpBrush *brush, GD
if(!rectsF)
return OutOfMemory;
for(i = 0; i < count; i++){
rectsF[i].X = (REAL)rects[i].X;
rectsF[i].Y = (REAL)rects[i].Y;
rectsF[i].Width = (REAL)rects[i].Width;
rectsF[i].Height = (REAL)rects[i].Height;
}
for(i = 0; i < count; i++)
set_rect(&rectsF[i], rects[i].X, rects[i].Y, rects[i].Width, rects[i].Height);
ret = GdipFillRectangles(graphics,brush,rectsF,count);
heap_free(rectsF);
......@@ -5620,10 +5587,7 @@ GpStatus WINGDIPAPI GdipMeasureString(GpGraphics *graphics,
get_font_hfont(graphics, font, format, &gdifont, NULL, NULL);
oldfont = SelectObject(hdc, gdifont);
bounds->X = rect->X;
bounds->Y = rect->Y;
bounds->Width = 0.0;
bounds->Height = 0.0;
set_rect(bounds, rect->X, rect->Y, 0.0f, 0.0f);
args.bounds = bounds;
args.codepointsfitted = &glyphs;
......@@ -6502,10 +6466,7 @@ GpStatus WINGDIPAPI GdipSetClipRect(GpGraphics *graphics, REAL x, REAL y,
return status;
}
rect.X = x;
rect.Y = y;
rect.Width = width;
rect.Height = height;
set_rect(&rect, x, y, width, height);
status = GdipCreateRegionRect(&rect, &region);
if (status == Ok)
{
......@@ -7098,12 +7059,7 @@ GpStatus WINGDIPAPI GdipMeasureDriverString(GpGraphics *graphics, GDIPCONST UINT
length = lstrlenW(text);
if (length == 0)
{
boundingBox->X = 0.0;
boundingBox->Y = 0.0;
boundingBox->Width = 0.0;
boundingBox->Height = 0.0;
}
set_rect(boundingBox, 0.0f, 0.0f, 0.0f, 0.0f);
if (flags & unsupported_flags)
FIXME("Ignoring flags %x\n", flags & unsupported_flags);
......
dlls/gdiplus/graphicspath.c
View file @ beefe76c
......@@ -1080,19 +1080,16 @@ GpStatus WINGDIPAPI GdipAddPathString(GpPath* path, GDIPCONST WCHAR* string, INT
return status;
}
GpStatus WINGDIPAPI GdipAddPathStringI(GpPath* path, GDIPCONST WCHAR* string, INT length, GDIPCONST GpFontFamily* family, INT style, REAL emSize, GDIPCONST Rect* layoutRect, GDIPCONST GpStringFormat* format)
GpStatus WINGDIPAPI GdipAddPathStringI(GpPath* path, GDIPCONST WCHAR* string, INT length, GDIPCONST GpFontFamily* family,
INT style, REAL emSize, GDIPCONST Rect* layoutRect, GDIPCONST GpStringFormat* format)
{
if (layoutRect)
{
RectF layoutRectF = {
(REAL)layoutRect->X,
(REAL)layoutRect->Y,
(REAL)layoutRect->Width,
(REAL)layoutRect->Height
};
return GdipAddPathString(path, string, length, family, style, emSize, &layoutRectF, format);
}
return InvalidParameter;
RectF rect;
if (!layoutRect)
return InvalidParameter;
set_rect(&rect, layoutRect->X, layoutRect->Y, layoutRect->Width, layoutRect->Height);
return GdipAddPathString(path, string, length, family, style, emSize, &rect, format);
}
/*************************************************************************
......@@ -2547,12 +2544,8 @@ GpStatus WINGDIPAPI GdipAddPathRectanglesI(GpPath *path, GDIPCONST GpRect *rects
rectsF = heap_alloc_zero(sizeof(GpRectF)*count);
for(i = 0;i < count;i++){
rectsF[i].X = (REAL)rects[i].X;
rectsF[i].Y = (REAL)rects[i].Y;
rectsF[i].Width = (REAL)rects[i].Width;
rectsF[i].Height = (REAL)rects[i].Height;
}
for(i = 0;i < count;i++)
set_rect(&rectsF[i], rects[i].X, rects[i].Y, rects[i].Width, rects[i].Height);
retstat = GdipAddPathRectangles(path, rectsF, count);
heap_free(rectsF);
......
dlls/gdiplus/matrix.c
View file @ beefe76c
......@@ -111,10 +111,7 @@ GpStatus WINGDIPAPI GdipCreateMatrix3I(GDIPCONST GpRect *rect, GDIPCONST GpPoint
TRACE("(%p, %p, %p)\n", rect, pt, matrix);
rectF.X = (REAL)rect->X;
rectF.Y = (REAL)rect->Y;
rectF.Width = (REAL)rect->Width;
rectF.Height = (REAL)rect->Height;
set_rect(&rectF, rect->X, rect->Y, rect->Width, rect->Height);
for (i = 0; i < 3; i++) {
ptF[i].X = (REAL)pt[i].X;
......
dlls/gdiplus/metafile.c
View file @ beefe76c
......@@ -777,10 +777,7 @@ GpStatus WINGDIPAPI GdipRecordMetafileI(HDC hdc, EmfType type, GDIPCONST GpRect
if (frameRect)
{
frameRectF.X = frameRect->X;
frameRectF.Y = frameRect->Y;
frameRectF.Width = frameRect->Width;
frameRectF.Height = frameRect->Height;
set_rect(&frameRectF, frameRect->X, frameRect->Y, frameRect->Width, frameRect->Height);
pFrameRectF = &frameRectF;
}
else
......@@ -798,10 +795,7 @@ GpStatus WINGDIPAPI GdipRecordMetafileStreamI(IStream *stream, HDC hdc, EmfType
if (frameRect)
{
frameRectF.X = frameRect->X;
frameRectF.Y = frameRect->Y;
frameRectF.Width = frameRect->Width;
frameRectF.Height = frameRect->Height;
set_rect(&frameRectF, frameRect->X, frameRect->Y, frameRect->Width, frameRect->Height);
pFrameRectF = &frameRectF;
}
else
......
dlls/gdiplus/region.c
View file @ beefe76c
......@@ -332,11 +332,7 @@ GpStatus WINGDIPAPI GdipCombineRegionRectI(GpRegion *region,
if (!rect)
return InvalidParameter;
rectf.X = (REAL)rect->X;
rectf.Y = (REAL)rect->Y;
rectf.Height = (REAL)rect->Height;
rectf.Width = (REAL)rect->Width;
set_rect(&rectf, rect->X, rect->Y, rect->Width, rect->Height);
return GdipCombineRegionRect(region, &rectf, mode);
}
......@@ -492,11 +488,7 @@ GpStatus WINGDIPAPI GdipCreateRegionRectI(GDIPCONST GpRect *rect,
TRACE("%p, %p\n", rect, region);
rectf.X = (REAL)rect->X;
rectf.Y = (REAL)rect->Y;
rectf.Width = (REAL)rect->Width;
rectf.Height = (REAL)rect->Height;
set_rect(&rectf, rect->X, rect->Y, rect->Width, rect->Height);
return GdipCreateRegionRect(&rectf, region);
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment