August 2015 - Wine-Commits - list.winehq.org

Iván Matellanes : msvcirt: Implement filebuf:: setmode.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: 43bb816373e4c1d6fbb581e27f68e1e1bc6df79d URL: http://source.winehq.org/git/wine.git/?a=commit;h=43bb816373e4c1d6fbb581e27… Author: Iván Matellanes <matellanesivan(a)gmail.com> Date: Mon Aug 10 20:09:01 2015 +0200 msvcirt: Implement filebuf::setmode. --- dlls/msvcirt/msvcirt.c | 16 ++++++++++++---- dlls/msvcirt/tests/msvcirt.c | 25 +++++++++++++++++++++++++ 2 files changed, 37 insertions(+), 4 deletions(-) diff --git a/dlls/msvcirt/msvcirt.c b/dlls/msvcirt/msvcirt.c index 7caa05b..83184a0 100644 --- a/dlls/msvcirt/msvcirt.c +++ b/dlls/msvcirt/msvcirt.c @@ -45,9 +45,9 @@ const int filebuf_sh_write = 0xc00; /* ?openprot(a)filebuf@@2HB */ const int filebuf_openprot = 420; /* ?binary(a)filebuf@@2HB */ -const int filebuf_binary = 0x8000; +const int filebuf_binary = _O_BINARY; /* ?text(a)filebuf@@2HB */ -const int filebuf_text = 0x4000; +const int filebuf_text = _O_TEXT; /* ?adjustfield(a)ios@@2JB */ const LONG ios_adjustfield = FLAGS_left | FLAGS_right | FLAGS_internal; @@ -1010,8 +1010,16 @@ streambuf* __thiscall filebuf_setbuf(filebuf *this, char *buffer, int length) DEFINE_THISCALL_WRAPPER(filebuf_setmode, 8) int __thiscall filebuf_setmode(filebuf *this, int mode) { - FIXME("(%p %d) stub\n", this, mode); - return 0; + int ret; + + TRACE("(%p %d)\n", this, mode); + if (mode != filebuf_text && mode != filebuf_binary) + return -1; + + streambuf_lock(&this->base); + ret = (call_streambuf_sync(&this->base) == EOF) ? -1 : _setmode(this->fd, mode); + streambuf_unlock(&this->base); + return ret; } /* ?sync(a)filebuf@@UAEHXZ */ diff --git a/dlls/msvcirt/tests/msvcirt.c b/dlls/msvcirt/tests/msvcirt.c index 0977173..5a58dfd 100644 --- a/dlls/msvcirt/tests/msvcirt.c +++ b/dlls/msvcirt/tests/msvcirt.c @@ -16,6 +16,7 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +#include <fcntl.h> #include <io.h> #include <stdio.h> #include <windef.h> @@ -65,6 +66,8 @@ const int filebuf_sh_none = 0x800; const int filebuf_sh_read = 0xa00; const int filebuf_sh_write = 0xc00; const int filebuf_openprot = 420; +const int filebuf_binary = _O_BINARY; +const int filebuf_text = _O_TEXT; /* class streambuf */ typedef struct { @@ -154,6 +157,7 @@ static void (*__thiscall p_filebuf_dtor)(filebuf*); static filebuf* (*__thiscall p_filebuf_attach)(filebuf*, filedesc); static filebuf* (*__thiscall p_filebuf_open)(filebuf*, const char*, ios_open_mode, int); static filebuf* (*__thiscall p_filebuf_close)(filebuf*); +static int (*__thiscall p_filebuf_setmode)(filebuf*, int); /* ios */ static ios* (*__thiscall p_ios_copy_ctor)(ios*, const ios*); @@ -292,6 +296,7 @@ static BOOL init(void) SET(p_filebuf_attach, "?attach(a)filebuf@@QEAAPEAV1(a)H@Z"); SET(p_filebuf_open, "?open(a)filebuf@@QEAAPEAV1(a)PEBDHH@Z"); SET(p_filebuf_close, "?close(a)filebuf@@QEAAPEAV1(a)XZ"); + SET(p_filebuf_setmode, "?setmode(a)filebuf@@QEAAHH(a)Z"); SET(p_ios_copy_ctor, "??0ios@@IEAA(a)AEBV0@@Z"); SET(p_ios_ctor, "??0ios@@IEAA(a)XZ"); @@ -350,6 +355,7 @@ static BOOL init(void) SET(p_filebuf_attach, "?attach(a)filebuf@@QAEPAV1(a)H@Z"); SET(p_filebuf_open, "?open(a)filebuf@@QAEPAV1(a)PBDHH@Z"); SET(p_filebuf_close, "?close(a)filebuf@@QAEPAV1(a)XZ"); + SET(p_filebuf_setmode, "?setmode(a)filebuf@@QAEHH(a)Z"); SET(p_ios_copy_ctor, "??0ios@@IAE(a)ABV0@@Z"); SET(p_ios_ctor, "??0ios@@IAE(a)XZ"); @@ -957,6 +963,7 @@ static void test_filebuf(void) const char filename2[] = "test2"; const char filename3[] = "test3"; char read_buffer[16]; + int ret; memset(&fb1, 0xab, sizeof(filebuf)); memset(&fb2, 0xab, sizeof(filebuf)); @@ -1106,6 +1113,24 @@ static void test_filebuf(void) ok(pret == NULL, "wrong return, expected %p got %p\n", NULL, pret); fb3.base.do_lock = -1; + /* setmode */ + fb1.base.do_lock = 0; + fb1.base.pbase = fb1.base.pptr = fb1.base.base; + fb1.base.epptr = fb1.base.ebuf; + ret = (int) call_func2(p_filebuf_setmode, &fb1, filebuf_binary); + ok(ret == filebuf_text, "wrong return, expected %d got %d\n", filebuf_text, ret); +todo_wine + ok(fb1.base.pptr == NULL, "wrong put pointer, expected %p got %p\n", NULL, fb1.base.pptr); + ret = (int) call_func2(p_filebuf_setmode, &fb1, filebuf_binary); + ok(ret == filebuf_binary, "wrong return, expected %d got %d\n", filebuf_binary, ret); + fb1.base.do_lock = -1; + ret = (int) call_func2(p_filebuf_setmode, &fb1, 0x9000); + ok(ret == -1, "wrong return, expected -1 got %d\n", ret); + fb2.base.do_lock = 0; + ret = (int) call_func2(p_filebuf_setmode, &fb2, filebuf_text); + ok(ret == -1, "wrong return, expected -1 got %d\n", ret); + fb2.base.do_lock = -1; + /* close */ pret = (filebuf*) call_func1(p_filebuf_close, &fb2); ok(pret == NULL, "wrong return, expected %p got %p\n", NULL, pret);

