wine-devel November 2008

wine-devel@winehq.org

110 participants
239 discussions

Re: [PATCH] comctl32: support drawing themed push buttons. [try 4] [resend]
by Alexandre Julliard 05 Nov '08

05 Nov '08

"Reece Dunn" <msclrhd(a)googlemail.com> writes: > +static WCHAR *get_button_text(HWND hwnd, int *len) > +{ > + WCHAR *text; > + *len = GetWindowTextLengthW(hwnd); > + text = HeapAlloc(GetProcessHeap(), 0, (*len + 1) * sizeof(WCHAR)); > + if (text) > + GetWindowTextW(hwnd, text, *len + 1); > + return text; > +} You shouldn't return the length you got from GetWindowTextLength, it can sometimes be larger than necessary. Use the actual length … [View More]

3 2

Re: [ddraw/tests] Fix a test on W2K3
by Henri Verbeet 04 Nov '08

04 Nov '08

2008/11/4 Paul Vriens <paul.vriens.wine(a)gmail.com>: > Hi, > > I upgraded to VMware 6.5.0 this morning and overlay tests are run now on my > W2K3 > box. As the return value indicates E_NOTIMPL I decided to add a skip() for > one particular test. > VMware is not a valid test platform for ddraw.

2 3

Re: [PATCH 2/6] server: Fix lacuna in server side implementation of SetFileSecurity()
by Alexandre Julliard 04 Nov '08

04 Nov '08

Paul Bryan Roberts <pbronline-wine(a)yahoo.co.uk> writes: > The bit I take issue with is not cloning the "FIXME: get Unix uid and > call fchown" in file_set_sd(), nor really the idea that this bit of > code does nothing, not even return an error to indicate it has not > succeeded but that it is always executed (as it were). > > This is because, as the code stands, the object's sd is not current. > There should be a call to file_get_sd() to make it so, then > … [View More]

1 0

Re: [PATCH 1/6] advapi32: GetFileSecurity() a couple of simple tests and a fix
by Alexandre Julliard 04 Nov '08

04 Nov '08

Paul Bryan Roberts <pbronline-wine(a)yahoo.co.uk> writes: > This patch adds a couple of conformance tests that check 'error' > return codes. GetFileSecurity() is expected to be called twice - once > to find out how much memory to allocate and again once the memory has > been allocated. It seems the Windows implementation is a bit fussy > and the implementation of GetFileSecurity() was changed to match. That's not necessary, unless there's an application that depends on … [View More]

1 0

Re: advapi32/tests: Run a test on more platforms (try2)
by Alexandre Julliard 04 Nov '08

04 Nov '08

Nicolas Le Cam <niko.lecam(a)gmail.com> writes: > @@ -614,7 +614,9 @@ static void test_enum_provider_types(void) > DWORD dwTypeCount; > > /* actual values */ > - DWORD index = 0; > + /* the first provider type registry key may not have a TypeName subkey > + * we use the second one instead */ > + DWORD index = 1; That's just moving the problem around, there's no reason that the second one would always be guaranteed to have a typename. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: user32: handle GetMenuItemCount returning -1 (coverity 229)
by Alexandre Julliard 04 Nov '08

04 Nov '08

Aric Stewart <aric(a)codeweavers.com> writes: > diff --git a/dlls/user32/mdi.c b/dlls/user32/mdi.c > index fe70b80..1404436 100644 > --- a/dlls/user32/mdi.c > +++ b/dlls/user32/mdi.c > @@ -393,7 +393,8 @@ static LRESULT MDISetMenu( HWND hwnd, HMENU hmenuFrame, > */ > static LRESULT MDI_RefreshMenu(MDICLIENTINFO *ci) > { > - UINT i, count, visible, id; > + UINT i, visible, id; > + INT count; This shouldn't happen since the menu has just been … [View More]

1 0

Re: wininet: let automated testers force all DNS lookups to return localhost
by James Mckenzie 04 Nov '08

04 Nov '08

Hans Leidekker <hans(a)codeweavers.com> at Oct 31, 2008 5:32 AM wrote about Re: wininet: let automated testers force all DNS lookups to return localhost >> Only include this in debugging builds. > >What happened to your /etc/hosts proposal? It has the >advantage that we don't need changes to Wine source code. > This would not work on Macs as /etc/hosts is locked to admins only and would be a problem to implement. James McKenzie

6 9

Re: [1/5] d3d9: Use the correct AddRef and Release Macros
by Henri Verbeet 03 Nov '08

03 Nov '08

I don't think this patch makes sense.

2 3

Re: Henri Verbeet : ddraw: Make sure pWineDirect3DCreate and pWineDirect3DCreateClipper match the header .
by Rob Shearman 03 Nov '08

03 Nov '08

2008/10/24 Alexandre Julliard <julliard(a)winehq.org>: > diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h > index 7ff3c5c..cd1ce15 100644 > --- a/dlls/ddraw/ddraw_private.h > +++ b/dlls/ddraw/ddraw_private.h > @@ -414,8 +414,7 @@ struct IDirectDrawClipperImpl > > const IDirectDrawClipperVtbl IDirectDrawClipper_Vtbl; > > -typedef IWineD3DClipper* (WINAPI *fnWineDirect3DCreateClipper)(IUnknown *); > -fnWineDirect3DCreateClipper … [View More]

2 2

Re: [resend 2] [try4] advapi32: Implement CredReadDomainCredentials stub and tests
by Alexandre Julliard 03 Nov '08

03 Nov '08

Andrey Turkin <andrey.turkin(a)gmail.com> writes: > + if (TargetInformation->TargetName) > + { > + TargetInformationW->TargetName = buffer; > + buffer += MultiByteToWideChar(CP_ACP, 0, TargetInformation->TargetName, -1, > + TargetInformationW->TargetName, -1); You need to pass the correct destination length, not -1. -- Alexandre Julliard julliard(a)winehq.org

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-devel November 2008