September 2014 - Wine-Devel

Re: [PATCH 5/8] Revert "wined3d: Restore the pixel format of the window whose pixel format was actually changed."
by Ken Thomases Sept. 14, 2014

Sept. 14, 2014

It seems I accidentally sent this one twice. Please ignore the duplicate, but the series should be OK otherwise. -Ken On Sep 14, 2014, at 7:47 PM, Ken Thomases <ken(a)codeweavers.com> wrote: > > This reverts commit 4c4552c5a1910a9d5adf8eccff0ac62d89ffe376. > > For: > https://bugs.winehq.org/show_bug.cgi?id=35655 > https://bugs.winehq.org/show_bug.cgi?id=35718 > https://bugs.winehq.org/show_bug.cgi?id=35950 > https://bugs.winehq.org/show_bug.cgi?id=35975 > https://bugs.winehq.org/show_bug.cgi?id=36900 > --- > dlls/wined3d/context.c | 87 +++++++++++------------------------------- > dlls/wined3d/wined3d_private.h | 1 - > 2 files changed, 22 insertions(+), 66 deletions(-) > > <0005-Revert-wined3d-Restore-the-pixel-format-of-the-windo.patch>

1 0

Re: wininet: Avoid compiler warnings due to negation before binary operator. (RESEND)
by Alexandre Julliard Sept. 12, 2014

Sept. 12, 2014

Gerald Pfeifer <gerald(a)pfeifer.com> writes: > As of last week, what will become GCC 5.0 does warn about this as > part of -Wall. > > There are a couple of cases in Wine, such as this, which I'd like > to address. Please find a way that doesn't make the code unnecessarily ugly. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/8] riched20: Stub for ITextRange interface and implement ITextDocument::Range. (try 5)
by Huw Davies Sept. 12, 2014

Sept. 12, 2014

On Fri, Sep 12, 2014 at 08:33:29PM +0800, Jactry Zeng wrote: > Thanks Huw and Sebastian. > > Superseded patch 106499. > > ChangeLog: > try 5: > - Alloc txtRge in CreateITextRange() > try 4: > - Simplify with macros in include/wine/list.h > - Using LONG instead of int in This->start and This->end > - Limit cp1 to (0, TextLength) > --- > dlls/riched20/richole.c | 730 > +++++++++++++++++++++++++++++++++++++++++- > dlls/riched20/tests/richole.c | 35 +- > 2 files changed, 762 insertions(+), 3 deletions(-) This first one looks ok to me. You'll need to re-send the rest of the series, but it would be better to wait until this gets in. Huw.

1 0

Re: [PATCH 1/8] riched20: Stub for ITextRange interface and implement ITextDocument::Range. (resend try 4)
by Huw Davies Sept. 12, 2014

Sept. 12, 2014

On Thu, Sep 11, 2014 at 05:35:38PM +0800, Jactry Zeng wrote: > +static HRESULT CreateITextRange(IRichEditOleImpl *reOle, LONG start, LONG end, > + ITextRangeImpl *txtRge, ITextRange** ppRange) > +{ > + txtRge->ITextRange_iface.lpVtbl = &trvt; > + txtRge->ref = 1; > + txtRge->reOle = reOle; > + txtRge->start = start; > + txtRge->end = end; > + list_add_head(&reOle->rangelist, &txtRge->entry); > + *ppRange = &txtRge->ITextRange_iface; > + return S_OK; > +} This looks very suspicious to me. You're storing a reference to 'reOle' so you should AddRef it somewhere. It may be better to pass the interface ptr, AddRef and store that. Also, is the purpose of rangelist just to be able to NULL out reOle when the parent object is destroyed? If so, this can go away if you correctly handle the ref counting. Huw.

3 13

Re: reg: Add reg.exe test suite. (try 3)
by Marvin Sept. 11, 2014

Sept. 11, 2014

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=8896 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: dpnet/tests: Make test_lobbyclient() static.
by Marvin Sept. 11, 2014

Sept. 11, 2014

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=8895 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 2/8] riched20: Implement ITextSelection::GetChar and ITextRange::GetChar. (resend try2)
by Marvin Sept. 11, 2014

Sept. 11, 2014

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=8888 Your paranoid android. === wxppro (32 bit richole) === Failure running script in VM: Timed out in create_socket while connecting to testnet.winehq.org:22001:10.42.42.134:4242

1 0

Re: reg: Add reg.exe test suite. (try 2)
by Alexandre Julliard Sept. 10, 2014