1 0

André Hentschel : rpcrt4: Add ARM support.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: ef783d62ccdf2cc9dce7490d07acf2e803ca1708 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ef783d62ccdf2cc9dce7490d0… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sun Aug 9 14:09:24 2015 +0200 rpcrt4: Add ARM support. --- dlls/rpcrt4/ndr_stubless.c | 34 ++++++++++++++++++++++++++++++++++ 1 file changed, 34 insertions(+) diff --git a/dlls/rpcrt4/ndr_stubless.c b/dlls/rpcrt4/ndr_stubless.c index dbdf873..033c650 100644 --- a/dlls/rpcrt4/ndr_stubless.c +++ b/dlls/rpcrt4/ndr_stubless.c @@ -1094,6 +1094,40 @@ __ASM_GLOBAL_FUNC( call_server_func, __ASM_CFI(".cfi_adjust_cfa_offset -8\n\t") __ASM_CFI(".cfi_same_value %rbp\n\t") "ret") +#elif defined __arm__ +LONG_PTR __cdecl call_server_func(SERVER_ROUTINE func, unsigned char *args, unsigned int stack_size); +__ASM_GLOBAL_FUNC( call_server_func, + ".arm\n\t" + "push {r4, r5, LR}\n\t" + "mov r4, r0\n\t" + "mov r5, SP\n\t" + "lsr r3, r2, #2\n\t" + "cmp r3, #0\n\t" + "beq 5f\n\t" + "sub SP, SP, r2\n\t" + "tst r3, #1\n\t" + "subeq SP, SP, #4\n\t" + "1:\tsub r2, r2, #4\n\t" + "ldr r0, [r1, r2]\n\t" + "str r0, [SP, r2]\n\t" + "cmp r2, #0\n\t" + "bgt 1b\n\t" + "cmp r3, #1\n\t" + "bgt 2f\n\t" + "pop {r0}\n\t" + "b 5f\n\t" + "2:\tcmp r3, #2\n\t" + "bgt 3f\n\t" + "pop {r0-r1}\n\t" + "b 5f\n\t" + "3:\tcmp r3, #3\n\t" + "bgt 4f\n\t" + "pop {r0-r2}\n\t" + "b 5f\n\t" + "4:\tpop {r0-r3}\n\t" + "5:\tblx r4\n\t" + "mov SP, r5\n\t" + "pop {r4, r5, PC}" ) #else #warning call_server_func not implemented for your architecture LONG_PTR __cdecl call_server_func(SERVER_ROUTINE func, unsigned char * args, unsigned short stack_size)

1 0

André Hentschel : vcomp: Add ARM support.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: 3ad5a07856e8fb9a99e5be78b400c09966e28e93 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3ad5a07856e8fb9a99e5be78b… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sun Aug 9 14:08:23 2015 +0200 vcomp: Add ARM support. --- dlls/vcomp/main.c | 36 ++++++++++++++++++++++++++++++++++++ 1 file changed, 36 insertions(+) diff --git a/dlls/vcomp/main.c b/dlls/vcomp/main.c index 10ac465..8143020 100644 --- a/dlls/vcomp/main.c +++ b/dlls/vcomp/main.c @@ -189,6 +189,42 @@ __ASM_GLOBAL_FUNC( _vcomp_fork_call_wrapper, __ASM_CFI(".cfi_same_value %rbp\n\t") "ret") +#elif defined(__arm__) + +extern void CDECL _vcomp_fork_call_wrapper(void *wrapper, int nargs, __ms_va_list args); +__ASM_GLOBAL_FUNC( _vcomp_fork_call_wrapper, + ".arm\n\t" + "push {r4, r5, LR}\n\t" + "mov r4, r0\n\t" + "mov r5, SP\n\t" + "lsl r3, r1, #2\n\t" + "cmp r3, #0\n\t" + "beq 5f\n\t" + "sub SP, SP, r3\n\t" + "tst r1, #1\n\t" + "subeq SP, SP, #4\n\t" + "1:\tsub r3, r3, #4\n\t" + "ldr r0, [r2, r3]\n\t" + "str r0, [SP, r3]\n\t" + "cmp r3, #0\n\t" + "bgt 1b\n\t" + "cmp r1, #1\n\t" + "bgt 2f\n\t" + "pop {r0}\n\t" + "b 5f\n\t" + "2:\tcmp r1, #2\n\t" + "bgt 3f\n\t" + "pop {r0-r1}\n\t" + "b 5f\n\t" + "3:\tcmp r1, #3\n\t" + "bgt 4f\n\t" + "pop {r0-r2}\n\t" + "b 5f\n\t" + "4:\tpop {r0-r3}\n\t" + "5:\tblx r4\n\t" + "mov SP, r5\n\t" + "pop {r4, r5, PC}" ) + #else static void CDECL _vcomp_fork_call_wrapper(void *wrapper, int nargs, __ms_va_list args)

