June 2007 - Wine-Commits - list.winehq.org

Alex Villacís Lasso : quartz: Fix video rendering sizing/positioning bugs.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: fb8df8ea8289487ada51fb2b126d9ad3d1081ff1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fb8df8ea8289487ada51fb2b1… Author: Alex Villacís Lasso <a_villacis(a)palosanto.com> Date: Tue Jun 19 11:47:32 2007 -0500 quartz: Fix video rendering sizing/positioning bugs. --- dlls/quartz/videorenderer.c | 34 ++++++++++++++++++++++------------ 1 files changed, 22 insertions(+), 12 deletions(-) diff --git a/dlls/quartz/videorenderer.c b/dlls/quartz/videorenderer.c index b42338c..9e76fd1 100644 --- a/dlls/quartz/videorenderer.c +++ b/dlls/quartz/videorenderer.c @@ -126,6 +126,20 @@ static LRESULT CALLBACK VideoWndProcA(HWND hwnd, UINT uMsg, WPARAM wParam, LPARA /* TRACE("WM_SIZING %d %d %d %d\n", lprect->left, lprect->top, lprect->right, lprect->bottom); */ SetWindowPos(hwnd, NULL, lprect->left, lprect->top, lprect->right - lprect->left, lprect->bottom - lprect->top, SWP_NOZORDER); GetClientRect(hwnd, &pVideoRenderer->DestRect); + TRACE("WM_SIZING: DestRect=(%d,%d),(%d,%d)\n", + pVideoRenderer->DestRect.left, + pVideoRenderer->DestRect.top, + pVideoRenderer->DestRect.right - pVideoRenderer->DestRect.left, + pVideoRenderer->DestRect.bottom - pVideoRenderer->DestRect.top); + return TRUE; + case WM_SIZE: + TRACE("WM_SIZE %d %d\n", LOWORD(lParam), HIWORD(lParam)); + GetClientRect(hwnd, &pVideoRenderer->DestRect); + TRACE("WM_SIZING: DestRect=(%d,%d),(%d,%d)\n", + pVideoRenderer->DestRect.left, + pVideoRenderer->DestRect.top, + pVideoRenderer->DestRect.right - pVideoRenderer->DestRect.left, + pVideoRenderer->DestRect.bottom - pVideoRenderer->DestRect.top); return TRUE; default: return DefWindowProcA(hwnd, uMsg, wParam, lParam); @@ -305,18 +319,14 @@ static DWORD VideoRenderer_SendSampleData(VideoRendererImpl* This, LPBYTE data, if (!This->init) { - /* Compute the size of the whole window so the client area size matches video one */ - RECT wrect, crect; - int h, v; - GetWindowRect(This->hWnd, &wrect); - GetClientRect(This->hWnd, &crect); - h = (wrect.right - wrect.left) - (crect.right - crect.left); - v = (wrect.bottom - wrect.top) - (crect.bottom - crect.top); - SetWindowPos(This->hWnd, NULL, 0, 0, width + h +20, height + v+20, SWP_NOZORDER|SWP_NOMOVE); - This->WindowPos.left = 0; - This->WindowPos.top = 0; - This->WindowPos.right = width; - This->WindowPos.bottom = abs(height); + /* Honor previously set WindowPos */ + TRACE("WindowPos: %d %d %d %d\n", This->WindowPos.left, This->WindowPos.top, This->WindowPos.right, This->WindowPos.bottom); + SetWindowPos(This->hWnd, NULL, + This->WindowPos.left, + This->WindowPos.top, + This->WindowPos.right - This->WindowPos.left, + This->WindowPos.bottom - This->WindowPos.top, + SWP_NOZORDER|SWP_NOMOVE); GetClientRect(This->hWnd, &This->DestRect); This->init = TRUE; }

1 0

Evan Stade : gdiplus: Implemented GdipDrawCurve2.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: 5c8b83c524c7059edc4b9ef14f88c08f9760506e URL: http://source.winehq.org/git/wine.git/?a=commit;h=5c8b83c524c7059edc4b9ef14… Author: Evan Stade <estade(a)gmail.com> Date: Tue Jun 19 19:31:28 2007 -0700 gdiplus: Implemented GdipDrawCurve2. --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/graphics.c | 83 +++++++++++++++++++++++++++++++++++++++++++++ include/gdiplusflat.h | 3 ++ include/gdiplusgpstubs.h | 1 + include/gdiplustypes.h | 52 +++++++++++++++++++++++++++- 5 files changed, 139 insertions(+), 2 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index 3036155..775bde3 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -153,7 +153,7 @@ @ stub GdipDrawClosedCurve2I @ stub GdipDrawClosedCurve @ stub GdipDrawClosedCurveI -@ stub GdipDrawCurve2 +@ stdcall GdipDrawCurve2(ptr ptr ptr long long) @ stub GdipDrawCurve2I @ stub GdipDrawCurve3 @ stub GdipDrawCurve3I diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index a4ba0ca..f917b65 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -27,6 +27,9 @@ #include "gdiplus_private.h" #include "wine/debug.h" +/* looks-right constant */ +#define TENSION_CONST (0.3) + static inline INT roundr(REAL x) { return (INT) floor(x+0.5); @@ -77,6 +80,34 @@ static GpStatus draw_pie(GpGraphics *graphics, HBRUSH gdibrush, HPEN gdipen, return Ok; } +/* GdipDrawCurve helper function. + * Calculates Bezier points from cardinal spline points. */ +static void calc_curve_bezier(CONST GpPointF *pts, REAL tension, REAL *x1, + REAL *y1, REAL *x2, REAL *y2) +{ + REAL xdiff, ydiff; + + /* calculate tangent */ + xdiff = pts[2].X - pts[0].X; + ydiff = pts[2].Y - pts[0].Y; + + /* apply tangent to get control points */ + *x1 = pts[1].X - tension * xdiff; + *y1 = pts[1].Y - tension * ydiff; + *x2 = pts[1].X + tension * xdiff; + *y2 = pts[1].Y + tension * ydiff; +} + +/* GdipDrawCurve helper function. + * Calculates Bezier points from cardinal spline endpoints. */ +static void calc_curve_bezier_endp(REAL xend, REAL yend, REAL xadj, REAL yadj, + REAL tension, REAL *x, REAL *y) +{ + /* tangent at endpoints is the line from the endpoint to the adjacent point */ + *x = roundr(tension * (xadj - xend) + xend); + *y = roundr(tension * (yadj - yend) + yend); +} + GpStatus WINGDIPAPI GdipCreateFromHDC(HDC hdc, GpGraphics **graphics) { if(hdc == NULL) @@ -160,6 +191,58 @@ GpStatus WINGDIPAPI GdipDrawBezier(GpGraphics *graphics, GpPen *pen, REAL x1, return Ok; } +/* Approximates cardinal spline with Bezier curves. */ +GpStatus WINGDIPAPI GdipDrawCurve2(GpGraphics *graphics, GpPen *pen, + GDIPCONST GpPointF *points, INT count, REAL tension) +{ + HGDIOBJ old_pen; + + /* PolyBezier expects count*3-2 points. */ + int i, len_pt = count*3-2; + POINT pt[len_pt]; + REAL x1, x2, y1, y2; + + if(!graphics || !pen) + return InvalidParameter; + + tension = tension * TENSION_CONST; + + calc_curve_bezier_endp(points[0].X, points[0].Y, points[1].X, points[1].Y, + tension, &x1, &y1); + + pt[0].x = roundr(points[0].X); + pt[0].y = roundr(points[0].Y); + pt[1].x = roundr(x1); + pt[1].y = roundr(y1); + + for(i = 0; i < count-2; i++){ + calc_curve_bezier(&(points[i]), tension, &x1, &y1, &x2, &y2); + + pt[3*i+2].x = roundr(x1); + pt[3*i+2].y = roundr(y1); + pt[3*i+3].x = roundr(points[i+1].X); + pt[3*i+3].y = roundr(points[i+1].Y); + pt[3*i+4].x = roundr(x2); + pt[3*i+4].y = roundr(y2); + } + + calc_curve_bezier_endp(points[count-1].X, points[count-1].Y, + points[count-2].X, points[count-2].Y, tension, &x1, &y1); + + pt[len_pt-2].x = x1; + pt[len_pt-2].y = y1; + pt[len_pt-1].x = roundr(points[count-1].X); + pt[len_pt-1].y = roundr(points[count-1].Y); + + old_pen = SelectObject(graphics->hdc, pen->gdipen); + + PolyBezier(graphics->hdc, pt, len_pt); + + SelectObject(graphics->hdc, old_pen); + + return Ok; +} + GpStatus WINGDIPAPI GdipDrawLineI(GpGraphics *graphics, GpPen *pen, INT x1, INT y1, INT x2, INT y2) { diff --git a/include/gdiplusflat.h b/include/gdiplusflat.h index 538f1f4..e5c46b3 100644 --- a/include/gdiplusflat.h +++ b/include/gdiplusflat.h @@ -21,6 +21,8 @@ #define WINGDIPAPI __stdcall +#define GDIPCONST const + #ifdef __cplusplus extern "C" { #endif @@ -34,6 +36,7 @@ GpStatus WINGDIPAPI GdipDeleteGraphics(GpGraphics *); GpStatus WINGDIPAPI GdipDrawArc(GpGraphics*,GpPen*,REAL,REAL,REAL,REAL,REAL,REAL); GpStatus WINGDIPAPI GdipDrawBezier(GpGraphics*,GpPen*,REAL,REAL,REAL,REAL,REAL, REAL,REAL,REAL); +GpStatus WINGDIPAPI GdipDrawCurve2(GpGraphics*,GpPen*,GDIPCONST GpPointF*,INT,REAL); GpStatus WINGDIPAPI GdipDrawLineI(GpGraphics*,GpPen*,INT,INT,INT,INT); GpStatus WINGDIPAPI GdipDrawPie(GpGraphics*,GpPen*,REAL,REAL,REAL,REAL,REAL,REAL); GpStatus WINGDIPAPI GdipDrawRectangleI(GpGraphics*,GpPen*,INT,INT,INT,INT); diff --git a/include/gdiplusgpstubs.h b/include/gdiplusgpstubs.h index dcafc11..727ca06 100644 --- a/include/gdiplusgpstubs.h +++ b/include/gdiplusgpstubs.h @@ -38,5 +38,6 @@ typedef struct GpSolidFill GpSolidFill; typedef Status GpStatus; typedef Unit GpUnit; typedef BrushType GpBrushType; +typedef PointF GpPointF; #endif diff --git a/include/gdiplustypes.h b/include/gdiplustypes.h index 006fe09..38fdf58 100644 --- a/include/gdiplustypes.h +++ b/include/gdiplustypes.h @@ -45,7 +45,57 @@ enum Status{ PropertyNotSupported = 20 }; -#ifndef __cplusplus +#ifdef __cplusplus + +class PointF +{ +public: + PointF() + { + X = Y = 0.0f; + } + + PointF(IN const PointF &pt) + { + X = pt.X; + Y = pt.Y; + } + + /* FIXME: missing constructor that takes a SizeF */ + + PointF(IN REAL x, IN REAL y) + { + X = x; + Y = y; + } + + PointF operator+(IN const PointF& pt) const + { + return PointF(X + pt.X, Y + pt.Y); + } + + PointF operator-(IN const PointF& pt) const + { + return PointF(X - pt.X, Y - pt.Y); + } + + BOOL Equals(IN const PointF& pt) + { + return (X == pt.X) && (Y == pt.Y); + } + +public: + REAL X; + REAL Y; +}; + +#else /* end of c++ typedefs */ + +typedef struct PointF +{ + REAL X; + REAL Y; +} PointF; typedef enum Status Status;

1 0

Stefan Dösinger : ddraw: Test for incorrect surface desc sizes in GetSurfaceDesc.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: bf23e5f3f738e80c9cc9873dbfbb6f4d5c33bec8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bf23e5f3f738e80c9cc9873db… Author: Stefan Dösinger <stefandoesinger(a)gmx.at> Date: Fri Jun 8 17:22:16 2007 +0200 ddraw: Test for incorrect surface desc sizes in GetSurfaceDesc. --- dlls/ddraw/surface.c | 7 ++--- dlls/ddraw/surface_thunks.c | 30 +++++++++++++++++++-- dlls/ddraw/tests/dsurface.c | 61 +++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 91 insertions(+), 7 deletions(-) diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index 43c8448..934999c 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -1592,11 +1592,10 @@ IDirectDrawSurfaceImpl_GetSurfaceDesc(IDirectDrawSurface7 *iface, if(!DDSD) return DDERR_INVALIDPARAMS; - if ((DDSD->dwSize < sizeof(DDSURFACEDESC)) || - (DDSD->dwSize > sizeof(DDSURFACEDESC2))) + if (DDSD->dwSize != sizeof(DDSURFACEDESC2)) { - ERR("Impossible/Strange struct size %d.\n",DDSD->dwSize); - return DDERR_GENERIC; + WARN("Incorrect struct size %d, returning DDERR_INVALIDPARAMS\n",DDSD->dwSize); + return DDERR_INVALIDPARAMS; } EnterCriticalSection(&ddraw_cs); diff --git a/dlls/ddraw/surface_thunks.c b/dlls/ddraw/surface_thunks.c index 299e4f6..d8ce9ba 100644 --- a/dlls/ddraw/surface_thunks.c +++ b/dlls/ddraw/surface_thunks.c @@ -20,6 +20,7 @@ #include "wine/port.h" #include "wine/debug.h" #include <stdarg.h> +#include <assert.h> #include "windef.h" #include "winbase.h" @@ -41,6 +42,7 @@ (pdds)) WINE_DEFAULT_DEBUG_CHANNEL(ddraw_thunk); +WINE_DECLARE_DEBUG_CHANNEL(ddraw); static HRESULT WINAPI IDirectDrawSurface3Impl_QueryInterface(LPDIRECTDRAWSURFACE3 This, REFIID iid, @@ -299,11 +301,33 @@ IDirectDrawSurface3Impl_GetPixelFormat(LPDIRECTDRAWSURFACE3 This, } static HRESULT WINAPI -IDirectDrawSurface3Impl_GetSurfaceDesc(LPDIRECTDRAWSURFACE3 This, +IDirectDrawSurface3Impl_GetSurfaceDesc(LPDIRECTDRAWSURFACE3 iface, LPDDSURFACEDESC pDDSD) { - return IDirectDrawSurface7_GetSurfaceDesc(CONVERT(This), - (LPDDSURFACEDESC2)pDDSD); + ICOM_THIS_FROM(IDirectDrawSurfaceImpl, IDirectDrawSurface3, iface); + + TRACE_(ddraw)("(%p)->(%p)\n",This,pDDSD); + + if(!pDDSD) + return DDERR_INVALIDPARAMS; + + if (pDDSD->dwSize != sizeof(DDSURFACEDESC)) + { + WARN("Incorrect struct size %d, returning DDERR_INVALIDPARAMS\n",pDDSD->dwSize); + return DDERR_INVALIDPARAMS; + } + + EnterCriticalSection(&ddraw_cs); + DD_STRUCT_COPY_BYSIZE(pDDSD,(DDSURFACEDESC *) &This->surface_desc); + TRACE("Returning surface desc:\n"); + if (TRACE_ON(ddraw)) + { + /* DDRAW_dump_surface_desc handles the smaller size */ + DDRAW_dump_surface_desc((DDSURFACEDESC2 *) pDDSD); + } + + LeaveCriticalSection(&ddraw_cs); + return DD_OK; } static HRESULT WINAPI diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c index 449e49f..55d7cbd 100644 --- a/dlls/ddraw/tests/dsurface.c +++ b/dlls/ddraw/tests/dsurface.c @@ -2259,6 +2259,66 @@ static void BltParamTest(void) IDirectDrawSurface_Release(surface2); } +static void StructSizeTest(void) +{ + IDirectDrawSurface *surface1; + IDirectDrawSurface7 *surface7; + union { + DDSURFACEDESC desc1; + DDSURFACEDESC2 desc2; + char blob[1024]; /* To get a buch of writeable memory */ + } desc; + DDSURFACEDESC create; + HRESULT hr; + + memset(&desc, 0, sizeof(desc)); + memset(&create, 0, sizeof(create)); + + memset(&create, 0, sizeof(create)); + create.dwSize = sizeof(create); + create.dwFlags = DDSD_CAPS | DDSD_HEIGHT | DDSD_WIDTH; + create.ddsCaps.dwCaps |= DDSCAPS_OFFSCREENPLAIN; + create.dwHeight = 128; + create.dwWidth = 128; + hr = IDirectDraw_CreateSurface(lpDD, &create, &surface1, NULL); + ok(hr == DD_OK, "Creating an offscreen plain surface failed with %08x\n", hr); + hr = IDirectDrawSurface_QueryInterface(surface1, &IID_IDirectDrawSurface7, (void **) &surface7); + ok(hr == DD_OK, "IDirectDrawSurface_QueryInterface failed with %08x\n", hr); + + desc.desc1.dwSize = sizeof(DDSURFACEDESC); + hr = IDirectDrawSurface_GetSurfaceDesc(surface1, &desc.desc1); + ok(hr == DD_OK, "IDirectDrawSurface_GetSurfaceDesc with desc size sizeof(DDSURFACEDESC) returned %08x\n", hr); + hr = IDirectDrawSurface7_GetSurfaceDesc(surface7, &desc.desc2); + ok(hr == DDERR_INVALIDPARAMS, "IDirectDrawSurface7_GetSurfaceDesc with desc size sizeof(DDSURFACEDESC) returned %08x\n", hr); + + desc.desc2.dwSize = sizeof(DDSURFACEDESC2); + hr = IDirectDrawSurface_GetSurfaceDesc(surface1, &desc.desc1); + ok(hr == DDERR_INVALIDPARAMS, "IDirectDrawSurface_GetSurfaceDesc with desc size sizeof(DDSURFACEDESC2) returned %08x\n", hr); + hr = IDirectDrawSurface7_GetSurfaceDesc(surface7, &desc.desc2); + ok(hr == DD_OK, "IDirectDrawSurface7_GetSurfaceDesc with desc size sizeof(DDSURFACEDESC2) returned %08x\n", hr); + + desc.desc2.dwSize = 0; + hr = IDirectDrawSurface_GetSurfaceDesc(surface1, &desc.desc1); + ok(hr == DDERR_INVALIDPARAMS, "IDirectDrawSurface_GetSurfaceDesc with desc size 0 returned %08x\n", hr); + hr = IDirectDrawSurface7_GetSurfaceDesc(surface7, &desc.desc2); + ok(hr == DDERR_INVALIDPARAMS, "IDirectDrawSurface7_GetSurfaceDesc with desc size 0 returned %08x\n", hr); + + desc.desc1.dwSize = sizeof(DDSURFACEDESC) + 1; + hr = IDirectDrawSurface_GetSurfaceDesc(surface1, &desc.desc1); + ok(hr == DDERR_INVALIDPARAMS, "IDirectDrawSurface_GetSurfaceDesc with desc size sizeof(DDSURFACEDESC) + 1 returned %08x\n", hr); + hr = IDirectDrawSurface7_GetSurfaceDesc(surface7, &desc.desc2); + ok(hr == DDERR_INVALIDPARAMS, "IDirectDrawSurface7_GetSurfaceDesc with desc size sizeof(DDSURFACEDESC) + 1 returned %08x\n", hr); + + desc.desc2.dwSize = sizeof(DDSURFACEDESC2) + 1; + hr = IDirectDrawSurface_GetSurfaceDesc(surface1, &desc.desc1); + ok(hr == DDERR_INVALIDPARAMS, "IDirectDrawSurface_GetSurfaceDesc with desc size sizeof(DDSURFACEDESC2) + 1returned %08x\n", hr); + hr = IDirectDrawSurface7_GetSurfaceDesc(surface7, &desc.desc2); + ok(hr == DDERR_INVALIDPARAMS, "IDirectDrawSurface7_GetSurfaceDesc with desc size sizeof(DDSURFACEDESC2) + 1returned %08x\n", hr); + + IDirectDrawSurface7_Release(surface7); + IDirectDrawSurface_Release(surface1); +} + START_TEST(dsurface) { if (!CreateDirectDraw()) @@ -2279,5 +2339,6 @@ START_TEST(dsurface) SizeTest(); PrivateDataTest(); BltParamTest(); + StructSizeTest(); ReleaseDirectDraw(); }

1 0

Marcus Meissner : wined3d: Fixed pointer casting and additions.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: 332d14953b8119e47d13f3e93005976f0c9bf347 URL: http://source.winehq.org/git/wine.git/?a=commit;h=332d14953b8119e47d13f3e93… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Wed Jun 20 08:57:20 2007 +0200 wined3d: Fixed pointer casting and additions. --- dlls/wined3d/vertexbuffer.c | 27 +++++++++++++++++---------- 1 files changed, 17 insertions(+), 10 deletions(-) diff --git a/dlls/wined3d/vertexbuffer.c b/dlls/wined3d/vertexbuffer.c index 7a79423..5bc32d8 100644 --- a/dlls/wined3d/vertexbuffer.c +++ b/dlls/wined3d/vertexbuffer.c @@ -104,13 +104,20 @@ static DWORD WINAPI IWineD3DVertexBufferImpl_GetPriority(IWineD3DVertexBuffer return IWineD3DResourceImpl_GetPriority((IWineD3DResource *)iface); } -static void fixup_vertices(BYTE *src, BYTE *dst, int stride, int num, BYTE *pos, BOOL haspos, BYTE *diffuse, BOOL hasdiffuse, BYTE *specular, BOOL hasspecular) { +static void fixup_vertices( + BYTE *src, BYTE *dst, + int stride, + int num, + int pos, BOOL haspos, + int diffuse, BOOL hasdiffuse, + int specular, BOOL hasspecular +) { int i; float x, y, z, w; for(i = num - 1; i >= 0; i--) { if(haspos) { - float *p = (float *) (((int) src + (int) pos) + i * stride); + float *p = (float *) ((src + pos) + i * stride); /* rhw conversion like in drawStridedSlow */ if(p[3] == 1.0 || ((p[3] < eps) && (p[3] > -eps))) { @@ -124,15 +131,15 @@ static void fixup_vertices(BYTE *src, BYTE *dst, int stride, int num, BYTE *pos, y = p[1] * w; z = p[2] * w; } - p = (float *) ((int) dst + i * stride + (int) pos); + p = (float *) (dst + i * stride + pos); p[0] = x; p[1] = y; p[2] = z; p[3] = w; } if(hasdiffuse) { - DWORD srcColor, *dstColor = (DWORD *) (dst + i * stride + (int) diffuse); - srcColor = * (DWORD *) ( ((int) src + (int) diffuse) + i * stride); + DWORD srcColor, *dstColor = (DWORD *) (dst + i * stride + diffuse); + srcColor = * (DWORD *) ( (src + diffuse) + i * stride); /* Color conversion like in drawStridedSlow. watch out for little endianity * If we want that stuff to work on big endian machines too we have to consider more things @@ -149,8 +156,8 @@ static void fixup_vertices(BYTE *src, BYTE *dst, int stride, int num, BYTE *pos, *dstColor |= (srcColor & 0x000000ff) << 16; /* Blue */ } if(hasspecular) { - DWORD srcColor, *dstColor = (DWORD *) (dst + i * stride + (int) specular); - srcColor = * (DWORD *) ( ((int) src + (int) specular) + i * stride); + DWORD srcColor, *dstColor = (DWORD *) (dst + i * stride + specular); + srcColor = * (DWORD *) ( (src + specular) + i * stride); /* Similar to diffuse * TODO: Write the alpha value out for fog coords @@ -365,13 +372,13 @@ static void WINAPI IWineD3DVertexBufferImpl_PreLoad(IWineD3DVertexBuffer *if fixup_vertices(data, data, stride, ( end - start) / stride, /* Position */ - This->strided.u.s.position.lpData, /* Data location */ + (int)This->strided.u.s.position.lpData, /* Data location */ This->strided.u.s.position_transformed, /* Do convert? */ /* Diffuse color */ - This->strided.u.s.diffuse.lpData, /* Location */ + (int)This->strided.u.s.diffuse.lpData, /* Location */ This->strided.u.s.diffuse.dwType == WINED3DDECLTYPE_SHORT4 || This->strided.u.s.diffuse.dwType == WINED3DDECLTYPE_D3DCOLOR, /* Convert? */ /* specular color */ - This->strided.u.s.specular.lpData, /* location */ + (int)This->strided.u.s.specular.lpData, /* location */ This->strided.u.s.specular.dwType == WINED3DDECLTYPE_SHORT4 || This->strided.u.s.specular.dwType == WINED3DDECLTYPE_D3DCOLOR); ENTER_GL();

1 0

Alexandre Julliard : server: Don' t crash when trying to set a file lock on a device.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: 60901b7542e68e466b457cadfe726100f7737387 URL: http://source.winehq.org/git/wine.git/?a=commit;h=60901b7542e68e466b457cadf… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Jun 20 13:14:55 2007 +0200 server: Don't crash when trying to set a file lock on a device. --- server/fd.c | 12 ++++++------ server/trace.c | 1 + 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/server/fd.c b/server/fd.c index 7a7d572..cf3a306 100644 --- a/server/fd.c +++ b/server/fd.c @@ -1184,12 +1184,6 @@ static struct file_lock *add_lock( struct fd *fd, int shared, file_pos_t start, { struct file_lock *lock; - if (!fd->inode) /* not a regular file */ - { - set_error( STATUS_INVALID_HANDLE ); - return NULL; - } - if (!(lock = alloc_object( &file_lock_ops ))) return NULL; lock->shared = shared; lock->start = start; @@ -1259,6 +1253,12 @@ obj_handle_t lock_fd( struct fd *fd, file_pos_t start, file_pos_t count, int sha struct list *ptr; file_pos_t end = start + count; + if (!fd->inode) /* not a regular file */ + { + set_error( STATUS_INVALID_DEVICE_REQUEST ); + return 0; + } + /* don't allow wrapping locks */ if (end && end < start) { diff --git a/server/trace.c b/server/trace.c index 312b68d..3dd614a 100644 --- a/server/trace.c +++ b/server/trace.c @@ -4246,6 +4246,7 @@ static const struct { "ILLEGAL_FUNCTION", STATUS_ILLEGAL_FUNCTION }, { "INSTANCE_NOT_AVAILABLE", STATUS_INSTANCE_NOT_AVAILABLE }, { "INVALID_CID", STATUS_INVALID_CID }, + { "INVALID_DEVICE_REQUEST", STATUS_INVALID_DEVICE_REQUEST }, { "INVALID_FILE_FOR_SECTION", STATUS_INVALID_FILE_FOR_SECTION }, { "INVALID_HANDLE", STATUS_INVALID_HANDLE }, { "INVALID_PARAMETER", STATUS_INVALID_PARAMETER },

1 0

Andrew Talbot : netapi32: Constify some variables.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: 2a2a3b4ea450baca299f60886ff243c19f61a036 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2a2a3b4ea450baca299f60886… Author: Andrew Talbot <Andrew.Talbot(a)talbotville.com> Date: Tue Jun 19 22:06:48 2007 +0100 netapi32: Constify some variables. --- dlls/netapi32/access.c | 2 +- dlls/netapi32/nbt.c | 8 ++++---- dlls/netapi32/netbios.c | 4 ++-- dlls/netapi32/netbios.h | 4 ++-- dlls/netapi32/wksta.c | 2 +- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/netapi32/access.c b/dlls/netapi32/access.c index 467c124..fb630fc 100644 --- a/dlls/netapi32/access.c +++ b/dlls/netapi32/access.c @@ -512,7 +512,7 @@ static void ACCESS_QueryGuestDisplayInformation(PNET_DISPLAY_USER *buf, PDWORD p /************************************************************ * Copies NET_DISPLAY_USER record. */ -static void ACCESS_CopyDisplayUser(PNET_DISPLAY_USER dest, LPWSTR *dest_buf, +static void ACCESS_CopyDisplayUser(const NET_DISPLAY_USER *dest, LPWSTR *dest_buf, PNET_DISPLAY_USER src) { LPWSTR str = *dest_buf; diff --git a/dlls/netapi32/nbt.c b/dlls/netapi32/nbt.c index 27dba60..a3e38ba 100644 --- a/dlls/netapi32/nbt.c +++ b/dlls/netapi32/nbt.c @@ -288,7 +288,7 @@ typedef BOOL (*NetBTAnswerCallback)(void *data, WORD answerCount, * Returns NRC_GOODRET on timeout or a valid response received, something else * on error. */ -static UCHAR NetBTWaitForNameResponse(NetBTAdapter *adapter, SOCKET fd, +static UCHAR NetBTWaitForNameResponse(const NetBTAdapter *adapter, SOCKET fd, DWORD waitUntil, NetBTAnswerCallback answerCallback, void *data) { BOOL found = FALSE; @@ -424,7 +424,7 @@ static BOOL NetBTFindNameAnswerCallback(void *pVoid, WORD answerCount, queryData->cacheEntry->numAddresses < answerCount) { queryData->cacheEntry->addresses[queryData->cacheEntry-> - numAddresses++] = *(PDWORD)(rData + 2); + numAddresses++] = *(const DWORD *)(rData + 2); ret = queryData->cacheEntry->numAddresses < answerCount; } else @@ -444,7 +444,7 @@ static BOOL NetBTFindNameAnswerCallback(void *pVoid, WORD answerCount, * Returns NRC_GOODRET on success, though this may not mean the name was * resolved--check whether *cacheEntry is NULL. */ -static UCHAR NetBTNameWaitLoop(NetBTAdapter *adapter, SOCKET fd, PNCB ncb, +static UCHAR NetBTNameWaitLoop(const NetBTAdapter *adapter, SOCKET fd, const NCB *ncb, DWORD sendTo, BOOL broadcast, DWORD timeout, DWORD maxQueries, NBNameCacheEntry **cacheEntry) { @@ -1274,7 +1274,7 @@ static void NetBTCleanup(void) } } -static UCHAR NetBTRegisterAdapter(PMIB_IPADDRROW ipRow) +static UCHAR NetBTRegisterAdapter(const MIB_IPADDRROW *ipRow) { UCHAR ret; NetBTAdapter *adapter; diff --git a/dlls/netapi32/netbios.c b/dlls/netapi32/netbios.c index 3362f03..a55f556 100644 --- a/dlls/netapi32/netbios.c +++ b/dlls/netapi32/netbios.c @@ -640,7 +640,7 @@ static UCHAR nbInternalHangup(NetBIOSAdapter *adapter, NetBIOSSession *session) return NRC_GOODRET; } -static UCHAR nbHangup(NetBIOSAdapter *adapter, PNCB ncb) +static UCHAR nbHangup(NetBIOSAdapter *adapter, const NCB *ncb) { UCHAR ret; NetBIOSSession *session; @@ -661,7 +661,7 @@ static UCHAR nbHangup(NetBIOSAdapter *adapter, PNCB ncb) return ret; } -void NetBIOSHangupSession(PNCB ncb) +void NetBIOSHangupSession(const NCB *ncb) { NetBIOSAdapter *adapter; diff --git a/dlls/netapi32/netbios.h b/dlls/netapi32/netbios.h index 7477410..1279a5a 100644 --- a/dlls/netapi32/netbios.h +++ b/dlls/netapi32/netbios.h @@ -92,7 +92,7 @@ void NetBIOSEnumAdapters(ULONG transport, NetBIOSEnumAdaptersCallback cb, * This function is intended for use by a transport, if the session is closed * by some error in the transport layer. */ -void NetBIOSHangupSession(PNCB ncb); +void NetBIOSHangupSession(const NCB *ncb); /** * Functions a transport implementation must implement @@ -132,7 +132,7 @@ typedef void (*NetBIOSCleanup)(void); * some calls (recv) will block indefinitely, so a reset, shutdown, etc. will * never occur. */ -#define NCB_CANCELLED(pncb) *(PBOOL)((pncb)->ncb_reserve) +#define NCB_CANCELLED(pncb) *(const BOOL *)((pncb)->ncb_reserve) typedef UCHAR (*NetBIOSAstat)(void *adapter, PNCB ncb); typedef UCHAR (*NetBIOSFindName)(void *adapter, PNCB ncb); diff --git a/dlls/netapi32/wksta.c b/dlls/netapi32/wksta.c index fbc4670..c51ef02 100644 --- a/dlls/netapi32/wksta.c +++ b/dlls/netapi32/wksta.c @@ -71,7 +71,7 @@ BOOL NETAPI_IsLocalComputer(LPCWSTR ServerName) } } -static void wprint_mac(WCHAR* buffer, int len, PMIB_IFROW ifRow) +static void wprint_mac(WCHAR* buffer, int len, const MIB_IFROW *ifRow) { int i; unsigned char val;

1 0

Evan Stade : gdiplus: Implemented GdipDrawArc.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: c42f8794a1519ac0dba46c7c638372ba3310b121 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c42f8794a1519ac0dba46c7c6… Author: Evan Stade <estade(a)gmail.com> Date: Tue Jun 19 19:31:19 2007 -0700 gdiplus: Implemented GdipDrawArc. --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/graphics.c | 25 +++++++++++++++++++++++++ include/gdiplusflat.h | 1 + 3 files changed, 27 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index 4916853..3036155 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -142,7 +142,7 @@ @ stub GdipDeleteStringFormat @ stub GdipDisposeImage @ stub GdipDisposeImageAttributes -@ stub GdipDrawArc +@ stdcall GdipDrawArc(ptr ptr long long long long long long) @ stub GdipDrawArcI @ stdcall GdipDrawBezier(ptr ptr long long long long long long long long) @ stub GdipDrawBezierI diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index b455cc4..a4ba0ca 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -116,6 +116,31 @@ GpStatus WINGDIPAPI GdipDeleteGraphics(GpGraphics *graphics) return Ok; } +GpStatus WINGDIPAPI GdipDrawArc(GpGraphics *graphics, GpPen *pen, REAL x, + REAL y, REAL width, REAL height, REAL startAngle, REAL sweepAngle) +{ + HGDIOBJ old_pen; + REAL x_0, y_0, x_1, y_1, x_2, y_2; + + if(!graphics || !pen) + return InvalidParameter; + + old_pen = SelectObject(graphics->hdc, pen->gdipen); + + x_0 = x + (width/2.0); + y_0 = y + (height/2.0); + + deg2xy(startAngle+sweepAngle, x_0, y_0, &x_1, &y_1); + deg2xy(startAngle, x_0, y_0, &x_2, &y_2); + + Arc(graphics->hdc, roundr(x), roundr(y), roundr(x+width), roundr(y+height), + roundr(x_1), roundr(y_1), roundr(x_2), roundr(y_2)); + + SelectObject(graphics->hdc, old_pen); + + return Ok; +} + GpStatus WINGDIPAPI GdipDrawBezier(GpGraphics *graphics, GpPen *pen, REAL x1, REAL y1, REAL x2, REAL y2, REAL x3, REAL y3, REAL x4, REAL y4) { diff --git a/include/gdiplusflat.h b/include/gdiplusflat.h index e949301..538f1f4 100644 --- a/include/gdiplusflat.h +++ b/include/gdiplusflat.h @@ -31,6 +31,7 @@ GpStatus WINGDIPAPI GdipDeletePen(GpPen*); GpStatus WINGDIPAPI GdipCreateFromHDC(HDC,GpGraphics**); GpStatus WINGDIPAPI GdipCreateFromHWND(HWND,GpGraphics**); GpStatus WINGDIPAPI GdipDeleteGraphics(GpGraphics *); +GpStatus WINGDIPAPI GdipDrawArc(GpGraphics*,GpPen*,REAL,REAL,REAL,REAL,REAL,REAL); GpStatus WINGDIPAPI GdipDrawBezier(GpGraphics*,GpPen*,REAL,REAL,REAL,REAL,REAL, REAL,REAL,REAL); GpStatus WINGDIPAPI GdipDrawLineI(GpGraphics*,GpPen*,INT,INT,INT,INT);

1 0

Michael Stefaniuc : msvcrt: Move definition of 2 static inline functions up and remove the forward declaration of those . Found by sparse.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: 814ca1295acf7b1c2329e5e3a69b34bb3c7968d0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=814ca1295acf7b1c2329e5e3a… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Jun 19 22:49:04 2007 +0200 msvcrt: Move definition of 2 static inline functions up and remove the forward declaration of those. Found by sparse. --- dlls/msvcrt/main.c | 47 ++++++++++++++++++++++------------------------- 1 files changed, 22 insertions(+), 25 deletions(-) diff --git a/dlls/msvcrt/main.c b/dlls/msvcrt/main.c index fa6affe..1edaa73 100644 --- a/dlls/msvcrt/main.c +++ b/dlls/msvcrt/main.c @@ -26,9 +26,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(msvcrt); /* Index to TLS */ DWORD msvcrt_tls_index; -static inline BOOL msvcrt_init_tls(void); -static inline BOOL msvcrt_free_tls(void); - static const char* msvcrt_get_reason(DWORD reason) { switch (reason) @@ -41,6 +38,28 @@ static const char* msvcrt_get_reason(DWORD reason) return "UNKNOWN"; } +static inline BOOL msvcrt_init_tls(void) +{ + msvcrt_tls_index = TlsAlloc(); + + if (msvcrt_tls_index == TLS_OUT_OF_INDEXES) + { + ERR("TlsAlloc() failed!\n"); + return FALSE; + } + return TRUE; +} + +static inline BOOL msvcrt_free_tls(void) +{ + if (!TlsFree(msvcrt_tls_index)) + { + ERR("TlsFree() failed!\n"); + return FALSE; + } + return TRUE; +} + /********************************************************************* * Init */ @@ -95,28 +114,6 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) return TRUE; } -static inline BOOL msvcrt_init_tls(void) -{ - msvcrt_tls_index = TlsAlloc(); - - if (msvcrt_tls_index == TLS_OUT_OF_INDEXES) - { - ERR("TlsAlloc() failed!\n"); - return FALSE; - } - return TRUE; -} - -static inline BOOL msvcrt_free_tls(void) -{ - if (!TlsFree(msvcrt_tls_index)) - { - ERR("TlsFree() failed!\n"); - return FALSE; - } - return TRUE; -} - /********************************************************************* * $I10_OUTPUT (MSVCRT.@) * Function not really understood but needed to make the DLL work

1 0

Jason Green : winscard: Add extended stub for SCardEstablishContext.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: 832c43f9a736aa1e19e03bacc450f99b86ed4465 URL: http://source.winehq.org/git/wine.git/?a=commit;h=832c43f9a736aa1e19e03bacc… Author: Jason Green <jave27(a)gmail.com> Date: Tue Jun 19 11:00:35 2007 -0400 winscard: Add extended stub for SCardEstablishContext. --- dlls/winscard/winscard.c | 8 ++++++++ dlls/winscard/winscard.spec | 2 +- 2 files changed, 9 insertions(+), 1 deletions(-) diff --git a/dlls/winscard/winscard.c b/dlls/winscard/winscard.c index d24e0f9..01b731d 100644 --- a/dlls/winscard/winscard.c +++ b/dlls/winscard/winscard.c @@ -50,3 +50,11 @@ BOOL WINAPI DllMain (HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) return TRUE; } + +LONG WINAPI SCardEstablishContext(DWORD dwScope, LPCVOID pvReserved1, + LPCVOID pvReserved2, LPSCARDCONTEXT phContext) +{ + FIXME("(%x,%p,%p,%p) stub\n", dwScope, pvReserved1, pvReserved2, phContext); + SetLastError(ERROR_CALL_NOT_IMPLEMENTED); + return SCARD_F_INTERNAL_ERROR; +} diff --git a/dlls/winscard/winscard.spec b/dlls/winscard/winscard.spec index 6dd83d8..f223415 100644 --- a/dlls/winscard/winscard.spec +++ b/dlls/winscard/winscard.spec @@ -12,7 +12,7 @@ @ stub SCardControl @ stub SCardDisconnect @ stub SCardEndTransaction -@ stub SCardEstablishContext +@ stdcall SCardEstablishContext(long ptr ptr ptr) @ stub SCardForgetCardTypeA @ stub SCardForgetCardTypeW @ stub SCardForgetReaderA

1 0

Rob Shearman : rpcss: Remove lazy timeout mechanism and use __wine_make_system_process instead.
by Alexandre Julliard June 20, 2007

June 20, 2007

Module: wine Branch: master Commit: b9a68259271da3b2e3c03e71445799ea5f84542e URL: http://source.winehq.org/git/wine.git/?a=commit;h=b9a68259271da3b2e3c03e714… Author: Rob Shearman <rob(a)codeweavers.com> Date: Tue Jun 19 13:12:07 2007 +0100 rpcss: Remove lazy timeout mechanism and use __wine_make_system_process instead. --- programs/rpcss/Makefile.in | 2 +- programs/rpcss/np_server.c | 65 ++++---------- programs/rpcss/rpcss.h | 7 +-- programs/rpcss/rpcss_main.c | 200 +++--------------------------------------- 4 files changed, 34 insertions(+), 240 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b9a68259271da3b2e3c03…

1 0