February 2016 - Wine-devel - list.winehq.org

Re: mshtml/tests: Trace the buffer when it is unexpectedly changed in mshtml:protocol.
by Marvin 17 Feb '16

17 Feb '16

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=20637 Your paranoid android. === w1064 (32 bit protocol) === protocol.c:723: Test failed: buf changed buf[0]='' === w1064 (64 bit protocol) === protocol.c:723: Test failed: buf changed buf[0]=''

1 0

Re: schedsvc/tests: Recognize Windows 10's RPC version.
by Marvin 17 Feb '16

17 Feb '16

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=20636 Your paranoid android. === w1064 (32 bit rpcapi) === rpcapi.c:151: Test failed: expected ERROR_FILE_NOT_FOUND, got 0

1 0

Re: mshtml/tests: Use todo_wine_if() in tests
by Marvin 17 Feb '16

17 Feb '16

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=20627 Your paranoid android. === wvistau64 (32 bit htmldoc) === htmldoc.c:5329: Test failed: elem == NULL

1 0

Re: [PATCH] comctl32/tooltip: Protect TTM_ADDTOOLW from invalid text pointers (resend)
by Alexandre Julliard 17 Feb '16

17 Feb '16

Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> writes: > @@ -1076,10 +1077,19 @@ TOOLTIPS_AddToolT (TOOLTIPS_INFO *infoPtr, const TTTOOLINFOW *ti, BOOL isW) > toolPtr->lpszText = LPSTR_TEXTCALLBACKW; > } > else if (isW) { > - INT len = lstrlenW (ti->lpszText); > - TRACE("add text %s!\n", debugstr_w(ti->lpszText)); > - toolPtr->lpszText = Alloc ((len + 1)*sizeof(WCHAR)); > - strcpyW (toolPtr->lpszText, ti->lpszText); > + __TRY > + { > + INT len = lstrlenW (ti->lpszText); > + TRACE("add text %s!\n", debugstr_w(ti->lpszText)); > + toolPtr->lpszText = Alloc ((len + 1)*sizeof(WCHAR)); > + strcpyW (toolPtr->lpszText, ti->lpszText); > + } > + __EXCEPT_PAGE_FAULT > + { > + WARN("Invalid lpszText.\n"); > + return FALSE; You can't just return like that without cleaning up. -- Alexandre Julliard julliard(a)winehq.org

3 3

Re: [PATCH] winemac.drv: Actually make the GL context current even if no drawables were given.
by Ken Thomases 17 Feb '16

17 Feb '16

On Feb 16, 2016, at 1:50 PM, Charles Davis <cdavis5x(a)gmail.com> wrote: > > The spec for GL_ARB_framebuffer_object (and thus, OpenGL 3.x and up) is > quite clear on what happens when a context is made current with no > drawable(s). In fact, the WGL_ARB_create_context extension amends > WGL_ARB_make_current_read (as well as the base spec for wglMakeCurrent) > specifically to allow this. > > Signed-off-by: Charles Davis <cdavis5x(a)gmail.com> > --- > dlls/winemac.drv/cocoa_opengl.m | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/dlls/winemac.drv/cocoa_opengl.m b/dlls/winemac.drv/cocoa_opengl.m > index ec2b191..4694dfd 100644 > --- a/dlls/winemac.drv/cocoa_opengl.m > +++ b/dlls/winemac.drv/cocoa_opengl.m > @@ -235,7 +235,10 @@ void macdrv_make_context_current(macdrv_opengl_context c, macdrv_view v) > } > > if (context) > + { > [context removeFromViews:YES]; > + [context makeCurrentContext]; > + } > } I don't think this can be sufficient. The logic in macdrv_wglMakeContextCurrentARB() essentially means that macdrv_make_context_current() will never be called with a NULL view and a non-NULL context. The "if" statement that you're changing is mostly just extra safety and, as things stand, shouldn't ever execute its statement. -Ken

1 0

Re: [PATCH] gdi32: Declare our intent explicitly to the compiler.
by Michael Stefaniuc 17 Feb '16

17 Feb '16