1 0

André Hentschel : vcomp/tests: Add ARM and ARM64 arch strings.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: e40970bce7bb14063d697caccac6d1794462f29f URL: http://source.winehq.org/git/wine.git/?a=commit;h=e40970bce7bb14063d697cacc… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sun Aug 9 14:06:03 2015 +0200 vcomp/tests: Add ARM and ARM64 arch strings. --- dlls/vcomp/tests/vcomp.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dlls/vcomp/tests/vcomp.c b/dlls/vcomp/tests/vcomp.c index a111d16..7bfddc2 100644 --- a/dlls/vcomp/tests/vcomp.c +++ b/dlls/vcomp/tests/vcomp.c @@ -86,6 +86,10 @@ static void (CDECL *pomp_set_num_threads)(int num_threads); #define ARCH "x86" #elif defined(__x86_64__) #define ARCH "amd64" +#elif defined __arm__ +#define ARCH "arm" +#elif defined __aarch64__ +#define ARCH "arm64" #else #define ARCH "none" #endif

1 0

Bruno Jesus : iphlpapi: Add a stub for SetPerTcpConnectionEStats.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: 89d3f08ca247697d41099772d87e4b4230faab06 URL: http://source.winehq.org/git/wine.git/?a=commit;h=89d3f08ca247697d41099772d… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Sun Aug 9 00:11:35 2015 +0800 iphlpapi: Add a stub for SetPerTcpConnectionEStats. --- dlls/iphlpapi/iphlpapi.spec | 2 +- dlls/iphlpapi/iphlpapi_main.c | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/dlls/iphlpapi/iphlpapi.spec b/dlls/iphlpapi/iphlpapi.spec index d98d5c0..50ea35b 100644 --- a/dlls/iphlpapi/iphlpapi.spec +++ b/dlls/iphlpapi/iphlpapi.spec @@ -285,7 +285,7 @@ #@ stub SetNetworkInformation #@ stub SetPerTcp6ConnectionEStats #@ stub SetPerTcp6ConnectionStats -#@ stub SetPerTcpConnectionEStats +@ stdcall SetPerTcpConnectionEStats( ptr long ptr long long long ) #@ stub SetPerTcpConnectionStats @ stub SetProxyArpEntryToStack @ stub SetRouteWithRef diff --git a/dlls/iphlpapi/iphlpapi_main.c b/dlls/iphlpapi/iphlpapi_main.c index e8daa8c..2efabd3 100644 --- a/dlls/iphlpapi/iphlpapi_main.c +++ b/dlls/iphlpapi/iphlpapi_main.c @@ -53,6 +53,7 @@ #include "fltdefs.h" #include "ifdef.h" #include "netioapi.h" +#include "tcpestats.h" #include "wine/debug.h" #include "wine/unicode.h" @@ -2708,6 +2709,17 @@ DWORD WINAPI SetTcpEntry(PMIB_TCPROW pTcpRow) return 0; } +/****************************************************************** + * SetPerTcpConnectionEStats (IPHLPAPI.@) + */ +DWORD WINAPI SetPerTcpConnectionEStats(PMIB_TCPROW row, TCP_ESTATS_TYPE state, PBYTE rw, + ULONG version, ULONG size, ULONG offset) +{ + FIXME("(row %p, state %d, rw %p, version %u, size %u, offset %u): stub\n", + row, state, rw, version, size, offset); + return ERROR_NOT_SUPPORTED; +} + /****************************************************************** * UnenableRouter (IPHLPAPI.@)

1 0