Sept. 10, 2014

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > Try 2: > - Avoid memory allocation in run_reg_exe_ function. > - Add more broken tests in WinXP. > - Compare the exit code outside of run_reg_exe function. > - Introduce REG_EXIT_SUCCESS/REG_EXIT_FAILURE constant and use it. It fails here on 64-bit: ../../../../wine/tools/runtest -q -P wine -T ../../.. -M reg.exe -p reg.exe_test.exe.so reg && touch reg.ok reg.c:180: Test succeeded inside todo block: got wrong data make: *** [reg.ok] Error 1 -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [2/3]dwrite: build non-system font collections as a list of font families
by Nikolay Sivov Sept. 10, 2014

Sept. 10, 2014

On 09/09/2014 04:10 AM, Aric Stewart wrote: > @@ -920,6 +939,8 @@ static UINT32 WINAPI dwritefontcollection_GetFontFamilyCount(IDWriteFontCollecti > { > struct dwrite_fontcollection *This = impl_from_IDWriteFontCollection(iface); > TRACE("(%p)\n", This); > + if (This->family_count) > + return This->family_count; > return This->count; This looks particularly suspicious, probably naming needs some work. Other parts need to be improved as well I think, like avoiding impl_from_* in your own internal functions.

1 0

Re: mshtml: Bind events to document node instead of body element.
by Marvin Sept. 10, 2014

Sept. 10, 2014

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=8870 Your paranoid android. === w2000pro (32 bit) === No test summary line found === wxppro (32 bit) === No test summary line found === wvistau64 (32 bit) === No test summary line found === w2008s64 (32 bit) === No test summary line found === w7u (32 bit) === No test summary line found === w8 (32 bit) === No test summary line found === wvistau64 (64 bit) === No test summary line found === w2008s64 (64 bit) === No test summary line found

1 0

Re: reg: Add reg.exe test suite.
by Marvin Sept. 9, 2014

Sept. 9, 2014

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=8862 Your paranoid android. === wxppro (32 bit reg) === reg.c:132: Test failed: got exit code 0, expected 1

1 0

Re: dxguid: Add dvoice.h
by Alexandre Julliard Sept. 9, 2014

Sept. 9, 2014

Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> writes: > Hi, > > > Changelog: > dxguid: Add dvoice.h There shouldn't be any need to do that. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: (try 7)[1/5] imm32: Move thread data from TLSEntry to an internal list
by Alexandre Julliard Sept. 9, 2014

Sept. 9, 2014

Aric Stewart <aric(a)codeweavers.com> writes: > + if (thread_data->defaultContext) /* someone beat us */ > + IMM_DestroyContext(defaultContext); I don't think you should be doing that inside the thread lock. > + IMMThreadData* thread_data = IMM_GetThreadData(0); > + if (thread_data->hwndDefault == NULL) > + thread_data->hwndDefault = CreateWindowExW( WS_EX_TOOLWINDOW, That one is suspicious too. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [2/3] wineps.drv: Allow text copy paste from print to pdf
by Huw Davies Sept. 9, 2014

Sept. 9, 2014

On Mon, Sep 08, 2014 at 09:19:54PM -0700, Daniel Horn wrote: > -void get_glyph_name(HDC hdc, WORD index, char *name) > +void get_glyph_name(HDC hdc, BOOL unicode, WORD index, char *name) > { > - /* FIXME */ > - sprintf(name, "g%04x", index); > + if(unicode) > + sprintf(name, "uni%04X", index); > + else > + sprintf(name, "g%04x", index); > return; > } The right way to do this is to look up the glyph's name in the font's 'post' table. Huw.

1 0

Re: [1/3] wineps.drv: Avoid marking composite unicode subglyphs as downloaded (resend)
by Huw Davies Sept. 9, 2014

Sept. 9, 2014

On Mon, Sep 08, 2014 at 09:18:51PM -0700, Daniel Horn wrote: > Fixes a bug with caching composite unicode glyphs (if a ñ came before any n in the document, > then all subsequent n's would appear as a box) > This was because the glyph was marked as having been sent down when it was only a component > of a more complex glyph This shouldn't be necessary, I suspect something else is going wrong. If the 'n' has been downloaded as part of a 'ñ' then it really shouldn't be necessary to download the 'n' again. Huw.

2 2

Re: [PATCH 1/4] mshtml: Return E_POINTER when p is NULL in HTMLStyle_get_pixelLeft.
by Jacek Caban Sept. 9, 2014

Sept. 9, 2014

Hi Jactry, On 09/09/14 08:06, Jactry Zeng wrote: > diff --git a/dlls/mshtml/htmlstyle.c b/dlls/mshtml/htmlstyle.c > index f098cb0..9469243 100644 > --- a/dlls/mshtml/htmlstyle.c > +++ b/dlls/mshtml/htmlstyle.c > @@ -2482,6 +2482,8 @@ static HRESULT WINAPI HTMLStyle_get_pixelLeft(IHTMLStyle *iface, LONG *p) > HTMLStyle *This = impl_from_IHTMLStyle(iface); > > TRACE("(%p)->(%p)\n", This, p); > + if(!p) > + return E_POINTER; Since you're adding those checks to many functions, it would probably make sense to move that to get_nsstyle_pixel_val. Jacek

2 1

Re: [PATCH 1/11] ntdll: Make nt_to_unix_file_name_attr return always safe to free.
by Alexandre Julliard Sept. 9, 2014

Sept. 9, 2014

"Erich E. Hoover" <erich.e.hoover(a)gmail.com> writes: > While working on an upcoming patch set I discovered that several of > the file handling primitives are not safe to free under all > circumstances. This would not normally be a problem (since a > comparison can usually be made against STATUS_SUCCESS). However, > there are several cases where this comparison is insufficient (mostly > when STATUS_NO_SUCH_FILE occurs). There are two ways to solve this > problem: > 1) Make these routines always safe to free (and always free them) > 2) Specifically add checks for STATUS_NO_SUCH_FILE and free all these > cases manually STATUS_NO_SUCH_FILE returning a valid string only happens when trying to create the file or things like that, not for normal opens, so most of your changes are unnecessary. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 4/4] mshtml: Add IHTMLStyle::pixelTop property implementation.
by Marvin Sept. 9, 2014

