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

Vincent Povirk : gdiplus: Add tests for GdipPlayMetafileRecord.
by Alexandre Julliard July 8, 2011

July 8, 2011

Module: wine Branch: master Commit: 71325800dc0d26c70229858e70a11d689972869a URL: http://source.winehq.org/git/wine.git/?a=commit;h=71325800dc0d26c70229858e7… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Jul 7 13:31:55 2011 -0500 gdiplus: Add tests for GdipPlayMetafileRecord. --- dlls/gdiplus/tests/metafile.c | 144 ++++++++++++++++++++++++++++++++++++----- 1 files changed, 127 insertions(+), 17 deletions(-) diff --git a/dlls/gdiplus/tests/metafile.c b/dlls/gdiplus/tests/metafile.c index 66e504f..5323751 100644 --- a/dlls/gdiplus/tests/metafile.c +++ b/dlls/gdiplus/tests/metafile.c @@ -29,6 +29,7 @@ typedef struct emfplus_record { ULONG todo; ULONG record_type; + ULONG playback_todo; } emfplus_record; typedef struct emfplus_check_state @@ -36,6 +37,7 @@ typedef struct emfplus_check_state const char *desc; int count; const struct emfplus_record *expected; + GpMetafile *metafile; } emfplus_check_state; static void check_record(int count, const char *desc, const struct emfplus_record *expected, const struct emfplus_record *actual) @@ -178,6 +180,7 @@ static void check_metafile(GpMetafile *metafile, const emfplus_record *expected, state.desc = desc; state.count = 0; state.expected = expected; + state.metafile = metafile; hdc = CreateCompatibleDC(0); @@ -198,6 +201,51 @@ static void check_metafile(GpMetafile *metafile, const emfplus_record *expected, DeleteDC(hdc); } +static BOOL CALLBACK play_metafile_proc(EmfPlusRecordType record_type, unsigned int flags, + unsigned int dataSize, const unsigned char *pStr, void *userdata) +{ + emfplus_check_state *state = (emfplus_check_state*)userdata; + GpStatus stat; + + stat = GdipPlayMetafileRecord(state->metafile, record_type, flags, dataSize, pStr); + + if (state->expected[state->count].record_type) + { + if (state->expected[state->count].playback_todo) + todo_wine ok(stat == Ok, "%s: GdipPlayMetafileRecord failed with stat %i\n", state->desc, stat); + else + ok(stat == Ok, "%s: GdipPlayMetafileRecord failed with stat %i\n", state->desc, stat); + state->count++; + } + else + { + if (state->expected[state->count].playback_todo) + todo_wine ok(0, "%s: too many records\n", state->desc); + else + ok(0, "%s: too many records\n", state->desc); + + return FALSE; + } + + return TRUE; +} + +static void play_metafile(GpMetafile *metafile, GpGraphics *graphics, const emfplus_record *expected, + const char *desc, const GpPointF *dst_points, const GpRectF *src_rect, Unit src_unit) +{ + GpStatus stat; + emfplus_check_state state; + + state.desc = desc; + state.count = 0; + state.expected = expected; + state.metafile = metafile; + + stat = GdipEnumerateMetafileSrcRectDestPoints(graphics, metafile, dst_points, + 3, src_rect, src_unit, play_metafile_proc, &state, NULL); + expect(Ok, stat); +} + static const emfplus_record empty_records[] = { {0, EMR_HEADER}, {0, EmfPlusRecordTypeHeader}, @@ -276,16 +324,16 @@ static void test_empty(void) } static const emfplus_record getdc_records[] = { - {0, EMR_HEADER}, - {0, EmfPlusRecordTypeHeader}, - {0, EmfPlusRecordTypeGetDC}, - {0, EMR_CREATEBRUSHINDIRECT}, - {0, EMR_SELECTOBJECT}, - {0, EMR_RECTANGLE}, - {0, EMR_SELECTOBJECT}, - {0, EMR_DELETEOBJECT}, - {0, EmfPlusRecordTypeEndOfFile}, - {0, EMR_EOF}, + {0, EMR_HEADER, 1}, + {0, EmfPlusRecordTypeHeader, 1}, + {0, EmfPlusRecordTypeGetDC, 1}, + {0, EMR_CREATEBRUSHINDIRECT, 1}, + {0, EMR_SELECTOBJECT, 1}, + {0, EMR_RECTANGLE, 1}, + {0, EMR_SELECTOBJECT, 1}, + {0, EMR_DELETEOBJECT, 1}, + {0, EmfPlusRecordTypeEndOfFile, 1}, + {0, EMR_EOF, 1}, {0} }; @@ -299,8 +347,11 @@ static void test_getdc(void) BOOL ret; static const GpRectF frame = {0.0, 0.0, 100.0, 100.0}; static const GpPointF dst_points[3] = {{0.0,0.0},{100.0,0.0},{0.0,100.0}}; + static const GpPointF dst_points_half[3] = {{0.0,0.0},{50.0,0.0},{0.0,50.0}}; static const WCHAR description[] = {'w','i','n','e','t','e','s','t',0}; HBRUSH hbrush, holdbrush; + GpBitmap *bitmap; + ARGB color; hdc = CreateCompatibleDC(0); @@ -346,6 +397,41 @@ static void test_getdc(void) check_metafile(metafile, getdc_records, "getdc metafile", dst_points, &frame, UnitPixel); + stat = GdipCreateBitmapFromScan0(100, 100, 0, PixelFormat32bppARGB, NULL, &bitmap); + expect(Ok, stat); + + stat = GdipGetImageGraphicsContext((GpImage*)bitmap, &graphics); + expect(Ok, stat); + + play_metafile(metafile, graphics, getdc_records, "getdc playback", dst_points, &frame, UnitPixel); + + stat = GdipBitmapGetPixel(bitmap, 15, 15, &color); + expect(Ok, stat); + expect(0, color); + + stat = GdipBitmapGetPixel(bitmap, 50, 50, &color); + expect(Ok, stat); + todo_wine expect(0xff0000ff, color); + + stat = GdipBitmapSetPixel(bitmap, 50, 50, 0); + expect(Ok, stat); + + play_metafile(metafile, graphics, getdc_records, "getdc playback", dst_points_half, &frame, UnitPixel); + + stat = GdipBitmapGetPixel(bitmap, 15, 15, &color); + expect(Ok, stat); + todo_wine expect(0xff0000ff, color); + + stat = GdipBitmapGetPixel(bitmap, 50, 50, &color); + expect(Ok, stat); + expect(0, color); + + stat = GdipDeleteGraphics(graphics); + expect(Ok, stat); + + stat = GdipDisposeImage((GpImage*)bitmap); + expect(Ok, stat); + stat = GdipGetHemfFromMetafile(metafile, &hemf); expect(Ok, stat); @@ -359,13 +445,13 @@ static void test_getdc(void) } static const emfplus_record emfonly_records[] = { - {0, EMR_HEADER}, - {0, EMR_CREATEBRUSHINDIRECT}, - {0, EMR_SELECTOBJECT}, - {0, EMR_RECTANGLE}, - {0, EMR_SELECTOBJECT}, - {0, EMR_DELETEOBJECT}, - {0, EMR_EOF}, + {0, EMR_HEADER, 1}, + {0, EMR_CREATEBRUSHINDIRECT, 1}, + {0, EMR_SELECTOBJECT, 1}, + {0, EMR_RECTANGLE, 1}, + {0, EMR_SELECTOBJECT, 1}, + {0, EMR_DELETEOBJECT, 1}, + {0, EMR_EOF, 1}, {0} }; @@ -381,6 +467,8 @@ static void test_emfonly(void) static const GpPointF dst_points[3] = {{0.0,0.0},{100.0,0.0},{0.0,100.0}}; static const WCHAR description[] = {'w','i','n','e','t','e','s','t',0}; HBRUSH hbrush, holdbrush; + GpBitmap *bitmap; + ARGB color; hdc = CreateCompatibleDC(0); @@ -426,6 +514,28 @@ static void test_emfonly(void) check_metafile(metafile, emfonly_records, "emfonly metafile", dst_points, &frame, UnitPixel); + stat = GdipCreateBitmapFromScan0(100, 100, 0, PixelFormat32bppARGB, NULL, &bitmap); + expect(Ok, stat); + + stat = GdipGetImageGraphicsContext((GpImage*)bitmap, &graphics); + expect(Ok, stat); + + play_metafile(metafile, graphics, emfonly_records, "emfonly playback", dst_points, &frame, UnitPixel); + + stat = GdipBitmapGetPixel(bitmap, 15, 15, &color); + expect(Ok, stat); + expect(0, color); + + stat = GdipBitmapGetPixel(bitmap, 50, 50, &color); + expect(Ok, stat); + todo_wine expect(0xff0000ff, color); + + stat = GdipDeleteGraphics(graphics); + expect(Ok, stat); + + stat = GdipDisposeImage((GpImage*)bitmap); + expect(Ok, stat); + stat = GdipGetHemfFromMetafile(metafile, &hemf); expect(Ok, stat);