Nikolay Sivov : dwrite: Implement outline mode for DrawGlyphRun().
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: 204d46c6741d76f13cd52359eb59990cf4a10d55 URL: http://source.winehq.org/git/wine.git/?a=commit;h=204d46c6741d76f13cd52359e… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Aug 13 18:55:52 2015 +0300 dwrite: Implement outline mode for DrawGlyphRun(). --- dlls/dwrite/gdiinterop.c | 168 ++++++++++++++++++++++++++++++++++++++++++++--- dlls/dwrite/tests/font.c | 1 - 2 files changed, 159 insertions(+), 10 deletions(-) diff --git a/dlls/dwrite/gdiinterop.c b/dlls/dwrite/gdiinterop.c index 8c0aa43..da4fc5f 100644 --- a/dlls/dwrite/gdiinterop.c +++ b/dlls/dwrite/gdiinterop.c @@ -45,6 +45,7 @@ struct dib_data { struct rendertarget { IDWriteBitmapRenderTarget1 IDWriteBitmapRenderTarget1_iface; + ID2D1SimplifiedGeometrySink ID2D1SimplifiedGeometrySink_iface; LONG ref; IDWriteFactory *factory; @@ -99,11 +100,111 @@ static inline struct rendertarget *impl_from_IDWriteBitmapRenderTarget1(IDWriteB return CONTAINING_RECORD(iface, struct rendertarget, IDWriteBitmapRenderTarget1_iface); } +static inline struct rendertarget *impl_from_ID2D1SimplifiedGeometrySink(ID2D1SimplifiedGeometrySink *iface) +{ + return CONTAINING_RECORD(iface, struct rendertarget, ID2D1SimplifiedGeometrySink_iface); +} + static inline struct gdiinterop *impl_from_IDWriteGdiInterop(IDWriteGdiInterop *iface) { return CONTAINING_RECORD(iface, struct gdiinterop, IDWriteGdiInterop_iface); } +static HRESULT WINAPI rendertarget_sink_QueryInterface(ID2D1SimplifiedGeometrySink *iface, REFIID riid, void **obj) +{ + if (IsEqualIID(riid, &IID_ID2D1SimplifiedGeometrySink) || + IsEqualIID(riid, &IID_IUnknown)) + { + *obj = iface; + ID2D1SimplifiedGeometrySink_AddRef(iface); + return S_OK; + } + + *obj = NULL; + + return E_NOINTERFACE; +} + +static ULONG WINAPI rendertarget_sink_AddRef(ID2D1SimplifiedGeometrySink *iface) +{ + struct rendertarget *This = impl_from_ID2D1SimplifiedGeometrySink(iface); + return IDWriteBitmapRenderTarget1_AddRef(&This->IDWriteBitmapRenderTarget1_iface); +} + +static ULONG WINAPI rendertarget_sink_Release(ID2D1SimplifiedGeometrySink *iface) +{ + struct rendertarget *This = impl_from_ID2D1SimplifiedGeometrySink(iface); + return IDWriteBitmapRenderTarget1_Release(&This->IDWriteBitmapRenderTarget1_iface); +} + +static void WINAPI rendertarget_sink_SetFillMode(ID2D1SimplifiedGeometrySink *iface, D2D1_FILL_MODE mode) +{ + struct rendertarget *This = impl_from_ID2D1SimplifiedGeometrySink(iface); + SetPolyFillMode(This->hdc, mode == D2D1_FILL_MODE_ALTERNATE ? ALTERNATE : WINDING); +} + +static void WINAPI rendertarget_sink_SetSegmentFlags(ID2D1SimplifiedGeometrySink *iface, D2D1_PATH_SEGMENT vertexFlags) +{ +} + +static void WINAPI rendertarget_sink_BeginFigure(ID2D1SimplifiedGeometrySink *iface, D2D1_POINT_2F startPoint, D2D1_FIGURE_BEGIN figureBegin) +{ + struct rendertarget *This = impl_from_ID2D1SimplifiedGeometrySink(iface); + MoveToEx(This->hdc, startPoint.x, startPoint.y, NULL); +} + +static void WINAPI rendertarget_sink_AddLines(ID2D1SimplifiedGeometrySink *iface, const D2D1_POINT_2F *points, UINT32 count) +{ + struct rendertarget *This = impl_from_ID2D1SimplifiedGeometrySink(iface); + + while (count--) { + LineTo(This->hdc, points->x, points->y); + points++; + } +} + +static void WINAPI rendertarget_sink_AddBeziers(ID2D1SimplifiedGeometrySink *iface, const D2D1_BEZIER_SEGMENT *beziers, UINT32 count) +{ + struct rendertarget *This = impl_from_ID2D1SimplifiedGeometrySink(iface); + POINT points[3]; + + while (count--) { + points[0].x = beziers->point1.x; + points[0].y = beziers->point1.y; + points[1].x = beziers->point2.x; + points[1].y = beziers->point2.y; + points[2].x = beziers->point3.x; + points[2].y = beziers->point3.y; + + PolyBezierTo(This->hdc, points, 3); + beziers++; + } +} + +static void WINAPI rendertarget_sink_EndFigure(ID2D1SimplifiedGeometrySink *iface, D2D1_FIGURE_END figureEnd) +{ + struct rendertarget *This = impl_from_ID2D1SimplifiedGeometrySink(iface); + CloseFigure(This->hdc); +} + +static HRESULT WINAPI rendertarget_sink_Close(ID2D1SimplifiedGeometrySink *iface) +{ + return S_OK; +} + +static const ID2D1SimplifiedGeometrySinkVtbl rendertargetsinkvtbl = { + rendertarget_sink_QueryInterface, + rendertarget_sink_AddRef, + rendertarget_sink_Release, + rendertarget_sink_SetFillMode, + rendertarget_sink_SetSegmentFlags, + rendertarget_sink_BeginFigure, + rendertarget_sink_AddLines, + rendertarget_sink_AddBeziers, + rendertarget_sink_EndFigure, + rendertarget_sink_Close +}; + static HRESULT WINAPI rendertarget_QueryInterface(IDWriteBitmapRenderTarget1 *iface, REFIID riid, void **obj) { struct rendertarget *This = impl_from_IDWriteBitmapRenderTarget1(iface); @@ -213,6 +314,8 @@ static HRESULT WINAPI rendertarget_DrawGlyphRun(IDWriteBitmapRenderTarget1 *ifac TRACE("(%p)->(%.2f %.2f %d %p %p 0x%08x %p)\n", This, originX, originY, measuring_mode, run, params, color, bbox_ret); + SetRectEmpty(bbox_ret); + if (!This->dib.ptr) return S_OK; @@ -221,9 +324,59 @@ static HRESULT WINAPI rendertarget_DrawGlyphRun(IDWriteBitmapRenderTarget1 *ifac if (FAILED(hr)) return hr; - /* FIXME: outline mode rendering is not supported for this target yet */ - if (rendermode == DWRITE_RENDERING_MODE_OUTLINE) - rendermode = DWRITE_RENDERING_MODE_ALIASED; + target.left = target.top = 0; + target.right = This->size.cx; + target.bottom = This->size.cy; + + if (rendermode == DWRITE_RENDERING_MODE_OUTLINE) { + static const XFORM identity = { 1.0f, 0.0f, 0.0f, 1.0f, 0.0f, 0.0f }; + const DWRITE_MATRIX *m = &This->m; + XFORM xform; + + /* target allows any transform to be set, filter it here */ + if (m->m11 * m->m22 == m->m12 * m->m21) { + xform.eM11 = 1.0f; + xform.eM12 = 0.0f; + xform.eM21 = 0.0f; + xform.eM22 = 1.0f; + xform.eDx = originX; + xform.eDy = originY; + } else { + xform.eM11 = m->m11; + xform.eM12 = m->m12; + xform.eM21 = m->m21; + xform.eM22 = m->m22; + xform.eDx = m->m11 * originX + m->m21 * originY + m->dx; + xform.eDy = m->m12 * originX + m->m22 * originY + m->dy; + } + SetWorldTransform(This->hdc, &xform); + + BeginPath(This->hdc); + + hr = IDWriteFontFace_GetGlyphRunOutline(run->fontFace, run->fontEmSize * This->ppdip, + run->glyphIndices, run->glyphAdvances, run->glyphOffsets, run->glyphCount, + run->isSideways, run->bidiLevel & 1, &This->ID2D1SimplifiedGeometrySink_iface); + + EndPath(This->hdc); + + if (hr == S_OK) { + HBRUSH brush = CreateSolidBrush(color); + + SelectObject(This->hdc, brush); + + FillPath(This->hdc); + + /* FIXME: one way to get affected rectangle bounds is to use region fill */ + if (bbox_ret) + *bbox_ret = target; + + DeleteObject(brush); + } + + SetWorldTransform(This->hdc, &identity); + + return hr; + } hr = IDWriteFactory_CreateGlyphRunAnalysis(This->factory, run, This->ppdip, &This->m, rendermode, measuring_mode, @@ -241,11 +394,6 @@ static HRESULT WINAPI rendertarget_DrawGlyphRun(IDWriteBitmapRenderTarget1 *ifac texturetype = DWRITE_TEXTURE_CLEARTYPE_3x1; } - target.left = target.top = 0; - target.right = This->size.cx; - target.bottom = This->size.cy; - - SetRectEmpty(bbox_ret); if (IntersectRect(&target, &target, &bounds)) { UINT32 size = (target.right - target.left) * (target.bottom - target.top); BYTE *bitmap; @@ -261,7 +409,7 @@ static HRESULT WINAPI rendertarget_DrawGlyphRun(IDWriteBitmapRenderTarget1 *ifac else blit_subpixel_888(&This->dib, This->size.cx, bitmap, &target, color); - *bbox_ret = target; + if (bbox_ret) *bbox_ret = target; } heap_free(bitmap); @@ -387,9 +535,11 @@ static HRESULT create_rendertarget(IDWriteFactory *factory, HDC hdc, UINT32 widt if (!target) return E_OUTOFMEMORY; target->IDWriteBitmapRenderTarget1_iface.lpVtbl = &rendertargetvtbl; + target->ID2D1SimplifiedGeometrySink_iface.lpVtbl = &rendertargetsinkvtbl; target->ref = 1; target->hdc = CreateCompatibleDC(hdc); + SetGraphicsMode(target->hdc, GM_ADVANCED); hr = create_target_dibsection(target, width, height); if (FAILED(hr)) { IDWriteBitmapRenderTarget1_Release(&target->IDWriteBitmapRenderTarget1_iface); diff --git a/dlls/dwrite/tests/font.c b/dlls/dwrite/tests/font.c index b132749..9ba0d18 100644 --- a/dlls/dwrite/tests/font.c +++ b/dlls/dwrite/tests/font.c @@ -794,7 +794,6 @@ if (0) /* crashes on native */ /* test mode */ ret = GetGraphicsMode(hdc); -todo_wine ok(ret == GM_ADVANCED, "got %d\n", ret); hbm = GetCurrentObject(hdc, OBJ_BITMAP);