Sept. 9, 2014

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=8853 Your paranoid android. === wxppro (32 bit style) === style.c:1295: Test failed: pixelHeight = 5 === wvistau64 (32 bit style) === style.c:1295: Test failed: pixelHeight = 5 === w2008s64 (32 bit style) === style.c:1295: Test failed: pixelHeight = 5 === w7u (32 bit style) === style.c:1295: Test failed: pixelHeight = 5 === w8 (32 bit style) === style.c:1295: Test failed: pixelHeight = 5 === wvistau64 (64 bit style) === style.c:1295: Test failed: pixelHeight = 5 === w2008s64 (64 bit style) === style.c:1295: Test failed: pixelHeight = 5

1 0

Re: [PATCH 3/4] mshtml: Add IHTMLStyle::pixelHeight property implementation.
by Marvin Sept. 9, 2014

Sept. 9, 2014

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=8852 Your paranoid android. === wxppro (32 bit style) === style.c:1274: Test failed: pixelHeight = 5 === wvistau64 (32 bit style) === style.c:1274: Test failed: pixelHeight = 5 === w2008s64 (32 bit style) === style.c:1274: Test failed: pixelHeight = 5 === w7u (32 bit style) === style.c:1274: Test failed: pixelHeight = 5 === w8 (32 bit style) === style.c:1274: Test failed: pixelHeight = 5 === wvistau64 (64 bit style) === style.c:1274: Test failed: pixelHeight = 5 === w2008s64 (64 bit style) === style.c:1274: Test failed: pixelHeight = 5

1 0

Re: user32: Also set cch member in GetMenuItemInfo when passing a null pointer.
by Nikolay Sivov Sept. 8, 2014

Sept. 8, 2014

A test would be nice.

1 0

Re: (resend 2)[1/4]dwrite: Implement GetInformationalStrings
by Alexandre Julliard Sept. 8, 2014

Sept. 8, 2014

Aric Stewart <aric(a)codeweavers.com> writes: > read strings from OpenType 'name' table You are not reading anything, once again this is mostly dead code. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 4/5] d2d1: Implement initial clipping support.
by Matteo Bruni Sept. 7, 2014

Sept. 7, 2014

