October 2016 - Wine-Commits

Vincent Povirk : gdiplus/tests: Add test for gdi32 SetWorldTransform record.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: 094516d47a0e6f2776787c05b5b73189f5103e95 URL: http://source.winehq.org/git/wine.git/?a=commit;h=094516d47a0e6f2776787c05b… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Oct 7 16:03:43 2016 -0500 gdiplus/tests: Add test for gdi32 SetWorldTransform record. Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/tests/metafile.c | 136 ++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 132 insertions(+), 4 deletions(-) diff --git a/dlls/gdiplus/tests/metafile.c b/dlls/gdiplus/tests/metafile.c index 7251ff3..83054ca 100644 --- a/dlls/gdiplus/tests/metafile.c +++ b/dlls/gdiplus/tests/metafile.c @@ -36,6 +36,8 @@ typedef struct emfplus_record BOOL todo; ULONG record_type; BOOL playback_todo; + void (*playback_fn)(GpMetafile* metafile, EmfPlusRecordType record_type, + unsigned int flags, unsigned int dataSize, const unsigned char *pStr); } emfplus_record; typedef struct emfplus_check_state @@ -205,12 +207,19 @@ static BOOL CALLBACK play_metafile_proc(EmfPlusRecordType record_type, unsigned 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) { - todo_wine_if (state->expected[state->count].playback_todo) - ok(stat == Ok, "%s.%i: GdipPlayMetafileRecord failed with stat %i\n", state->desc, state->count, stat); + BOOL match = (state->expected[state->count].record_type == record_type); + + if (match && state->expected[state->count].playback_fn) + state->expected[state->count].playback_fn(state->metafile, record_type, flags, dataSize, pStr); + else + { + stat = GdipPlayMetafileRecord(state->metafile, record_type, flags, dataSize, pStr); + todo_wine_if (state->expected[state->count].playback_todo) + ok(stat == Ok, "%s.%i: GdipPlayMetafileRecord failed with stat %i\n", state->desc, state->count, stat); + } + todo_wine_if (state->expected[state->count].todo) ok(state->expected[state->count].record_type == record_type, "%s.%i: expected record type 0x%x, got 0x%x\n", state->desc, state->count, @@ -2145,6 +2154,124 @@ static void test_clipping(void) expect(Ok, stat); } +static void test_gditransform_cb(GpMetafile* metafile, EmfPlusRecordType record_type, + unsigned int flags, unsigned int dataSize, const unsigned char *pStr) +{ + static const XFORM xform = {0.5, 0, 0, 0.5, 0, 0}; + static const RECTL rectangle = {0,0,100,100}; + GpStatus stat; + + stat = GdipPlayMetafileRecord(metafile, EMR_SETWORLDTRANSFORM, 0, sizeof(xform), (void*)&xform); + expect(Ok, stat); + + stat = GdipPlayMetafileRecord(metafile, EMR_RECTANGLE, 0, sizeof(rectangle), (void*)&rectangle); + expect(Ok, stat); +} + +static const emfplus_record gditransform_records[] = { + {0, EMR_HEADER}, + {0, EMR_CREATEBRUSHINDIRECT}, + {0, EMR_SELECTOBJECT}, + {0, EMR_GDICOMMENT, 0, test_gditransform_cb}, + {0, EMR_SELECTOBJECT}, + {0, EMR_DELETEOBJECT}, + {0, EMR_EOF}, + {0} +}; + +static void test_gditransform(void) +{ + GpStatus stat; + GpMetafile *metafile; + GpGraphics *graphics; + HDC hdc, metafile_dc; + HENHMETAFILE hemf; + MetafileHeader header; + static const GpRectF frame = {0.0, 0.0, 100.0, 100.0}; + static const GpPointF dst_points[3] = {{0.0,0.0},{40.0,0.0},{0.0,40.0}}; + static const WCHAR description[] = {'w','i','n','e','t','e','s','t',0}; + HBRUSH hbrush, holdbrush; + GpBitmap *bitmap; + ARGB color; + + 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); + + memset(&header, 0xaa, sizeof(header)); + stat = GdipGetMetafileHeaderFromMetafile(metafile, &header); + expect(Ok, stat); + expect(MetafileTypeEmf, header.Type); + ok(header.Version == 0xdbc01001 || header.Version == 0xdbc01002, "Unexpected version %x\n", header.Version); + + 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(0xff); + + holdbrush = SelectObject(metafile_dc, hbrush); + + GdiComment(metafile_dc, 8, (const BYTE*)"winetest"); + + SelectObject(metafile_dc, holdbrush); + + DeleteObject(hbrush); + + stat = GdipReleaseDC(graphics, metafile_dc); + expect(Ok, stat); + + stat = GdipDeleteGraphics(graphics); + expect(Ok, stat); + + check_metafile(metafile, gditransform_records, "gditransform metafile", dst_points, &frame, UnitPixel); + + sync_metafile(&metafile, "gditransform.emf"); + + stat = GdipCreateBitmapFromScan0(100, 100, 0, PixelFormat32bppARGB, NULL, &bitmap); + expect(Ok, stat); + + stat = GdipGetImageGraphicsContext((GpImage*)bitmap, &graphics); + expect(Ok, stat); + + play_metafile(metafile, graphics, gditransform_records, "gditransform playback", dst_points, &frame, UnitPixel); + + stat = GdipBitmapGetPixel(bitmap, 10, 10, &color); + expect(Ok, stat); + expect(0xffff0000, color); + + stat = GdipBitmapGetPixel(bitmap, 30, 30, &color); + expect(Ok, stat); + todo_wine expect(0x00000000, color); + + stat = GdipDeleteGraphics(graphics); + expect(Ok, stat); + + stat = GdipDisposeImage((GpImage*)bitmap); + expect(Ok, stat); + + stat = GdipDisposeImage((GpImage*)metafile); + expect(Ok, stat); +} + START_TEST(metafile) { struct GdiplusStartupInput gdiplusStartupInput; @@ -2181,6 +2308,7 @@ START_TEST(metafile) test_frameunit(); test_containers(); test_clipping(); + test_gditransform(); GdiplusShutdown(gdiplusToken); }