1 0

Huw Davies : gdi32: Add tests for GetFontRealizationInfo, GetFontFileInfo and GetFontFileData.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: 76c321d5e643120921c442017aec75bfd8cc7dcd URL: http://source.winehq.org/git/wine.git/?a=commit;h=76c321d5e643120921c442017… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Aug 13 15:39:19 2015 +0100 gdi32: Add tests for GetFontRealizationInfo, GetFontFileInfo and GetFontFileData. --- dlls/gdi32/freetype.c | 2 +- dlls/gdi32/gdi_private.h | 3 +- dlls/gdi32/tests/font.c | 79 ++++++++++++++++++++++++++++++++++++++++++++++-- 3 files changed, 78 insertions(+), 6 deletions(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index 1187598..c67465a 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -8151,7 +8151,7 @@ static BOOL freetype_GdiRealizationInfo( PHYSDEV dev, void *ptr ) info->flags |= 2; info->cache_num = physdev->font->cache_num; - info->unknown2 = -1; + info->instance_id = -1; return TRUE; } diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index 430e652..2083606 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -281,8 +281,7 @@ typedef struct { DWORD flags; /* 1 for bitmap fonts, 3 for scalable fonts */ DWORD cache_num; /* keeps incrementing - num of fonts that have been created allowing for caching?? */ - DWORD unknown2; /* fixed for a given font - looks like it could be the order of the face in the font list or the order - in which the face was first rendered. */ + DWORD instance_id; /* identifies a realized font instance */ } realization_info_t; diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c index 8595a4a..68ade9a 100644 --- a/dlls/gdi32/tests/font.c +++ b/dlls/gdi32/tests/font.c @@ -55,6 +55,9 @@ static HANDLE (WINAPI *pAddFontMemResourceEx)(PVOID, DWORD, PVOID, DWORD *); static BOOL (WINAPI *pRemoveFontMemResourceEx)(HANDLE); static INT (WINAPI *pAddFontResourceExA)(LPCSTR, DWORD, PVOID); static BOOL (WINAPI *pRemoveFontResourceExA)(LPCSTR, DWORD, PVOID); +static BOOL (WINAPI *pGetFontRealizationInfo)(HDC hdc, DWORD *); +static BOOL (WINAPI *pGetFontFileInfo)(DWORD, DWORD, void *, DWORD, DWORD *); +static BOOL (WINAPI *pGetFontFileData)(DWORD, DWORD, ULONGLONG, void *, DWORD); static HMODULE hgdi32 = 0; static const MAT2 mat = { {0,1}, {0,0}, {0,0}, {0,1} }; @@ -97,6 +100,9 @@ static void init(void) pRemoveFontMemResourceEx = (void *)GetProcAddress(hgdi32, "RemoveFontMemResourceEx"); pAddFontResourceExA = (void *)GetProcAddress(hgdi32, "AddFontResourceExA"); pRemoveFontResourceExA = (void *)GetProcAddress(hgdi32, "RemoveFontResourceExA"); + pGetFontRealizationInfo = (void *)GetProcAddress(hgdi32, "GetFontRealizationInfo"); + pGetFontFileInfo = (void *)GetProcAddress(hgdi32, "GetFontFileInfo"); + pGetFontFileData = (void *)GetProcAddress(hgdi32, "GetFontFileData"); system_lang_id = PRIMARYLANGID(GetSystemDefaultLangID()); } @@ -4106,13 +4112,24 @@ todo_wine DeleteDC(hdc); } -static void test_GdiRealizationInfo(void) +static void test_RealizationInfo(void) { HDC hdc; - DWORD info[4]; + DWORD info[4], info2[10]; BOOL r; HFONT hfont, hfont_old; LOGFONTA lf; + DWORD needed, read; + HANDLE h; + BYTE file[16], data[14]; + struct file_info + { + FILETIME time; + LARGE_INTEGER size; + WCHAR path[MAX_PATH]; + } file_info; + FILETIME time; + LARGE_INTEGER size; if(!pGdiRealizationInfo) { @@ -4147,6 +4164,62 @@ static void test_GdiRealizationInfo(void) ok((info[0] & 0xf) == 3, "info[0] = %x for arial\n", info[0]); ok(info[3] == 0xcccccccc, "structure longer than 3 dwords\n"); + if (pGetFontRealizationInfo) + { + /* The first DWORD represents a struct size. On a + newly rebooted system setting this to < 16 results + in GetFontRealizationInfo failing. However there + appears to be some caching going on which results + in calls after a successful call also succeeding even + if the size < 16. This means we can't reliably test + this behaviour. */ + + memset(info2, 0xcc, sizeof(info2)); + info2[0] = 16; + r = pGetFontRealizationInfo(hdc, info2); + ok(r != 0, "ret 0\n"); + /* We may get the '24' version here if that has been previously + requested. */ + ok(info2[0] == 16 || info2[0] == 24, "got %d\n", info2[0]); + ok(!memcmp(info2 + 1, info, 3 * sizeof(DWORD)), "mismatch\n"); + ok(info2[6] == 0xcccccccc, "structure longer than 6 dwords\n"); + + memset(info2, 0xcc, sizeof(info2)); + info2[0] = 28; + r = pGetFontRealizationInfo(hdc, info2); + ok(r == FALSE, "got %d\n", r); + + memset(info2, 0xcc, sizeof(info2)); + info2[0] = 24; + r = pGetFontRealizationInfo(hdc, info2); + ok(r != 0, "ret 0\n"); + ok(info2[0] == 24, "got %d\n", info2[0]); + ok(!memcmp(info2 + 1, info, 3 * sizeof(DWORD)), "mismatch\n"); + ok(info2[6] == 0xcccccccc, "structure longer than 6 dwords\n"); + + /* Test GetFontFileInfo() */ + r = pGetFontFileInfo(info2[3], 0, &file_info, sizeof(file_info), &needed); + ok(r != 0, "ret 0 gle %d\n", GetLastError()); + + h = CreateFileW(file_info.path, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, 0, NULL); + ok(h != INVALID_HANDLE_VALUE, "Unable to open file %d\n", GetLastError()); + + GetFileTime(h, NULL, NULL, &time); + ok(!CompareFileTime(&file_info.time, &time), "time mismatch\n"); + GetFileSizeEx(h, &size); + ok(file_info.size.QuadPart == size.QuadPart, "size mismatch\n"); + + /* Read first 16 bytes from the file */ + ReadFile(h, file, sizeof(file), &read, NULL); + CloseHandle(h); + + /* Get bytes 2 - 16 using GetFontFileData */ + r = pGetFontFileData(info2[3], 0, 2, data, sizeof(data)); + ok(r != 0, "ret 0 gle %d\n", GetLastError()); + + ok(!memcmp(data, file + 2, sizeof(data)), "mismatch\n"); + } + DeleteObject(SelectObject(hdc, hfont_old)); end: @@ -6374,7 +6447,7 @@ START_TEST(font) skip("Arial Black or Symbol/Wingdings is not installed\n"); test_EnumFontFamiliesEx_default_charset(); test_GetTextMetrics(); - test_GdiRealizationInfo(); + test_RealizationInfo(); test_GetTextFace(); test_GetGlyphOutline(); test_GetTextMetrics2("Tahoma", -11);