2014-09-05 15:50 GMT+02:00 Henri Verbeet <hverbeet(a)codeweavers.com>: > +static BOOL d2d_clip_stack_push(struct d2d_clip_stack *stack, const D2D1_RECT_F *rect) > +{ > + if (stack->current == stack->stack_size - 1) > + { > + D2D1_RECT_F *new_stack; > + unsigned int new_size; > + > + if (stack->stack_size > UINT_MAX / 2) > + return FALSE; > + > + new_size = stack->stack_size * 2; > + if (!(new_stack = HeapReAlloc(GetProcessHeap(), 0, stack->stack, new_size * sizeof(*stack->stack)))) > + return FALSE; > + > + stack->stack = new_stack; > + stack->stack_size = new_size; > + } > + > + stack->stack[stack->current++] = *rect; > + d2d_rect_intersect(&stack->clip_rect, rect); > + > + return TRUE; > +} > + > +static void d2d_clip_stack_pop(struct d2d_clip_stack *stack, unsigned int w, unsigned int h) > +{ > + unsigned int i; > + > + if (!stack->current) > + return; > + > + --stack->current; > + stack->clip_rect.left = 0.0f; > + stack->clip_rect.top = 0.0f; > + stack->clip_rect.right = w; > + stack->clip_rect.bottom = h; > + > + for (i = 0; i < stack->current; ++i) > + { > + d2d_rect_intersect(&stack->clip_rect, &stack->stack[i]); > + } > +} Would it be possible to store the clipped rectangles in the stack instead (so that stack->clip_rect == stack->stack[stack->current] at any given moment)? That way you don't need to recompute clip_rect on each pop.

2 1

[RFC 0/11] Linux FF: Intro
by Elias Vanderstuyft Sept. 7, 2014

Sept. 7, 2014

Hi, This is the sequel of the mail with title: "Discussion and Improvement of Wine's Linux Force Feedback implementation". This mail is meant to give an introduction to the next 11 mails: All tests were performed using wine-1.7.11-206-g82b3813 (but none of the files of which I propose changes to be made in, are affected in the current (2014-03-04) wine-version 1.7.13) Because I haven't created any (formal) patch yet (and instead first request for comments), I tried to keep the line numbers to reference the affected files more or less based to the version I started from, where I could. However, the orig code of a (informal) patch that has to be applied after (and thus is dependent of) a previous patch, may be different from the initial code of the wine-version I started from. After having discussed everything, I will send a proper patchset. In these mails, sometimes 'FEdit' is mentioned: it's a MS application that allows to create all FF effect types (except CustomForce and rumble/vibration effects), and therefore the ideal test application. Physical behavior is compared between Windows7 (using Logitech MOMO wheel with latest drivers installed) running in Virtualbox, and my patched Wine-version. In the end, (using a dummy FF driver on the Linux side, thanks to Michal Malý), the behavior of Wine is almost (limitations caused by the dummy ff driver, such as updating an effect while playing is not implemented) identical, if not better (in some aspects) then the Windows equivalent. Sometimes I refer to linux-input mailing lists to prove a statement, because the links may vanish (unlikely though), I provide the titles corresponding with the links: (http://www.mail-archive.com/linux-input(a)vger.kernel.org/msg08513.html) "ff-core effect id handling in case of a failed effect upload" (http://www.mail-archive.com/linux-input(a)vger.kernel.org/msg08459.html) "Questions about the documentation/specification of Linux ForceFeedback input.h" Subjects [11] ------------- - Joystick control panel program [2] - Linux effect status management [3] - Linux effect parameter: start delay [1] - Linux effect parameter: periodic effect's phase [1] - Linux effect parameters: time parameters [3] - Linux effect: conditional effect's parameters [2] - Linux effect: axes [1] - Linux effect direction [7] - Linux effect parameters (global) [6] - Linux joystick initialization [1] - Debug message fixes/additions [6] Elias

2 3

Re: [PATCH 5/5] d2d1/tests: Add a basic clipping tests.
by Marvin Sept. 5, 2014

Sept. 5, 2014

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=8807 Your paranoid android. === build (build) === Recreation of tests/Makefile failed

2 1

Re: [PATCH 1/5] widl: Handle aggregate returns in a MSVC compatible way.
by Piotr Caban Sept. 5, 2014

Sept. 5, 2014

On 09/05/14 15:50, Henri Verbeet wrote: > -static D2D1_COLOR_F STDMETHODCALLTYPE d2d_solid_color_brush_GetColor(ID2D1SolidColorBrush *iface) > +static void STDMETHODCALLTYPE d2d_solid_color_brush_GetColor(ID2D1SolidColorBrush *iface, D2D1_COLOR_F *color) > { > static const D2D1_COLOR_F black = {0.0f, 0.0f, 0.0f, 1.0f}; > > FIXME("iface %p stub!\n", iface); > > - return black; > + *color = black; > } Shouldn't it return pointer to D2D1_COLOR_F?

3 2