1 0

Vincent Povirk : gdiplus: Add test for recording emf-only metafiles.
by Alexandre Julliard July 8, 2011

July 8, 2011

Module: wine Branch: master Commit: 351a78bb4a85ceff41f4577e901dfe67faeb9669 URL: http://source.winehq.org/git/wine.git/?a=commit;h=351a78bb4a85ceff41f4577e9… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Jul 7 11:37:07 2011 -0500 gdiplus: Add test for recording emf-only metafiles. --- dlls/gdiplus/tests/metafile.c | 81 +++++++++++++++++++++++++++++++++++++++++ 1 files changed, 81 insertions(+), 0 deletions(-) diff --git a/dlls/gdiplus/tests/metafile.c b/dlls/gdiplus/tests/metafile.c index 1a0da15..66e504f 100644 --- a/dlls/gdiplus/tests/metafile.c +++ b/dlls/gdiplus/tests/metafile.c @@ -358,6 +358,86 @@ static void test_getdc(void) ok(ret != 0, "Failed to delete enhmetafile %p\n", hemf); } +static const emfplus_record emfonly_records[] = { + {0, EMR_HEADER}, + {0, EMR_CREATEBRUSHINDIRECT}, + {0, EMR_SELECTOBJECT}, + {0, EMR_RECTANGLE}, + {0, EMR_SELECTOBJECT}, + {0, EMR_DELETEOBJECT}, + {0, EMR_EOF}, + {0} +}; + +static void test_emfonly(void) +{ + GpStatus stat; + GpMetafile *metafile; + GpGraphics *graphics; + HDC hdc, metafile_dc; + HENHMETAFILE hemf; + BOOL ret; + static const GpRectF frame = {0.0, 0.0, 100.0, 100.0}; + static const GpPointF dst_points[3] = {{0.0,0.0},{100.0,0.0},{0.0,100.0}}; + static const WCHAR description[] = {'w','i','n','e','t','e','s','t',0}; + HBRUSH hbrush, holdbrush; + + hdc = CreateCompatibleDC(0); + + stat = GdipRecordMetafile(hdc, EmfTypeEmfOnly, &frame, MetafileFrameUnitPixel, description, &metafile); + expect(Ok, stat); + + DeleteDC(hdc); + + if (stat != Ok) + return; + + stat = GdipGetHemfFromMetafile(metafile, &hemf); + expect(InvalidParameter, stat); + + stat = GdipGetImageGraphicsContext((GpImage*)metafile, &graphics); + expect(Ok, stat); + + stat = GdipGetDC(graphics, &metafile_dc); + expect(Ok, stat); + + if (stat != Ok) + { + GdipDeleteGraphics(graphics); + GdipDisposeImage((GpImage*)metafile); + return; + } + + hbrush = CreateSolidBrush(0xff0000); + + holdbrush = SelectObject(metafile_dc, hbrush); + + Rectangle(metafile_dc, 25, 25, 75, 75); + + SelectObject(metafile_dc, holdbrush); + + DeleteObject(hbrush); + + stat = GdipReleaseDC(graphics, metafile_dc); + expect(Ok, stat); + + stat = GdipDeleteGraphics(graphics); + expect(Ok, stat); + + check_metafile(metafile, emfonly_records, "emfonly metafile", dst_points, &frame, UnitPixel); + + stat = GdipGetHemfFromMetafile(metafile, &hemf); + expect(Ok, stat); + + stat = GdipDisposeImage((GpImage*)metafile); + expect(Ok, stat); + + check_emfplus(hemf, emfonly_records, "emfonly emf"); + + ret = DeleteEnhMetaFile(hemf); + ok(ret != 0, "Failed to delete enhmetafile %p\n", hemf); +} + START_TEST(metafile) { struct GdiplusStartupInput gdiplusStartupInput; @@ -372,6 +452,7 @@ START_TEST(metafile) test_empty(); test_getdc(); + test_emfonly(); GdiplusShutdown(gdiplusToken); }