1 0

Carlo Bramini : msvcrt: Use MSVCRT_xxx macros instead of the same ones from GLIBC.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: 13f34481b26e89fddeb0f2231aca3a721b972c8f URL: http://source.winehq.org/git/wine.git/?a=commit;h=13f34481b26e89fddeb0f2231… Author: Carlo Bramini <carlo.bramix(a)libero.it> Date: Sun Aug 16 12:48:38 2015 +0200 msvcrt: Use MSVCRT_xxx macros instead of the same ones from GLIBC. --- dlls/msvcrt/file.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index 75fd9cc..6cace71 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -1696,13 +1696,13 @@ int CDECL MSVCRT__fstat64(int fd, struct MSVCRT__stat64* buf) if (type == FILE_TYPE_PIPE) { buf->st_dev = buf->st_rdev = fd; - buf->st_mode = S_IFIFO; + buf->st_mode = MSVCRT__S_IFIFO; buf->st_nlink = 1; } else if (type == FILE_TYPE_CHAR) { buf->st_dev = buf->st_rdev = fd; - buf->st_mode = S_IFCHR; + buf->st_mode = MSVCRT__S_IFCHR; buf->st_nlink = 1; } else /* FILE_TYPE_DISK etc. */ @@ -1714,7 +1714,7 @@ int CDECL MSVCRT__fstat64(int fd, struct MSVCRT__stat64* buf) release_ioinfo(info); return -1; } - buf->st_mode = S_IFREG | 0444; + buf->st_mode = MSVCRT__S_IFREG | 0444; if (!(hfi.dwFileAttributes & FILE_ATTRIBUTE_READONLY)) buf->st_mode |= 0222; buf->st_size = ((__int64)hfi.nFileSizeHigh << 32) + hfi.nFileSizeLow; @@ -2835,7 +2835,7 @@ int CDECL MSVCRT__setmode(int fd,int mode) if(info == &MSVCRT___badioinfo) { *MSVCRT__errno() = MSVCRT_EBADF; - return -1; + return MSVCRT_EOF; } if(mode == MSVCRT__O_BINARY) { @@ -3745,7 +3745,7 @@ int CDECL MSVCRT__getw(MSVCRT_FILE* file) if (k == MSVCRT_EOF) { file->_flag |= MSVCRT__IOEOF; MSVCRT__unlock_file(file); - return EOF; + return MSVCRT_EOF; } ch[j] = k; } @@ -4295,7 +4295,7 @@ MSVCRT_size_t CDECL MSVCRT__fread_nolock_s(void *buf, MSVCRT_size_t buf_size, MS }else { int c = MSVCRT__filbuf(stream); - if(c == EOF) + if(c == MSVCRT_EOF) break; if(!MSVCRT_CHECK_PMT_ERR(buf_size != buf_pos, MSVCRT_ERANGE)) {