1 0

Marcus Meissner : gdiplus: Assign error return (Coverity).
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: 4c3f494aa099c7213dcbaa829dfafb4ad6f3d521 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4c3f494aa099c7213dcbaa829… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Sun Oct 9 09:24:52 2016 +0200 gdiplus: Assign error return (Coverity). Signed-off-by: Marcus Meissner <marcus(a)jet.franken.de> Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/metafile.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c index 9de584c..f9ed14c 100644 --- a/dlls/gdiplus/metafile.c +++ b/dlls/gdiplus/metafile.c @@ -1361,7 +1361,7 @@ GpStatus WINGDIPAPI GdipPlayMetafileRecord(GDIPCONST GpMetafile *metafile, return stat; } - GdipBeginContainer2(metafile->playback_graphics, &cont->state); + stat = GdipBeginContainer2(metafile->playback_graphics, &cont->state); if (stat != Ok) {

1 0

Akihiro Sagawa : webservices: Add support for decoding decimal numeric character references.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: dd4c126e956281269dca5a9025159dd1c802ed27 URL: http://source.winehq.org/git/wine.git/?a=commit;h=dd4c126e956281269dca5a902… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Mon Oct 10 22:53:14 2016 +0900 webservices: Add support for decoding decimal numeric character references. Signed-off-by: Akihiro Sagawa <sagawa.aki(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/webservices/reader.c | 50 +++++++++++++++++++++++++++++------------ dlls/webservices/tests/reader.c | 7 ++++++ 2 files changed, 43 insertions(+), 14 deletions(-) diff --git a/dlls/webservices/reader.c b/dlls/webservices/reader.c index 655288a..de92603 100644 --- a/dlls/webservices/reader.c +++ b/dlls/webservices/reader.c @@ -1249,23 +1249,45 @@ static HRESULT decode_text( const unsigned char *str, ULONG len, unsigned char * int len_utf8, cp = 0; p++; len--; - if (!len || *p != 'x') return WS_E_INVALID_FORMAT; - p++; len--; - - start = len; - while (len && isxdigit( *p )) { p++; len--; }; if (!len) return WS_E_INVALID_FORMAT; - - p -= nb_digits = start - len; - if (!nb_digits || nb_digits > 5 || p[nb_digits] != ';') return WS_E_INVALID_FORMAT; - for (i = 0; i < nb_digits; i++) + else if (*p == 'x') + { + p++; len--; + + start = len; + while (len && isxdigit( *p )) { p++; len--; }; + if (!len) return WS_E_INVALID_FORMAT; + + p -= nb_digits = start - len; + if (!nb_digits || nb_digits > 5 || p[nb_digits] != ';') return WS_E_INVALID_FORMAT; + for (i = 0; i < nb_digits; i++) + { + cp *= 16; + if (*p >= '0' && *p <= '9') cp += *p - '0'; + else if (*p >= 'a' && *p <= 'f') cp += *p - 'a' + 10; + else cp += *p - 'A' + 10; + p++; + } + } + else if (isdigit( *p )) { - cp *= 16; - if (*p >= '0' && *p <= '9') cp += *p - '0'; - else if (*p >= 'a' && *p <= 'f') cp += *p - 'a' + 10; - else cp += *p - 'A' + 10; - p++; + while (len && *p == '0') { p++; len--; }; + if (!len) return WS_E_INVALID_FORMAT; + + start = len; + while (len && isdigit( *p )) { p++; len--; }; + if (!len) return WS_E_INVALID_FORMAT; + + p -= nb_digits = start - len; + if (!nb_digits || nb_digits > 7 || p[nb_digits] != ';') return WS_E_INVALID_FORMAT; + for (i = 0; i < nb_digits; i++) + { + cp *= 10; + cp += *p - '0'; + p++; + } } + else return WS_E_INVALID_FORMAT; p++; len--; if ((len_utf8 = codepoint_to_utf8( cp, q )) < 0) return WS_E_INVALID_FORMAT; *ret_len += len_utf8; diff --git a/dlls/webservices/tests/reader.c b/dlls/webservices/tests/reader.c index 76de14a..f0fe729 100644 --- a/dlls/webservices/tests/reader.c +++ b/dlls/webservices/tests/reader.c @@ -3611,10 +3611,14 @@ static void test_entities(void) static const char str26[] = "<t></t>"; static const char str27[] = "<t>&LT;</t>"; static const char str28[] = "<t></t>"; + static const char str29[] = "<t></t>"; + static const char str30[] = "<t>A</t>"; + static const char str31[] = "<t>ｱ</t>"; static const char res4[] = {0xea, 0xaa, 0xaa, 0x00}; static const char res5[] = {0xf2, 0xaa, 0xaa, 0xaa, 0x00}; static const char res21[] = {0xed, 0x9f, 0xbf, 0x00}; static const char res24[] = {0xee, 0x80, 0x80, 0x00}; + static const char res31[] = {0xef, 0xbd, 0xb1, 0x00}; static const struct { const char *str; @@ -3651,6 +3655,9 @@ static void test_entities(void) { str26, WS_E_INVALID_FORMAT }, { str27, WS_E_INVALID_FORMAT }, { str28, WS_E_INVALID_FORMAT }, + { str29, WS_E_INVALID_FORMAT }, + { str30, S_OK, "A" }, + { str31, S_OK, res31 }, }; HRESULT hr; WS_XML_READER *reader;

1 0

Akihiro Sagawa : webservices: Avoid decoding a nul character reference.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: 064edd2d8b3f02b57fd881d40e89cf7ae9c1adba URL: http://source.winehq.org/git/wine.git/?a=commit;h=064edd2d8b3f02b57fd881d40… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Mon Oct 10 22:53:12 2016 +0900 webservices: Avoid decoding a nul character reference. Signed-off-by: Akihiro Sagawa <sagawa.aki(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/webservices/reader.c | 2 ++ dlls/webservices/tests/reader.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/dlls/webservices/reader.c b/dlls/webservices/reader.c index 8b9faf2..655288a 100644 --- a/dlls/webservices/reader.c +++ b/dlls/webservices/reader.c @@ -1166,6 +1166,8 @@ static HRESULT parse_name( const unsigned char *str, unsigned int len, static int codepoint_to_utf8( int cp, unsigned char *dst ) { + if (!cp) + return -1; if (cp < 0x80) { *dst = cp; diff --git a/dlls/webservices/tests/reader.c b/dlls/webservices/tests/reader.c index 06c18b2..76de14a 100644 --- a/dlls/webservices/tests/reader.c +++ b/dlls/webservices/tests/reader.c @@ -3610,6 +3610,7 @@ static void test_entities(void) static const char str25[] = "<t></t>"; static const char str26[] = "<t></t>"; static const char str27[] = "<t>&LT;</t>"; + static const char str28[] = "<t></t>"; static const char res4[] = {0xea, 0xaa, 0xaa, 0x00}; static const char res5[] = {0xf2, 0xaa, 0xaa, 0xaa, 0x00}; static const char res21[] = {0xed, 0x9f, 0xbf, 0x00}; @@ -3649,6 +3650,7 @@ static void test_entities(void) { str25, WS_E_INVALID_FORMAT }, { str26, WS_E_INVALID_FORMAT }, { str27, WS_E_INVALID_FORMAT }, + { str28, WS_E_INVALID_FORMAT }, }; HRESULT hr; WS_XML_READER *reader;

1 0

Huw Davies : riched20: Add support for letter labelled lists.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: 9265a9151375565e10a9e2cc2aaf10f2ab6c9896 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9265a9151375565e10a9e2cc2… Author: Huw Davies <huw(a)codeweavers.com> Date: Mon Oct 10 13:13:03 2016 +0100 riched20: Add support for letter labelled lists. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/riched20/para.c | 35 ++++++++++++++++++++++++++++++++++- 1 file changed, 34 insertions(+), 1 deletion(-) diff --git a/dlls/riched20/para.c b/dlls/riched20/para.c index ffd0a1d..b762df7 100644 --- a/dlls/riched20/para.c +++ b/dlls/riched20/para.c @@ -155,13 +155,46 @@ static ME_String *para_num_get_str( ME_Paragraph *para, WORD num ) ME_String *str = ME_MakeStringEmpty( 5 + 2 ); WCHAR *p = str->szData; static const WCHAR fmtW[] = {'%', 'd', 0}; + static const WORD letter_base[] = { 1, 26, 26 * 26, 26 * 26 * 26 }; + int i, len; + WORD letter, total, char_offset = 0; if (!str) return NULL; if ((para->fmt.wNumberingStyle & 0xf00) == PFNS_PARENS) *p++ = '('; - p += sprintfW( p, fmtW, num ); + switch (para->fmt.wNumbering) + { + case PFN_ARABIC: + default: + p += sprintfW( p, fmtW, num ); + break; + + case PFN_LCLETTER: + char_offset = 'a' - 'A'; + /* fall through */ + case PFN_UCLETTER: + if (!num) num = 1; + + /* This is not base-26 (or 27) as zeros don't count unless they are leading zeros. + It's simplest to start with the least significant letter, so first calculate how many letters are needed. */ + for (i = 0, total = 0; i < sizeof(letter_base) / sizeof(letter_base[0]); i++) + { + total += letter_base[i]; + if (num < total) break; + } + len = i; + for (i = 0; i < len; i++) + { + num -= letter_base[i]; + letter = (num / letter_base[i]) % 26; + p[len - i - 1] = letter + 'A' + char_offset; + } + p += len; + *p = 0; + break; + } switch (para->fmt.wNumberingStyle & 0xf00) {

1 0

Józef Kucia : wined3d: Call glFlush() through gl_info.gl_ops.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: 83544c4cc243b78771ba4d868de9c693d694e4b6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=83544c4cc243b78771ba4d868… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Mon Oct 10 10:22:36 2016 +0200 wined3d: Call glFlush() through gl_info.gl_ops. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/context.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 98516f8..25a3ac6 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -1296,8 +1296,9 @@ BOOL context_set_current(struct wined3d_context *ctx) { if (wglGetCurrentContext()) { + const struct wined3d_gl_info *gl_info = old->gl_info; TRACE("Flushing context %p before switching to %p.\n", old, ctx); - glFlush(); + gl_info->gl_ops.gl.p_glFlush(); } old->current = 0; } @@ -1316,7 +1317,7 @@ BOOL context_set_current(struct wined3d_context *ctx) return FALSE; ctx->current = 1; } - else if(wglGetCurrentContext()) + else if (wglGetCurrentContext()) { TRACE("Clearing current D3D context.\n"); if (!wglMakeCurrent(NULL, NULL))

1 0

Józef Kucia : wined3d: Call glGetIntegerv() through gl_info.gl_ops.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: 055d3ff571c3fcb65266583a22a32772b1582395 URL: http://source.winehq.org/git/wine.git/?a=commit;h=055d3ff571c3fcb65266583a2… Author: Józef Kucia <jkucia(a)codeweavers.com> Date: Mon Oct 10 10:22:35 2016 +0200 wined3d: Call glGetIntegerv() through gl_info.gl_ops. Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/directx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 98b05c1..6cd8535 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -2546,7 +2546,7 @@ static void enumerate_gl_extensions(struct wined3d_gl_info *gl_info, unsigned int i, j; GLint extensions_count; - glGetIntegerv(GL_NUM_EXTENSIONS, &extensions_count); + gl_info->gl_ops.gl.p_glGetIntegerv(GL_NUM_EXTENSIONS, &extensions_count); for (i = 0; i < extensions_count; ++i) { gl_extension_name = (const char *)GL_EXTCALL(glGetStringi(GL_EXTENSIONS, i));

1 0

Andrey Gusev : wined3d: Add missing '\n' to FIXME() messages.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: 9fb120563564c3871c0451c7b0cd6253f923de18 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9fb120563564c3871c0451c7b… Author: Andrey Gusev <andrey.goosev(a)gmail.com> Date: Fri Oct 7 17:01:22 2016 +0300 wined3d: Add missing '\n' to FIXME() messages. Signed-off-by: Andrey Gusev <andrey.goosev(a)gmail.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/shader.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/shader.c b/dlls/wined3d/shader.c index 4455416..61509d9 100644 --- a/dlls/wined3d/shader.c +++ b/dlls/wined3d/shader.c @@ -1663,7 +1663,7 @@ static void shader_dump_decl_usage(struct wined3d_string_buffer *buffer, default: shader_addline(buffer, "<unknown_semantic(%#x)>", semantic->usage); - FIXME("unknown_semantics(0x%08x)", semantic->usage); + FIXME("Unrecognised semantic usage %#x.\n", semantic->usage); } } } @@ -2087,7 +2087,7 @@ static void shader_dump_ins_modifiers(struct wined3d_string_buffer *buffer, if (mmask & WINED3DSPDM_MSAMPCENTROID) shader_addline(buffer, "_centroid"); mmask &= ~(WINED3DSPDM_SATURATE | WINED3DSPDM_PARTIALPRECISION | WINED3DSPDM_MSAMPCENTROID); - if (mmask) FIXME("_unrecognized_modifier(%#x)", mmask); + if (mmask) FIXME("Unrecognised modifier %#x.\n", mmask); } static void shader_dump_primitive_type(struct wined3d_string_buffer *buffer,

1 0

Vincent Povirk : gdiplus: Implement EMR_SETWORLDTRANSFORM playback.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: 28cfa306b467abb026da1f9442bc247c4c9dc790 URL: http://source.winehq.org/git/wine.git/?a=commit;h=28cfa306b467abb026da1f944… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Oct 7 16:03:44 2016 -0500 gdiplus: Implement EMR_SETWORLDTRANSFORM playback. Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/gdiplus_private.h | 1 + dlls/gdiplus/metafile.c | 50 +++++++++++++++++++++++++++++------------- dlls/gdiplus/tests/metafile.c | 2 +- 3 files changed, 37 insertions(+), 16 deletions(-) diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index 86a29a0..340e9bd 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -374,6 +374,7 @@ struct GpMetafile{ GpRectF src_rect; HANDLETABLE *handle_table; int handle_count; + XFORM gdiworldtransform; GpMatrix *world_transform; GpUnit page_unit; REAL page_scale; diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c index 409a6c4..32cd04e 100644 --- a/dlls/gdiplus/metafile.c +++ b/dlls/gdiplus/metafile.c @@ -1040,6 +1040,30 @@ GpStatus WINGDIPAPI GdipGetHemfFromMetafile(GpMetafile *metafile, HENHMETAFILE * return Ok; } +static GpStatus METAFILE_PlaybackUpdateGdiTransform(GpMetafile *metafile) +{ + XFORM combined, final; + const GpRectF *rect; + const GpPointF *pt; + + /* This transforms metafile device space to output points. */ + rect = &metafile->src_rect; + pt = metafile->playback_points; + final.eM11 = (pt[1].X - pt[0].X) / rect->Width; + final.eM21 = (pt[2].X - pt[0].X) / rect->Height; + final.eDx = pt[0].X - final.eM11 * rect->X - final.eM21 * rect->Y; + final.eM12 = (pt[1].Y - pt[0].Y) / rect->Width; + final.eM22 = (pt[2].Y - pt[0].Y) / rect->Height; + final.eDy = pt[0].Y - final.eM12 * rect->X - final.eM22 * rect->Y; + + CombineTransform(&combined, &metafile->gdiworldtransform, &final); + + SetGraphicsMode(metafile->playback_dc, GM_ADVANCED); + SetWorldTransform(metafile->playback_dc, &combined); + + return Ok; +} + static GpStatus METAFILE_PlaybackGetDC(GpMetafile *metafile) { GpStatus stat = Ok; @@ -1048,20 +1072,10 @@ static GpStatus METAFILE_PlaybackGetDC(GpMetafile *metafile) if (stat == Ok) { - /* The result of GdipGetDC always expects device co-ordinates, but the - * device co-ordinates of the source metafile do not correspond to - * device co-ordinates of the destination. Therefore, we set up the DC - * so that the metafile's bounds map to the destination points where we - * are drawing this metafile. */ - SetMapMode(metafile->playback_dc, MM_ANISOTROPIC); - - SetWindowOrgEx(metafile->playback_dc, metafile->bounds.X, metafile->bounds.Y, NULL); - SetWindowExtEx(metafile->playback_dc, metafile->bounds.Width, metafile->bounds.Height, NULL); - - SetViewportOrgEx(metafile->playback_dc, metafile->playback_points[0].X, metafile->playback_points[0].Y, NULL); - SetViewportExtEx(metafile->playback_dc, - metafile->playback_points[1].X - metafile->playback_points[0].X, - metafile->playback_points[2].Y - metafile->playback_points[0].Y, NULL); + static const XFORM identity = {1, 0, 0, 1, 0, 0}; + + metafile->gdiworldtransform = identity; + METAFILE_PlaybackUpdateGdiTransform(metafile); } return stat; @@ -1141,12 +1155,18 @@ GpStatus WINGDIPAPI GdipPlayMetafileRecord(GDIPCONST GpMetafile *metafile, case EMR_SETVIEWPORTORGEX: case EMR_SETVIEWPORTEXTEX: case EMR_EXTSELECTCLIPRGN: - case EMR_SETWORLDTRANSFORM: case EMR_SCALEVIEWPORTEXTEX: case EMR_SCALEWINDOWEXTEX: case EMR_MODIFYWORLDTRANSFORM: FIXME("not implemented for record type %x\n", recordType); break; + case EMR_SETWORLDTRANSFORM: + { + const XFORM* xform = (void*)data; + real_metafile->gdiworldtransform = *xform; + METAFILE_PlaybackUpdateGdiTransform(real_metafile); + break; + } default: break; } diff --git a/dlls/gdiplus/tests/metafile.c b/dlls/gdiplus/tests/metafile.c index 83054ca..a61206d 100644 --- a/dlls/gdiplus/tests/metafile.c +++ b/dlls/gdiplus/tests/metafile.c @@ -2260,7 +2260,7 @@ static void test_gditransform(void) stat = GdipBitmapGetPixel(bitmap, 30, 30, &color); expect(Ok, stat); - todo_wine expect(0x00000000, color); + expect(0x00000000, color); stat = GdipDeleteGraphics(graphics); expect(Ok, stat);

1 0

Akihiro Sagawa : webservices: Add support for decoding supplementary characters' references.
by Alexandre Julliard Oct. 10, 2016

Oct. 10, 2016

Module: wine Branch: master Commit: b98293e405eda13a67cae6da654d9774c90f3650 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b98293e405eda13a67cae6da6… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Mon Oct 10 22:53:15 2016 +0900 webservices: Add support for decoding supplementary characters' references. Signed-off-by: Akihiro Sagawa <sagawa.aki(a)gmail.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/webservices/reader.c | 3 ++- dlls/webservices/tests/reader.c | 9 +++++++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/dlls/webservices/reader.c b/dlls/webservices/reader.c index de92603..a6f5d6b 100644 --- a/dlls/webservices/reader.c +++ b/dlls/webservices/reader.c @@ -1190,6 +1190,7 @@ static int codepoint_to_utf8( int cp, unsigned char *dst ) dst[0] = 0xe0 | cp; return 3; } + if (cp >= 0x110000) return -1; dst[3] = 0x80 | (cp & 0x3f); cp >>= 6; dst[2] = 0x80 | (cp & 0x3f); @@ -1259,7 +1260,7 @@ static HRESULT decode_text( const unsigned char *str, ULONG len, unsigned char * if (!len) return WS_E_INVALID_FORMAT; p -= nb_digits = start - len; - if (!nb_digits || nb_digits > 5 || p[nb_digits] != ';') return WS_E_INVALID_FORMAT; + if (!nb_digits || nb_digits > 6 || p[nb_digits] != ';') return WS_E_INVALID_FORMAT; for (i = 0; i < nb_digits; i++) { cp *= 16; diff --git a/dlls/webservices/tests/reader.c b/dlls/webservices/tests/reader.c index f0fe729..a020c01 100644 --- a/dlls/webservices/tests/reader.c +++ b/dlls/webservices/tests/reader.c @@ -3614,11 +3614,16 @@ static void test_entities(void) static const char str29[] = "<t></t>"; static const char str30[] = "<t>A</t>"; static const char str31[] = "<t>ｱ</t>"; + static const char str32[] = "<t>􏿿</t>"; + static const char str33[] = "<t>&#x110000;</t>"; + static const char str34[] = "<t>􏿿</t>"; + static const char str35[] = "<t>&#1114112;</t>"; static const char res4[] = {0xea, 0xaa, 0xaa, 0x00}; static const char res5[] = {0xf2, 0xaa, 0xaa, 0xaa, 0x00}; static const char res21[] = {0xed, 0x9f, 0xbf, 0x00}; static const char res24[] = {0xee, 0x80, 0x80, 0x00}; static const char res31[] = {0xef, 0xbd, 0xb1, 0x00}; + static const char res32[] = {0xf4, 0x8f, 0xbf, 0xbf, 0x00}; static const struct { const char *str; @@ -3658,6 +3663,10 @@ static void test_entities(void) { str29, WS_E_INVALID_FORMAT }, { str30, S_OK, "A" }, { str31, S_OK, res31 }, + { str32, S_OK, res32 }, + { str33, WS_E_INVALID_FORMAT }, + { str34, S_OK, res32 }, + { str35, WS_E_INVALID_FORMAT }, }; HRESULT hr; WS_XML_READER *reader;

1 0