December 2009 - Wine-Commits

Vincent Povirk : ole32: Always allow changes to read-only transacted storages.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: ffc4a49bcb9c98ee76a7b210232db35423e06de1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ffc4a49bcb9c98ee76a7b2102… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Dec 4 18:14:51 2009 -0600 ole32: Always allow changes to read-only transacted storages. The only time the write permissions of transacted storages matter is when committing. --- dlls/ole32/storage32.c | 30 +++++++++++++++--------------- dlls/ole32/tests/storage32.c | 2 +- 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c index 629709a..229fe76 100644 --- a/dlls/ole32/storage32.c +++ b/dlls/ole32/storage32.c @@ -864,15 +864,16 @@ static HRESULT WINAPI StorageBaseImpl_CreateStream( (grfMode & STGM_TRANSACTED)) return STG_E_INVALIDFUNCTION; - /* Can't create a stream on read-only storage */ - if ( STGM_ACCESS_MODE( This->openFlags ) == STGM_READ ) - return STG_E_ACCESSDENIED; - /* - * Check that we're compatible with the parent's storage mode - * if not in transacted mode + * Don't worry about permissions in transacted mode, as we can always write + * changes; we just can't always commit them. */ if(!(This->openFlags & STGM_TRANSACTED)) { + /* Can't create a stream on read-only storage */ + if ( STGM_ACCESS_MODE( This->openFlags ) == STGM_READ ) + return STG_E_ACCESSDENIED; + + /* Can't create a stream with greater access than the parent. */ if ( STGM_ACCESS_MODE( grfMode ) > STGM_ACCESS_MODE( This->openFlags ) ) return STG_E_ACCESSDENIED; } @@ -899,11 +900,6 @@ static HRESULT WINAPI StorageBaseImpl_CreateStream( else return STG_E_FILEALREADYEXISTS; } - else if (STGM_ACCESS_MODE(This->openFlags) == STGM_READ) - { - WARN("read-only storage\n"); - return STG_E_ACCESSDENIED; - } /* * memset the empty entry @@ -1058,7 +1054,8 @@ static HRESULT WINAPI StorageBaseImpl_CreateStorage( /* * Check that we're compatible with the parent's storage mode */ - if ( STGM_ACCESS_MODE( grfMode ) > STGM_ACCESS_MODE( This->openFlags ) ) + if ( !(This->openFlags & STGM_TRANSACTED) && + STGM_ACCESS_MODE( grfMode ) > STGM_ACCESS_MODE( This->openFlags ) ) { WARN("access denied\n"); return STG_E_ACCESSDENIED; @@ -1075,7 +1072,8 @@ static HRESULT WINAPI StorageBaseImpl_CreateStorage( * An element with this name already exists */ if (STGM_CREATE_MODE(grfMode) == STGM_CREATE && - STGM_ACCESS_MODE(This->openFlags) != STGM_READ) + ((This->openFlags & STGM_TRANSACTED) || + STGM_ACCESS_MODE(This->openFlags) != STGM_READ)) { hr = IStorage_DestroyElement(iface, pwcsName); if (FAILED(hr)) @@ -1087,7 +1085,8 @@ static HRESULT WINAPI StorageBaseImpl_CreateStorage( return STG_E_FILEALREADYEXISTS; } } - else if (STGM_ACCESS_MODE(This->openFlags) == STGM_READ) + else if (!(This->openFlags & STGM_TRANSACTED) && + STGM_ACCESS_MODE(This->openFlags) == STGM_READ) { WARN("read-only storage\n"); return STG_E_ACCESSDENIED; @@ -1831,7 +1830,8 @@ static HRESULT WINAPI StorageBaseImpl_DestroyElement( if (This->reverted) return STG_E_REVERTED; - if ( STGM_ACCESS_MODE( This->openFlags ) == STGM_READ ) + if ( !(This->openFlags & STGM_TRANSACTED) && + STGM_ACCESS_MODE( This->openFlags ) == STGM_READ ) return STG_E_ACCESSDENIED; entryToDeleteRef = findElement( diff --git a/dlls/ole32/tests/storage32.c b/dlls/ole32/tests/storage32.c index 2011375..e8a7ec9 100644 --- a/dlls/ole32/tests/storage32.c +++ b/dlls/ole32/tests/storage32.c @@ -1505,7 +1505,7 @@ static void test_nonroot_transacted(void) { /* The storage can be modified. */ r = IStorage_CreateStorage(stg2, stgname, STGM_READWRITE | STGM_SHARE_EXCLUSIVE, 0, 0, &stg3); - todo_wine ok(r==S_OK, "IStorage->CreateStorage failed, hr=%08x\n", r); + ok(r==S_OK, "IStorage->CreateStorage failed, hr=%08x\n", r); if (r == S_OK) IStream_Release(stg3);

1 0

Vincent Povirk : gdiplus: Add tests for multi-frame GIF images.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: d41f68fdb8c79c75891bda45fe46135e6638cd07 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d41f68fdb8c79c75891bda45f… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Dec 24 15:49:23 2009 -0500 gdiplus: Add tests for multi-frame GIF images. --- dlls/gdiplus/tests/image.c | 137 ++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 137 insertions(+), 0 deletions(-) diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c index e090de9..60d8027 100644 --- a/dlls/gdiplus/tests/image.c +++ b/dlls/gdiplus/tests/image.c @@ -1195,6 +1195,142 @@ static void test_colormatrix(void) GdipDisposeImageAttributes(imageattr); } +/* 1x1 pixel gif, 2 frames; first frame is white, second is black */ +static const unsigned char gifanimation[72] = { +0x47,0x49,0x46,0x38,0x39,0x61,0x01,0x00,0x01,0x00,0xa1,0x00,0x00,0x00,0x00,0x00, +0xff,0xff,0xff,0x00,0x00,0x00,0x00,0x00,0x00,0x21,0xf9,0x04,0x00,0x0a,0x00,0xff, +0x00,0x2c,0x00,0x00,0x00,0x00,0x01,0x00,0x01,0x00,0x00,0x02,0x02,0x4c,0x01,0x00, +0x21,0xf9,0x04,0x01,0x0a,0x00,0x01,0x00,0x2c,0x00,0x00,0x00,0x00,0x01,0x00,0x01, +0x00,0x00,0x02,0x02,0x44,0x01,0x00,0x3b +}; + +static void test_multiframegif(void) +{ + LPSTREAM stream; + HGLOBAL hglob; + LPBYTE data; + HRESULT hres; + GpStatus stat; + GpBitmap *bmp; + ARGB color; + UINT count; + GUID dimension; + + /* Test frame functions with an animated GIF */ + hglob = GlobalAlloc (0, sizeof(gifanimation)); + data = GlobalLock (hglob); + memcpy(data, gifanimation, sizeof(gifanimation)); + GlobalUnlock(hglob); + + hres = CreateStreamOnHGlobal(hglob, TRUE, &stream); + ok(hres == S_OK, "Failed to create a stream\n"); + if(hres != S_OK) return; + + stat = GdipCreateBitmapFromStream(stream, &bmp); + ok(stat == Ok, "Failed to create a Bitmap\n"); + if(stat != Ok){ + IStream_Release(stream); + return; + } + + /* Bitmap starts at frame 0 */ + color = 0xdeadbeef; + stat = GdipBitmapGetPixel(bmp, 0, 0, &color); + expect(Ok, stat); + expect(0xffffffff, color); + + /* Check that we get correct metadata */ + stat = GdipImageGetFrameDimensionsCount((GpImage*)bmp,&count); + expect(Ok, stat); + expect(1, count); + + stat = GdipImageGetFrameDimensionsList((GpImage*)bmp, &dimension, 1); + expect(Ok, stat); + expect_guid(&FrameDimensionTime, &dimension, __LINE__, TRUE); + + count = 12345; + stat = GdipImageGetFrameCount((GpImage*)bmp, &dimension, &count); + todo_wine expect(Ok, stat); + todo_wine expect(2, count); + + /* SelectActiveFrame overwrites our current data */ + stat = GdipImageSelectActiveFrame((GpImage*)bmp, &dimension, 1); + expect(Ok, stat); + + color = 0xdeadbeef; + GdipBitmapGetPixel(bmp, 0, 0, &color); + expect(Ok, stat); + todo_wine expect(0xff000000, color); + + stat = GdipImageSelectActiveFrame((GpImage*)bmp, &dimension, 0); + expect(Ok, stat); + + color = 0xdeadbeef; + GdipBitmapGetPixel(bmp, 0, 0, &color); + expect(Ok, stat); + expect(0xffffffff, color); + + /* Write over the image data */ + stat = GdipBitmapSetPixel(bmp, 0, 0, 0xff000000); + expect(Ok, stat); + + /* Switching to the same frame does not overwrite our changes */ + stat = GdipImageSelectActiveFrame((GpImage*)bmp, &dimension, 0); + expect(Ok, stat); + + stat = GdipBitmapGetPixel(bmp, 0, 0, &color); + expect(Ok, stat); + expect(0xff000000, color); + + /* But switching to another frame and back does */ + stat = GdipImageSelectActiveFrame((GpImage*)bmp, &dimension, 1); + expect(Ok, stat); + + stat = GdipImageSelectActiveFrame((GpImage*)bmp, &dimension, 0); + expect(Ok, stat); + + stat = GdipBitmapGetPixel(bmp, 0, 0, &color); + expect(Ok, stat); + todo_wine expect(0xffffffff, color); + + GdipDisposeImage((GpImage*)bmp); + IStream_Release(stream); + + /* Test with a non-animated gif */ + hglob = GlobalAlloc (0, sizeof(gifimage)); + data = GlobalLock (hglob); + memcpy(data, gifimage, sizeof(gifimage)); + GlobalUnlock(hglob); + + hres = CreateStreamOnHGlobal(hglob, TRUE, &stream); + ok(hres == S_OK, "Failed to create a stream\n"); + if(hres != S_OK) return; + + stat = GdipCreateBitmapFromStream(stream, &bmp); + ok(stat == Ok, "Failed to create a Bitmap\n"); + if(stat != Ok){ + IStream_Release(stream); + return; + } + + /* Check metadata */ + stat = GdipImageGetFrameDimensionsCount((GpImage*)bmp,&count); + expect(Ok, stat); + expect(1, count); + + stat = GdipImageGetFrameDimensionsList((GpImage*)bmp, &dimension, 1); + expect(Ok, stat); + expect_guid(&FrameDimensionTime, &dimension, __LINE__, TRUE); + + count = 12345; + stat = GdipImageGetFrameCount((GpImage*)bmp, &dimension, &count); + todo_wine expect(Ok, stat); + todo_wine expect(1, count); + + GdipDisposeImage((GpImage*)bmp); + IStream_Release(stream); +} + START_TEST(image) { struct GdiplusStartupInput gdiplusStartupInput; @@ -1224,6 +1360,7 @@ START_TEST(image) test_getsetpixel(); test_palette(); test_colormatrix(); + test_multiframegif(); GdiplusShutdown(gdiplusToken); }

1 0

Vincent Povirk : gdiplus: Fix memory leak in GdipSetStringFormatMeasurableCharacterRanges.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: 00e1e5ddd5a97fd15b2c94c14ff7bc6e6ce49ba7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=00e1e5ddd5a97fd15b2c94c14… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Dec 24 15:14:45 2009 -0500 gdiplus: Fix memory leak in GdipSetStringFormatMeasurableCharacterRanges. --- dlls/gdiplus/stringformat.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/dlls/gdiplus/stringformat.c b/dlls/gdiplus/stringformat.c index 392f11f..cf6ea1f 100644 --- a/dlls/gdiplus/stringformat.c +++ b/dlls/gdiplus/stringformat.c @@ -252,15 +252,19 @@ GpStatus WINGDIPAPI GdipSetStringFormatLineAlign(GpStringFormat *format, GpStatus WINGDIPAPI GdipSetStringFormatMeasurableCharacterRanges( GpStringFormat *format, INT rangeCount, GDIPCONST CharacterRange *ranges) { + CharacterRange *new_ranges; + if (!(format && ranges)) return InvalidParameter; TRACE("%p, %d, %p\n", format, rangeCount, ranges); - format->character_ranges = GdipAlloc(rangeCount * sizeof(CharacterRange)); - if (!format->character_ranges) + new_ranges = GdipAlloc(rangeCount * sizeof(CharacterRange)); + if (!new_ranges) return OutOfMemory; + GdipFree(format->character_ranges); + format->character_ranges = new_ranges; memcpy(format->character_ranges, ranges, sizeof(CharacterRange) * rangeCount); format->range_count = rangeCount;

1 0

Vincent Povirk : gdiplus: Add some traces for the values of point arguments .
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: 7ded3d83c6148d6ad831772228d666179b5c2687 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7ded3d83c6148d6ad83177222… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Dec 18 15:45:02 2009 -0600 gdiplus: Add some traces for the values of point arguments. --- dlls/gdiplus/brush.c | 6 +++--- dlls/gdiplus/gdiplus.c | 6 ++++++ dlls/gdiplus/gdiplus_private.h | 2 ++ dlls/gdiplus/graphics.c | 3 +++ 4 files changed, 14 insertions(+), 3 deletions(-) diff --git a/dlls/gdiplus/brush.c b/dlls/gdiplus/brush.c index 49e9e0e..ad372c0 100644 --- a/dlls/gdiplus/brush.c +++ b/dlls/gdiplus/brush.c @@ -311,8 +311,8 @@ GpStatus WINGDIPAPI GdipCreateLineBrush(GDIPCONST GpPointF* startpoint, { COLORREF col = ARGB2COLORREF(startcolor); - TRACE("(%p, %p, %x, %x, %d, %p)\n", startpoint, endpoint, - startcolor, endcolor, wrap, line); + TRACE("(%s, %s, %x, %x, %d, %p)\n", debugstr_pointf(startpoint), + debugstr_pointf(endpoint), startcolor, endcolor, wrap, line); if(!line || !startpoint || !endpoint || wrap == WrapModeClamp) return InvalidParameter; @@ -1448,7 +1448,7 @@ GpStatus WINGDIPAPI GdipSetPathGradientCenterColor(GpPathGradient *grad, GpStatus WINGDIPAPI GdipSetPathGradientCenterPoint(GpPathGradient *grad, GpPointF *point) { - TRACE("(%p, %p)\n", grad, point); + TRACE("(%p, %s)\n", grad, debugstr_pointf(point)); if(!grad || !point) return InvalidParameter; diff --git a/dlls/gdiplus/gdiplus.c b/dlls/gdiplus/gdiplus.c index 9fe9332..77cbc5c 100644 --- a/dlls/gdiplus/gdiplus.c +++ b/dlls/gdiplus/gdiplus.c @@ -444,3 +444,9 @@ const char *debugstr_rectf(CONST RectF* rc) if (!rc) return "(null)"; return wine_dbg_sprintf("(%0.2f,%0.2f,%0.2f,%0.2f)", rc->X, rc->Y, rc->Width, rc->Height); } + +const char *debugstr_pointf(CONST PointF* pt) +{ + if (!pt) return "(null)"; + return wine_dbg_sprintf("(%0.2f,%0.2f)", pt->X, pt->Y); +} diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index 6d778a4..7ce4bd5 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -80,6 +80,8 @@ static inline REAL deg2rad(REAL degrees) extern const char *debugstr_rectf(CONST RectF* rc); +extern const char *debugstr_pointf(CONST PointF* pt); + extern void convert_32bppARGB_to_32bppPARGB(UINT width, UINT height, BYTE *dst_bits, INT dst_stride, const BYTE *src_bits, INT src_stride); diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 5ab544d..7d5caf8 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -1876,6 +1876,9 @@ GpStatus WINGDIPAPI GdipDrawImagePointsRect(GpGraphics *graphics, GpImage *image if(!graphics || !image || !points || count != 3) return InvalidParameter; + TRACE("%s %s %s\n", debugstr_pointf(&points[0]), debugstr_pointf(&points[1]), + debugstr_pointf(&points[2])); + memcpy(ptf, points, 3 * sizeof(GpPointF)); transform_and_round_points(graphics, pti, ptf, 3);

1 0

Vincent Povirk : gdiplus: Add traces to unimplemented functions in brush.c.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: d49510e7f009acc39a311af8c210dc12d775b95a URL: http://source.winehq.org/git/wine.git/?a=commit;h=d49510e7f009acc39a311af8c… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Dec 18 15:54:26 2009 -0600 gdiplus: Add traces to unimplemented functions in brush.c. --- dlls/gdiplus/brush.c | 16 ++++++++++++++++ 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/dlls/gdiplus/brush.c b/dlls/gdiplus/brush.c index ad372c0..0f6d707 100644 --- a/dlls/gdiplus/brush.c +++ b/dlls/gdiplus/brush.c @@ -1160,6 +1160,8 @@ GpStatus WINGDIPAPI GdipGetPathGradientSurroundColorsWithCount(GpPathGradient { static int calls; + TRACE("(%p,%p,%p)\n", grad, argb, count); + if(!grad || !argb || !count || (*count < grad->pathdata.Count)) return InvalidParameter; @@ -1415,6 +1417,8 @@ GpStatus WINGDIPAPI GdipSetPathGradientBlend(GpPathGradient *brush, GDIPCONST RE { static int calls; + TRACE("(%p,%p,%p,%i)\n", brush, blend, pos, count); + if(!(calls++)) FIXME("not implemented\n"); @@ -1507,6 +1511,8 @@ GpStatus WINGDIPAPI GdipSetPathGradientSigmaBlend(GpPathGradient *grad, { static int calls; + TRACE("(%p,%0.2f,%0.2f)\n", grad, focus, scale); + if(!grad || focus < 0.0 || focus > 1.0 || scale < 0.0 || scale > 1.0) return InvalidParameter; @@ -1521,6 +1527,8 @@ GpStatus WINGDIPAPI GdipSetPathGradientSurroundColorsWithCount(GpPathGradient { static int calls; + TRACE("(%p,%p,%p)\n", grad, argb, count); + if(!grad || !argb || !count || (*count <= 0) || (*count > grad->pathdata.Count)) return InvalidParameter; @@ -1734,6 +1742,8 @@ GpStatus WINGDIPAPI GdipResetLineTransform(GpLineGradient *brush) { static int calls; + TRACE("(%p)\n", brush); + if(!(calls++)) FIXME("not implemented\n"); @@ -1745,6 +1755,8 @@ GpStatus WINGDIPAPI GdipSetLineTransform(GpLineGradient *brush, { static int calls; + TRACE("(%p,%p)\n", brush, matrix); + if(!(calls++)) FIXME("not implemented\n"); @@ -1756,6 +1768,8 @@ GpStatus WINGDIPAPI GdipScaleLineTransform(GpLineGradient *brush, REAL sx, REAL { static int calls; + TRACE("(%p,%0.2f,%0.2f,%u)\n", brush, sx, sy, order); + if(!(calls++)) FIXME("not implemented\n"); @@ -1823,6 +1837,8 @@ GpStatus WINGDIPAPI GdipRotateLineTransform(GpLineGradient* brush, { static int calls; + TRACE("(%p,%0.2f,%u)\n", brush, angle, order); + if(!brush) return InvalidParameter;

1 0

Vincent Povirk : gdiplus: Remove fixme from GdipImageGetFrameDimensionsCount.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: 7d62e1f776000a44983ab1ed39c89943cabcb982 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7d62e1f776000a44983ab1ed3… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Dec 24 15:52:02 2009 -0500 gdiplus: Remove fixme from GdipImageGetFrameDimensionsCount. --- dlls/gdiplus/image.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/gdiplus/image.c b/dlls/gdiplus/image.c index 9858f33..2716a34 100644 --- a/dlls/gdiplus/image.c +++ b/dlls/gdiplus/image.c @@ -1725,13 +1725,13 @@ GpStatus WINGDIPAPI GdipImageGetFrameCount(GpImage *image, GpStatus WINGDIPAPI GdipImageGetFrameDimensionsCount(GpImage *image, UINT* count) { + /* Native gdiplus 1.1 does not yet support multiple frame dimensions. */ + if(!image || !count) return InvalidParameter; *count = 1; - FIXME("stub\n"); - return Ok; }

1 0

Vincent Povirk : gdiplus: Add traces for values of newly-created pens.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: 7cb42df39b93ba19efcfe0b5522883f57f0e7d12 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7cb42df39b93ba19efcfe0b55… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Dec 18 15:24:51 2009 -0600 gdiplus: Add traces for values of newly-created pens. --- dlls/gdiplus/pen.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/dlls/gdiplus/pen.c b/dlls/gdiplus/pen.c index 4a68a27..6299d67 100644 --- a/dlls/gdiplus/pen.c +++ b/dlls/gdiplus/pen.c @@ -101,6 +101,8 @@ GpStatus WINGDIPAPI GdipClonePen(GpPen *pen, GpPen **clonepen) GdipCloneCustomLineCap(pen->customend, &(*clonepen)->customend); GdipCloneBrush(pen->brush, &(*clonepen)->brush); + TRACE("<-- %p\n", *clonepen); + return Ok; } @@ -154,6 +156,8 @@ GpStatus WINGDIPAPI GdipCreatePen2(GpBrush *brush, REAL width, GpUnit unit, *pen = gp_pen; + TRACE("<-- %p\n", *pen); + return Ok; }

1 0

Alexandre Julliard : gdi32: Use a standard critical section instead of a syslevel for the GDI lock.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: 0d07420ecf920142ba0f6aea80d7140c3ddc6df6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0d07420ecf920142ba0f6aea8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 24 17:42:13 2009 +0100 gdi32: Use a standard critical section instead of a syslevel for the GDI lock. --- dlls/gdi32/gdiobj.c | 34 +++++++++++++++++++--------------- 1 files changed, 19 insertions(+), 15 deletions(-) diff --git a/dlls/gdi32/gdiobj.c b/dlls/gdi32/gdiobj.c index 245b1cc..d32450a 100644 --- a/dlls/gdi32/gdiobj.c +++ b/dlls/gdi32/gdiobj.c @@ -67,14 +67,14 @@ static const LOGPEN DCPen = { PS_SOLID, { 0, 0 }, RGB(0,0,0) }; static HGDIOBJ stock_objects[NB_STOCK_OBJECTS]; -static SYSLEVEL GDI_level; +static CRITICAL_SECTION gdi_section; static CRITICAL_SECTION_DEBUG critsect_debug = { - 0, 0, &GDI_level.crst, + 0, 0, &gdi_section, { &critsect_debug.ProcessLocksList, &critsect_debug.ProcessLocksList }, - 0, 0, { (DWORD_PTR)(__FILE__ ": GDI_level") } + 0, 0, { (DWORD_PTR)(__FILE__ ": gdi_section") } }; -static SYSLEVEL GDI_level = { { &critsect_debug, -1, 0, 0, 0, 0 }, 3 }; +static CRITICAL_SECTION gdi_section = { &critsect_debug, -1, 0, 0, 0, 0 }; /**************************************************************************** @@ -649,18 +649,18 @@ HGDIOBJ alloc_gdi_handle( GDIOBJHDR *obj, WORD type, const struct gdi_obj_funcs obj->funcs = funcs; obj->hdcs = NULL; - _EnterSysLevel( &GDI_level ); + EnterCriticalSection( &gdi_section ); for (i = next_large_handle + 1; i < MAX_LARGE_HANDLES; i++) if (!large_handles[i]) goto found; for (i = 0; i <= next_large_handle; i++) if (!large_handles[i]) goto found; - _LeaveSysLevel( &GDI_level ); + LeaveCriticalSection( &gdi_section ); return 0; found: large_handles[i] = obj; next_large_handle = i; - _LeaveSysLevel( &GDI_level ); + LeaveCriticalSection( &gdi_section ); return (HGDIOBJ)(ULONG_PTR)((i + FIRST_LARGE_HANDLE) << 2); } @@ -678,10 +678,10 @@ void *free_gdi_handle( HGDIOBJ handle ) i = ((ULONG_PTR)handle >> 2) - FIRST_LARGE_HANDLE; if (i >= 0 && i < MAX_LARGE_HANDLES) { - _EnterSysLevel( &GDI_level ); + EnterCriticalSection( &gdi_section ); object = large_handles[i]; large_handles[i] = NULL; - _LeaveSysLevel( &GDI_level ); + LeaveCriticalSection( &gdi_section ); } if (object) { @@ -704,7 +704,7 @@ void *GDI_GetObjPtr( HGDIOBJ handle, WORD type ) GDIOBJHDR *ptr = NULL; int i; - _EnterSysLevel( &GDI_level ); + EnterCriticalSection( &gdi_section ); i = ((UINT_PTR)handle >> 2) - FIRST_LARGE_HANDLE; if (i >= 0 && i < MAX_LARGE_HANDLES) @@ -715,10 +715,10 @@ void *GDI_GetObjPtr( HGDIOBJ handle, WORD type ) if (!ptr) { - _LeaveSysLevel( &GDI_level ); + LeaveCriticalSection( &gdi_section ); WARN( "Invalid handle %p\n", handle ); } - else TRACE("(%p): enter %d\n", handle, GDI_level.crst.RecursionCount); + else TRACE("(%p): enter %d\n", handle, gdi_section.RecursionCount); return ptr; } @@ -730,8 +730,8 @@ void *GDI_GetObjPtr( HGDIOBJ handle, WORD type ) */ void GDI_ReleaseObj( HGDIOBJ handle ) { - TRACE("(%p): leave %d\n", handle, GDI_level.crst.RecursionCount); - _LeaveSysLevel( &GDI_level ); + TRACE("(%p): leave %d\n", handle, gdi_section.RecursionCount); + LeaveCriticalSection( &gdi_section ); } @@ -740,7 +740,11 @@ void GDI_ReleaseObj( HGDIOBJ handle ) */ void GDI_CheckNotLock(void) { - _CheckNotSysLevel( &GDI_level ); + if (gdi_section.OwningThread == ULongToHandle(GetCurrentThreadId()) && gdi_section.RecursionCount) + { + ERR( "BUG: holding GDI lock\n" ); + DebugBreak(); + } }

1 0

Alexandre Julliard : user32: Don' t pass the 16-bit instance through to 32-bit MessageBoxIndirectA.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: d51a6b673c22ad31321822bcb1c9a9ed0675eca0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d51a6b673c22ad31321822bcb… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 23 17:11:53 2009 +0100 user32: Don't pass the 16-bit instance through to 32-bit MessageBoxIndirectA. --- dlls/user.exe16/window.c | 31 +++++++++++++++++++++++++++---- 1 files changed, 27 insertions(+), 4 deletions(-) diff --git a/dlls/user.exe16/window.c b/dlls/user.exe16/window.c index c746a1f..244924f 100644 --- a/dlls/user.exe16/window.c +++ b/dlls/user.exe16/window.c @@ -23,6 +23,9 @@ #include "user_private.h" #include "wine/list.h" #include "wine/server.h" +#include "wine/debug.h" + +WINE_DEFAULT_DEBUG_CHANNEL(win); /* size of buffer needed to store an atom string */ #define ATOM_BUFFER_SIZE 256 @@ -2099,17 +2102,37 @@ INT16 WINAPI SetWindowRgn16( HWND16 hwnd, HRGN16 hrgn, BOOL16 redraw ) */ INT16 WINAPI MessageBoxIndirect16( LPMSGBOXPARAMS16 msgbox ) { + char caption[256], text[256]; MSGBOXPARAMSA msgbox32; msgbox32.cbSize = msgbox->cbSize; msgbox32.hwndOwner = WIN_Handle32( msgbox->hwndOwner ); - msgbox32.hInstance = HINSTANCE_32(msgbox->hInstance); - msgbox32.lpszText = MapSL(msgbox->lpszText); - msgbox32.lpszCaption = MapSL(msgbox->lpszCaption); + msgbox32.hInstance = 0; msgbox32.dwStyle = msgbox->dwStyle; - msgbox32.lpszIcon = MapSL(msgbox->lpszIcon); + msgbox32.lpszIcon = NULL; msgbox32.dwContextHelpId = msgbox->dwContextHelpId; msgbox32.lpfnMsgBoxCallback = msgbox->lpfnMsgBoxCallback; msgbox32.dwLanguageId = msgbox->dwLanguageId; + + if (!HIWORD(msgbox->lpszCaption)) + { + LoadString16( msgbox->hInstance, LOWORD(msgbox->lpszCaption), caption, sizeof(caption) ); + msgbox32.lpszCaption = caption; + } + else msgbox32.lpszCaption = MapSL(msgbox->lpszCaption); + + if (!HIWORD(msgbox->lpszText)) + { + LoadString16( msgbox->hInstance, LOWORD(msgbox->lpszText), text, sizeof(text) ); + msgbox32.lpszText = text; + } + else msgbox32.lpszText = MapSL(msgbox->lpszText); + + if ((msgbox->dwStyle & MB_ICONMASK) == MB_USERICON) + { + FIXME( "user icon %s not supported\n", debugstr_a( MapSL(msgbox->lpszIcon) )); + msgbox32.dwStyle &= ~MB_USERICON; + } + return MessageBoxIndirectA( &msgbox32 ); }

1 0

Vincent Povirk : gdiplus: Add a trace for values of new ImageAttributes objects.
by Alexandre Julliard Dec. 28, 2009

Dec. 28, 2009

Module: wine Branch: master Commit: e68c866956735b0fbf082fe488dfe6568a0aefa9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e68c866956735b0fbf082fe48… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Dec 18 15:21:30 2009 -0600 gdiplus: Add a trace for values of new ImageAttributes objects. --- dlls/gdiplus/imageattributes.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/gdiplus/imageattributes.c b/dlls/gdiplus/imageattributes.c index 6bfd23e..f74b6db 100644 --- a/dlls/gdiplus/imageattributes.c +++ b/dlls/gdiplus/imageattributes.c @@ -47,14 +47,14 @@ GpStatus WINGDIPAPI GdipCloneImageAttributes(GDIPCONST GpImageAttributes *imagea GpStatus WINGDIPAPI GdipCreateImageAttributes(GpImageAttributes **imageattr) { - TRACE("(%p)\n", imageattr); - if(!imageattr) return InvalidParameter; *imageattr = GdipAlloc(sizeof(GpImageAttributes)); if(!*imageattr) return OutOfMemory; + TRACE("<-- %p\n", *imageattr); + return Ok; }

1 0