1 0

Sebastian Lackner : server: Initialize irp-> thread immediately after creation of irp_call object.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: b279901546299b327313ba19f2e63d5bcf4d4ff9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b279901546299b327313ba19f… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Sun Aug 16 07:59:48 2015 +0200 server: Initialize irp->thread immediately after creation of irp_call object. --- server/device.c | 1 + 1 file changed, 1 insertion(+) diff --git a/server/device.c b/server/device.c index e6dc15e..812bce3 100644 --- a/server/device.c +++ b/server/device.c @@ -259,6 +259,7 @@ static struct irp_call *create_irp( struct device_file *file, const irp_params_t if ((irp = alloc_object( &irp_call_ops ))) { irp->file = (struct device_file *)grab_object( file ); + irp->thread = NULL; irp->async = NULL; irp->params = *params; irp->status = STATUS_PENDING;

1 0

Carlo Bramini : winecfg: Simplify code by using SetDlgItemInt.
by Alexandre Julliard Aug. 17, 2015

Aug. 17, 2015

Module: wine Branch: master Commit: 75e03c13e156691127d881b7a5914b4e91c33f97 URL: http://source.winehq.org/git/wine.git/?a=commit;h=75e03c13e156691127d881b7a… Author: Carlo Bramini <carlo.bramix(a)libero.it> Date: Sun Aug 16 12:08:41 2015 +0200 winecfg: Simplify code by using SetDlgItemInt. --- programs/winecfg/x11drvdlg.c | 31 ++++++++++--------------------- 1 file changed, 10 insertions(+), 21 deletions(-) diff --git a/programs/winecfg/x11drvdlg.c b/programs/winecfg/x11drvdlg.c index 809068e..9a14fb6 100644 --- a/programs/winecfg/x11drvdlg.c +++ b/programs/winecfg/x11drvdlg.c @@ -78,13 +78,13 @@ static void update_gui_for_desktop_mode(HWND dialog) buf = get_reg_keyW(config_key, explorer_desktopsW, desktop_name, NULL); if (buf && (bufindex = strchrW(buf, 'x'))) { - *bufindex = 0; - ++bufindex; - SetWindowTextW(GetDlgItem(dialog, IDC_DESKTOP_WIDTH), buf); - SetWindowTextW(GetDlgItem(dialog, IDC_DESKTOP_HEIGHT), bufindex); + *bufindex++ = 0; + + SetDlgItemTextW(dialog, IDC_DESKTOP_WIDTH, buf); + SetDlgItemTextW(dialog, IDC_DESKTOP_HEIGHT, bufindex); } else { - SetWindowTextA(GetDlgItem(dialog, IDC_DESKTOP_WIDTH), "800"); - SetWindowTextA(GetDlgItem(dialog, IDC_DESKTOP_HEIGHT), "600"); + SetDlgItemTextA(dialog, IDC_DESKTOP_WIDTH, "800"); + SetDlgItemTextA(dialog, IDC_DESKTOP_HEIGHT, "600"); } HeapFree(GetProcessHeap(), 0, buf); @@ -244,17 +244,14 @@ static INT read_logpixels_reg(void) static void init_dpi_editbox(HWND hDlg) { - static const WCHAR fmtW[] = {'%','u',0}; DWORD dwLogpixels; - WCHAR szLogpixels[MAXBUFLEN]; updating_ui = TRUE; dwLogpixels = read_logpixels_reg(); WINE_TRACE("%u\n", dwLogpixels); - sprintfW(szLogpixels, fmtW, dwLogpixels); - SetDlgItemTextW(hDlg, IDC_RES_DPIEDIT, szLogpixels); + SetDlgItemInt(hDlg, IDC_RES_DPIEDIT, dwLogpixels, FALSE); updating_ui = FALSE; } @@ -276,7 +273,6 @@ static void init_trackbar(HWND hDlg) static void update_dpi_trackbar_from_edit(HWND hDlg, BOOL fix) { - static const WCHAR fmtW[] = {'%','u',0}; DWORD dpi; updating_ui = TRUE; @@ -292,11 +288,8 @@ static void update_dpi_trackbar_from_edit(HWND hDlg, BOOL fix) if (fixed_dpi != dpi) { - WCHAR buf[16]; - dpi = fixed_dpi; - sprintfW(buf, fmtW, dpi); - SetDlgItemTextW(hDlg, IDC_RES_DPIEDIT, buf); + SetDlgItemInt(hDlg, IDC_RES_DPIEDIT, dpi, FALSE); } } @@ -420,13 +413,9 @@ GraphDlgProc(HWND hDlg, UINT uMsg, WPARAM wParam, LPARAM lParam) case WM_HSCROLL: switch (wParam) { default: { - static const WCHAR fmtW[] = {'%','d',0}; - WCHAR buf[MAXBUFLEN]; int i = SendMessageW(GetDlgItem(hDlg, IDC_RES_TRACKBAR), TBM_GETPOS, 0, 0); - buf[0] = 0; - sprintfW(buf, fmtW, i); - SendMessageW(GetDlgItem(hDlg, IDC_RES_DPIEDIT), WM_SETTEXT, 0, (LPARAM) buf); - update_font_preview(hDlg); + SetDlgItemInt(hDlg, IDC_RES_DPIEDIT, i, TRUE); + update_font_preview(hDlg); set_reg_key_dwordW(HKEY_LOCAL_MACHINE, logpixels_reg, logpixels, i); break; }

1 0