On 02/17/2016 10:27 AM, Charles Davis wrote: > Clang warns about this abs(3) call because the argument is of > unsigned type. Apparently, though, this is actually a signed value > that just happens to be stored in a DWORD. (Then why is this > parameter to ExtCreatePen() a DWORD?) > > In any case, quiesce this warning with a cast. Well, clang is wrong here. abs() takes a signed integer and the implicit cast from unsigned to signed is perfectly valid C. The policy in Wine is to not uglify the code just to silence misguided static analyzers. And other static code checkers might complain about the superfluous and unneeded explicit cast (I'm working on and off on that). bye michael > > Signed-off-by: Charles Davis <cdavis5x(a)gmail.com> --- > dlls/gdi32/pen.c | 2 +- 1 file changed, 1 insertion(+), 1 > deletion(-) > > diff --git a/dlls/gdi32/pen.c b/dlls/gdi32/pen.c index > 2ae2f2b..cf4b080 100644 --- a/dlls/gdi32/pen.c +++ > b/dlls/gdi32/pen.c @@ -195,7 +195,7 @@ HPEN WINAPI ExtCreatePen( > DWORD style, DWORD width, if (logbrush.lbStyle == BS_DIBPATTERN) > logbrush.lbStyle = BS_DIBPATTERNPT; > > penPtr->logpen.elpPenStyle = style; - penPtr->logpen.elpWidth = > abs(width); + penPtr->logpen.elpWidth = abs((int)width); > penPtr->logpen.elpBrushStyle = logbrush.lbStyle; > penPtr->logpen.elpColor = logbrush.lbColor; penPtr->logpen.elpHatch > = brush->lbHatch;

2 2

Re: [PATCH] wined3d: Implement conversion function for paletted format
by Henri Verbeet 17 Feb '16

17 Feb '16

On 16 February 2016 at 21:15, Sergey Isakov <isakov-sl(a)bk.ru> wrote: > This function is required by game Urban Assault. http://www.old-games.com/download/5663/urban-assault > Bug 37398. > With the patch the game is working. > Stefan already described in the bug report why this is suspicious. This would need tests to show this is needed and correct.

1 0

Re: [v2] user32: Use IsTextUnicode instead of a helper.
by Jactry Zeng 17 Feb '16

17 Feb '16

Sorry, please ignore this patch. I find another bug of Wine's IsTextUnicode. So I can't use it directly. 2016-02-16 16:13 GMT+08:00 Jactry Zeng <jzeng(a)codeweavers.com>: > Hi, > > This patch try to fix tests in patch 119212: > http://source.winehq.org/patches/data/119212 > > I'm not very sure is IsTextUnicode(IS_TEXT_UNICODE_STATISTICS) good > enough for dealing with all cases may be fed by DDE. But it works > for all cases we have in dlls/user32/tests/dde.c . > > ChangeLog: > - Use IS_TEXT_UNICODE_STATISTICS. > > > Signed-off-by: Jactry Zeng <jzeng(a)codeweavers.com> > --- > dlls/user32/dde_server.c | 15 ++++----------- > dlls/user32/tests/dde.c | 20 ++++++++++---------- > 2 files changed, 14 insertions(+), 21 deletions(-) > > > > > > -- Regards, Jactry Zeng

1 0

Re: [PATCH] advapi32: Don't print 32-bit values as long integers.
by Alexandre Julliard 17 Feb '16

17 Feb '16

Charles Davis <cdavis5x(a)gmail.com> writes: > diff --git a/dlls/advapi32/cred.c b/dlls/advapi32/cred.c > index 10e58eb..ff763ee 100644 > --- a/dlls/advapi32/cred.c > +++ b/dlls/advapi32/cred.c > @@ -260,7 +260,7 @@ static DWORD mac_read_credential_from_item(SecKeychainItemRef item, BOOL require > } > if (status != noErr) > { > - WARN("SecKeychainItemCopyAttributesAndData returned status %ld\n", status); > + WARN("SecKeychainItemCopyAttributesAndData returned status %d\n", (int)status); It would probably be better to change the type instead of adding casts everywhere. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 5/6] wined3d: Enable SM5 shaders in the GLSL backend.
by Henri Verbeet 17 Feb '16

17 Feb '16

On 16 February 2016 at 20:48, Matteo Bruni <mbruni(a)codeweavers.com> wrote: > + /* FIXME: Check for the extensions required for SM5 support > + * (ARB_compute_shader, ARB_tessellation_shader, ARB_gpu_shader5, ...) as > + * soon as we introduce them. */ > if (gl_info->glsl_version >= MAKEDWORD_VERSION(1, 50) && gl_info->supported[WINED3D_GL_VERSION_3_2] > && gl_info->supported[ARB_SHADER_BIT_ENCODING] && gl_info->supported[ARB_SAMPLER_OBJECTS]) > - shader_model = 4; > + shader_model = 5; I'd rather not do this without at least a check for the GL and GLSL versions. It would potentially create the kind of situation where some applications start working a little bit and then stop working once we add the proper check.

1 0