1 0

Henri Verbeet : wined3d: Make the surface parameter to d3dfmt_get_conv() const.
by Alexandre Julliard July 8, 2011

July 8, 2011

Module: wine Branch: master Commit: a26c5f9b1366696e5c27214a73d4d2b07534a715 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a26c5f9b1366696e5c27214a7… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Jul 7 21:01:39 2011 +0200 wined3d: Make the surface parameter to d3dfmt_get_conv() const. --- dlls/wined3d/surface.c | 4 ++-- dlls/wined3d/wined3d_private.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index f162764..ce669e1 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -4131,11 +4131,11 @@ static void flush_to_framebuffer_drawpixels(struct wined3d_surface *surface, context_release(context); } -HRESULT d3dfmt_get_conv(struct wined3d_surface *surface, BOOL need_alpha_ck, +HRESULT d3dfmt_get_conv(const struct wined3d_surface *surface, BOOL need_alpha_ck, BOOL use_texturing, struct wined3d_format *format, CONVERT_TYPES *convert) { BOOL colorkey_active = need_alpha_ck && (surface->CKeyFlags & WINEDDSD_CKSRCBLT); - struct wined3d_device *device = surface->resource.device; + const struct wined3d_device *device = surface->resource.device; const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; BOOL blit_supported = FALSE; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index a0941b6..374c631 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2159,7 +2159,7 @@ typedef enum { CONVERT_RGB32_888 } CONVERT_TYPES; -HRESULT d3dfmt_get_conv(struct wined3d_surface *surface, BOOL need_alpha_ck, BOOL use_texturing, +HRESULT d3dfmt_get_conv(const struct wined3d_surface *surface, BOOL need_alpha_ck, BOOL use_texturing, struct wined3d_format *format, CONVERT_TYPES *convert) DECLSPEC_HIDDEN; void d3dfmt_p8_init_palette(struct wined3d_surface *surface, BYTE table[256][4], BOOL colorkey) DECLSPEC_HIDDEN;

1 0

Henri Verbeet : wined3d: Make the surface parameter to surface_upload_data( ) const.
by Alexandre Julliard July 8, 2011

July 8, 2011

Module: wine Branch: master Commit: 0d18f5df0ee6715764eb864d7da338d99829b2b9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0d18f5df0ee6715764eb864d7… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Jul 7 21:01:37 2011 +0200 wined3d: Make the surface parameter to surface_upload_data() const. --- dlls/wined3d/surface.c | 2 +- dlls/wined3d/wined3d_private.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 1403e1e..f26cffe 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -2161,7 +2161,7 @@ static void surface_download_data(struct wined3d_surface *surface, const struct /* This call just uploads data, the caller is responsible for binding the * correct texture. */ /* Context activation is done by the caller. */ -void surface_upload_data(struct wined3d_surface *surface, const struct wined3d_gl_info *gl_info, +void surface_upload_data(const struct wined3d_surface *surface, const struct wined3d_gl_info *gl_info, const struct wined3d_format *format, const RECT *src_rect, UINT src_w, const POINT *dst_point, BOOL srgb, const struct wined3d_bo_address *data) { diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 7599e2d..a33d94f 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2087,7 +2087,7 @@ void surface_set_container(struct wined3d_surface *surface, void surface_set_texture_name(struct wined3d_surface *surface, GLuint name, BOOL srgb_name) DECLSPEC_HIDDEN; void surface_set_texture_target(struct wined3d_surface *surface, GLenum target) DECLSPEC_HIDDEN; void surface_translate_drawable_coords(const struct wined3d_surface *surface, HWND window, RECT *rect) DECLSPEC_HIDDEN; -void surface_upload_data(struct wined3d_surface *surface, const struct wined3d_gl_info *gl_info, +void surface_upload_data(const struct wined3d_surface *surface, const struct wined3d_gl_info *gl_info, const struct wined3d_format *format, const RECT *src_rect, UINT src_w, const POINT *dst_point, BOOL srgb, const struct wined3d_bo_address *data) DECLSPEC_HIDDEN;

1 0

Henri Verbeet : wined3d: Make the surface parameter to surface_translate_drawable_coords() const.
by Alexandre Julliard July 8, 2011

July 8, 2011

Module: wine Branch: master Commit: faebcc023604ec1a07055c6c1fb4a2e5cfabe9c9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=faebcc023604ec1a07055c6c1… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Jul 7 21:01:36 2011 +0200 wined3d: Make the surface parameter to surface_translate_drawable_coords() const. --- dlls/wined3d/surface.c | 2 +- dlls/wined3d/wined3d_private.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 65ab767..1403e1e 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -4975,7 +4975,7 @@ static void fb_copy_to_texture_hwstretch(struct wined3d_surface *dst_surface, st * drawable is limited to the window's client area. The sysmem and texture * copies do have the full screen size. Note that GL has a bottom-left * origin, while D3D has a top-left origin. */ -void surface_translate_drawable_coords(struct wined3d_surface *surface, HWND window, RECT *rect) +void surface_translate_drawable_coords(const struct wined3d_surface *surface, HWND window, RECT *rect) { UINT drawable_height; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 5e7930a..7599e2d 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2086,7 +2086,7 @@ void surface_set_container(struct wined3d_surface *surface, enum wined3d_container_type type, void *container) DECLSPEC_HIDDEN; void surface_set_texture_name(struct wined3d_surface *surface, GLuint name, BOOL srgb_name) DECLSPEC_HIDDEN; void surface_set_texture_target(struct wined3d_surface *surface, GLenum target) DECLSPEC_HIDDEN; -void surface_translate_drawable_coords(struct wined3d_surface *surface, HWND window, RECT *rect) DECLSPEC_HIDDEN; +void surface_translate_drawable_coords(const struct wined3d_surface *surface, HWND window, RECT *rect) DECLSPEC_HIDDEN; void surface_upload_data(struct wined3d_surface *surface, const struct wined3d_gl_info *gl_info, const struct wined3d_format *format, const RECT *src_rect, UINT src_w, const POINT *dst_point, BOOL srgb, const struct wined3d_bo_address *data) DECLSPEC_HIDDEN;

1 0

Vincent Povirk : gdiplus: Implement GetDC for metafiles.
by Alexandre Julliard July 8, 2011

July 8, 2011

Module: wine Branch: master Commit: 44c57121ea017adfc16def03ca8f3799bebc040f URL: http://source.winehq.org/git/wine.git/?a=commit;h=44c57121ea017adfc16def03c… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu Jul 7 11:30:06 2011 -0500 gdiplus: Implement GetDC for metafiles. --- dlls/gdiplus/gdiplus_private.h | 2 ++ dlls/gdiplus/graphics.c | 36 +++++++++++++++++++++++++----------- dlls/gdiplus/metafile.c | 30 ++++++++++++++++++++++++++++++ dlls/gdiplus/tests/metafile.c | 6 +++--- 4 files changed, 60 insertions(+), 14 deletions(-) diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index 9cae489..d6888d9 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -52,6 +52,8 @@ extern REAL convert_unit(REAL logpixels, GpUnit unit) DECLSPEC_HIDDEN; extern GpStatus graphics_from_image(GpImage *image, GpGraphics **graphics) DECLSPEC_HIDDEN; extern GpStatus METAFILE_GetGraphicsContext(GpMetafile* metafile, GpGraphics **result) DECLSPEC_HIDDEN; +extern GpStatus METAFILE_GetDC(GpMetafile* metafile, HDC *hdc) DECLSPEC_HIDDEN; +extern GpStatus METAFILE_ReleaseDC(GpMetafile* metafile, HDC hdc) DECLSPEC_HIDDEN; extern GpStatus METAFILE_GraphicsDeleted(GpMetafile* metafile) DECLSPEC_HIDDEN; extern void calc_curve_bezier(CONST GpPointF *pts, REAL tension, REAL *x1, diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index be25653..1e3f946 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -5311,6 +5311,8 @@ static const COLORREF DC_BACKGROUND_KEY = 0x0c0b0d; GpStatus WINGDIPAPI GdipGetDC(GpGraphics *graphics, HDC *hdc) { + GpStatus stat=Ok; + TRACE("(%p, %p)\n", graphics, hdc); if(!graphics || !hdc) @@ -5319,13 +5321,16 @@ GpStatus WINGDIPAPI GdipGetDC(GpGraphics *graphics, HDC *hdc) if(graphics->busy) return ObjectBusy; - if (!graphics->hdc || + if (graphics->image && graphics->image->type == ImageTypeMetafile) + { + stat = METAFILE_GetDC((GpMetafile*)graphics->image, hdc); + } + else if (!graphics->hdc || (graphics->image && graphics->image->type == ImageTypeBitmap && ((GpBitmap*)graphics->image)->format & PixelFormatAlpha)) { /* Create a fake HDC and fill it with a constant color. */ HDC temp_hdc; HBITMAP hbitmap; - GpStatus stat; GpRectF bounds; BITMAPINFOHEADER bmih; int i; @@ -5374,22 +5379,26 @@ GpStatus WINGDIPAPI GdipGetDC(GpGraphics *graphics, HDC *hdc) *hdc = graphics->hdc; } - graphics->busy = TRUE; + if (stat == Ok) + graphics->busy = TRUE; - return Ok; + return stat; } GpStatus WINGDIPAPI GdipReleaseDC(GpGraphics *graphics, HDC hdc) { - TRACE("(%p, %p)\n", graphics, hdc); + GpStatus stat=Ok; - if(!graphics || !hdc) - return InvalidParameter; + TRACE("(%p, %p)\n", graphics, hdc); - if((graphics->hdc != hdc && graphics->temp_hdc != hdc) || !(graphics->busy)) + if(!graphics || !hdc || !graphics->busy) return InvalidParameter; - if (graphics->temp_hdc == hdc) + if (graphics->image && graphics->image->type == ImageTypeMetafile) + { + stat = METAFILE_ReleaseDC((GpMetafile*)graphics->image, hdc); + } + else if (graphics->temp_hdc == hdc) { DWORD* pos; int i; @@ -5416,10 +5425,15 @@ GpStatus WINGDIPAPI GdipReleaseDC(GpGraphics *graphics, HDC hdc) graphics->temp_hdc = NULL; graphics->temp_hbitmap = NULL; } + else if (hdc != graphics->hdc) + { + stat = InvalidParameter; + } - graphics->busy = FALSE; + if (stat == Ok) + graphics->busy = FALSE; - return Ok; + return stat; } GpStatus WINGDIPAPI GdipGetClip(GpGraphics *graphics, GpRegion *region) diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c index 3d26b5c..3e79eb7 100644 --- a/dlls/gdiplus/metafile.c +++ b/dlls/gdiplus/metafile.c @@ -297,6 +297,36 @@ GpStatus METAFILE_GetGraphicsContext(GpMetafile* metafile, GpGraphics **result) return stat; } +GpStatus METAFILE_GetDC(GpMetafile* metafile, HDC *hdc) +{ + if (metafile->metafile_type == MetafileTypeEmfPlusOnly || metafile->metafile_type == MetafileTypeEmfPlusDual) + { + EmfPlusRecordHeader *record; + GpStatus stat; + + stat = METAFILE_AllocateRecord(metafile, sizeof(EmfPlusRecordHeader), (void**)&record); + if (stat != Ok) + return stat; + + record->Type = EmfPlusRecordTypeGetDC; + record->Flags = 0; + + METAFILE_WriteRecords(metafile); + } + + *hdc = metafile->record_dc; + + return Ok; +} + +GpStatus METAFILE_ReleaseDC(GpMetafile* metafile, HDC hdc) +{ + if (hdc != metafile->record_dc) + return InvalidParameter; + + return Ok; +} + GpStatus METAFILE_GraphicsDeleted(GpMetafile* metafile) { GpStatus stat; diff --git a/dlls/gdiplus/tests/metafile.c b/dlls/gdiplus/tests/metafile.c index c85de00..1a0da15 100644 --- a/dlls/gdiplus/tests/metafile.c +++ b/dlls/gdiplus/tests/metafile.c @@ -278,9 +278,9 @@ static void test_empty(void) static const emfplus_record getdc_records[] = { {0, EMR_HEADER}, {0, EmfPlusRecordTypeHeader}, - {1, EmfPlusRecordTypeGetDC}, - {1, EMR_CREATEBRUSHINDIRECT}, - {1, EMR_SELECTOBJECT}, + {0, EmfPlusRecordTypeGetDC}, + {0, EMR_CREATEBRUSHINDIRECT}, + {0, EMR_SELECTOBJECT}, {0, EMR_RECTANGLE}, {0, EMR_SELECTOBJECT}, {0, EMR_DELETEOBJECT},

1 0

Alexandre Julliard : usp10: Make the various lookup tables more compact.
by Alexandre Julliard July 7, 2011

July 7, 2011

Module: wine Branch: master Commit: b99b9565e808652e8f50204aafdff587f6948a8f URL: http://source.winehq.org/git/wine.git/?a=commit;h=b99b9565e808652e8f50204aa… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jul 7 17:30:35 2011 +0200 usp10: Make the various lookup tables more compact. --- dlls/usp10/breaking.c | 2 +- dlls/usp10/indicsyllable.c | 495 ++---------- dlls/usp10/linebreak.c | 1825 +++++-------------------------------------- dlls/usp10/shape.c | 6 +- dlls/usp10/shaping.c | 1113 +++----------------------- dlls/usp10/usp10_internal.h | 5 + tools/make_unicode | 46 +- 7 files changed, 438 insertions(+), 3054 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b99b9565e808652e8f502…

1 0

Alistair Leslie-Hughes : oleaut32: Implement ICreateTypeInfo2 SetHelpStringContext.
by Alexandre Julliard July 7, 2011

July 7, 2011

Module: wine Branch: master Commit: 1a74194f9ed29ea2fbaa8b2df3632c0c69e8c016 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1a74194f9ed29ea2fbaa8b2df… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Mon Jun 27 22:26:30 2011 +1000 oleaut32: Implement ICreateTypeInfo2 SetHelpStringContext. --- dlls/oleaut32/typelib2.c | 11 ++++++++--- 1 files changed, 8 insertions(+), 3 deletions(-) diff --git a/dlls/oleaut32/typelib2.c b/dlls/oleaut32/typelib2.c index 3cbd4b5..d319ede 100644 --- a/dlls/oleaut32/typelib2.c +++ b/dlls/oleaut32/typelib2.c @@ -3070,10 +3070,15 @@ static HRESULT WINAPI ICreateTypeInfo2_fnSetImplTypeCustData( */ static HRESULT WINAPI ICreateTypeInfo2_fnSetHelpStringContext( ICreateTypeInfo2* iface, /* [I] The typeinfo on which to set the help string context. */ - ULONG dwHelpStringContext) /* [I] The help string context. */ + ULONG helpcontext) /* [I] The help string context. */ { - FIXME("(%p,%d), stub!\n", iface, dwHelpStringContext); - return E_OUTOFMEMORY; + ICreateTypeInfo2Impl *This = impl_from_ICreateTypeInfo2(iface); + + TRACE("(%p, %d)\n", iface, helpcontext); + + This->typelib->typelib_header.helpcontext = helpcontext; + + return S_OK; } /******************************************************************************

1 0

Alistair Leslie-Hughes : oledb32: Add DBTYPE_UI2 support to DataConvert.
by Alexandre Julliard July 7, 2011

July 7, 2011

Module: wine Branch: master Commit: d911374b9c009a389c19d1d618c21db6e3a99abc URL: http://source.winehq.org/git/wine.git/?a=commit;h=d911374b9c009a389c19d1d61… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Tue Jun 28 20:41:04 2011 +1000 oledb32: Add DBTYPE_UI2 support to DataConvert. --- dlls/oledb32/convert.c | 25 ++++++++++++++++++++++++ dlls/oledb32/tests/convert.c | 43 ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 68 insertions(+), 0 deletions(-) diff --git a/dlls/oledb32/convert.c b/dlls/oledb32/convert.c index 3bb663f..1691792 100644 --- a/dlls/oledb32/convert.c +++ b/dlls/oledb32/convert.c @@ -533,6 +533,31 @@ static HRESULT WINAPI convert_DataConvert(IDataConvert* iface, } break; } + case DBTYPE_UI2: + { + WORD *d = dst; + switch(src_type) + { + case DBTYPE_EMPTY: *d = 0; hr = S_OK; break; + case DBTYPE_I2: hr = VarUI2FromI2(*(signed short*)src, d); break; + case DBTYPE_I4: hr = VarUI2FromI4(*(signed int*)src, d); break; + case DBTYPE_R4: hr = VarUI2FromR4(*(FLOAT*)src, d); break; + case DBTYPE_R8: hr = VarUI2FromR8(*(double*)src, d); break; + case DBTYPE_CY: hr = VarUI2FromCy(*(CY*)src, d); break; + case DBTYPE_DATE: hr = VarUI2FromDate(*(DATE*)src, d); break; + case DBTYPE_BSTR: hr = VarUI2FromStr(*(WCHAR**)src, LOCALE_USER_DEFAULT, 0, d); break; + case DBTYPE_BOOL: hr = VarUI2FromBool(*(VARIANT_BOOL*)src, d); break; + case DBTYPE_DECIMAL: hr = VarUI2FromDec((DECIMAL*)src, d); break; + case DBTYPE_I1: hr = VarUI2FromI1(*(signed char*)src, d); break; + case DBTYPE_UI1: hr = VarUI2FromUI1(*(BYTE*)src, d); break; + case DBTYPE_UI2: *d = *(WORD*)src; hr = S_OK; break; + case DBTYPE_UI4: hr = VarUI2FromUI4(*(DWORD*)src, d); break; + case DBTYPE_I8: hr = VarUI2FromI8(*(LONGLONG*)src, d); break; + case DBTYPE_UI8: hr = VarUI2FromUI8(*(ULONGLONG*)src, d); break; + default: FIXME("Unimplemented conversion %04x -> UI2\n", src_type); return E_NOTIMPL; + } + break; + } case DBTYPE_UI4: { diff --git a/dlls/oledb32/tests/convert.c b/dlls/oledb32/tests/convert.c index a5ccdb9..1a1ea9a 100644 --- a/dlls/oledb32/tests/convert.c +++ b/dlls/oledb32/tests/convert.c @@ -610,6 +610,49 @@ todo_wine ok(dst_len == sizeof(dst), "got %ld\n", dst_len); ok(dst == 10, "got %08x\n", dst); + /* */ + dst_len = dst = 0x1234; + *(WORD*)src = 0x4321; + hr = IDataConvert_DataConvert(convert, DBTYPE_UI2, DBTYPE_UI2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); + ok(hr == S_OK, "got %08x\n", hr); + ok(dst_status == DBSTATUS_S_OK, "got %08x\n", dst_status); + ok(dst_len == sizeof(dst), "got %ld\n", dst_len); + ok(dst == 0x4321, "got %08x\n", dst); + + dst_len = dst = 0x1234; + *(DWORD*)src = 0xabcd1234; + hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_UI2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); +todo_wine + ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); + ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); + ok(dst_len == sizeof(dst), "got %ld\n", dst_len); + ok(dst == 0x1234, "got %08x\n", dst); + + dst_len = dst = 0x1234; + *(DWORD*)src = 0x1234abcd; + hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_UI2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); +todo_wine + ok(hr == DB_E_DATAOVERFLOW, "got %08x\n", hr); + ok(dst_status == DBSTATUS_E_DATAOVERFLOW, "got %08x\n", dst_status); + ok(dst_len == sizeof(dst), "got %ld\n", dst_len); + ok(dst == 0x1234, "got %08x\n", dst); + + dst_len = dst = 0x1234; + *(DWORD*)src = 0x4321; + hr = IDataConvert_DataConvert(convert, DBTYPE_UI4, DBTYPE_UI2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, 0); + ok(hr == S_OK, "got %08x\n", hr); + ok(dst_status == DBSTATUS_S_OK, "got %08x\n", dst_status); + ok(dst_len == sizeof(dst), "got %ld\n", dst_len); + ok(dst == 0x4321, "got %08x\n", dst); + + dst_len = dst = 0x1234; + memcpy(src, ten, sizeof(ten)); + hr = IDataConvert_DataConvert(convert, DBTYPE_WSTR, DBTYPE_UI2, 0, &dst_len, src, &dst, sizeof(dst), 0, &dst_status, 0, 0, DBDATACONVERT_LENGTHFROMNTS); + ok(hr == S_OK, "got %08x\n", hr); + ok(dst_status == DBSTATUS_S_OK, "got %08x\n", dst_status); + ok(dst_len == sizeof(dst), "got %ld\n", dst_len); + ok(dst == 10, "got %08x\n", dst); + IDataConvert_Release(convert); }

1 0

Jacek Caban : widl: Use proper calling convention attributes.
by Alexandre Julliard July 7, 2011

July 7, 2011

Module: wine Branch: master Commit: f8c517978bf5c5f164ff91cb1b277e1e7433e422 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f8c517978bf5c5f164ff91cb1… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Jul 7 15:09:15 2011 +0200 widl: Use proper calling convention attributes. --- tools/widl/parser.y | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index f034074..d4852d1 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -588,10 +588,10 @@ uuid_string: $$ = parse_uuid($1); } ; -callconv: tCDECL { $$ = $<str>1; } - | tFASTCALL { $$ = $<str>1; } - | tPASCAL { $$ = $<str>1; } - | tSTDCALL { $$ = $<str>1; } +callconv: tCDECL { $$ = xstrdup("__cdecl"); } + | tFASTCALL { $$ = xstrdup("__fastcall"); } + | tPASCAL { $$ = xstrdup("__pascal"); } + | tSTDCALL { $$ = xstrdup("__stdcall"); } ; cases: { $$ = NULL; }

1 0