Re: include[1/2]: add a strcmpW-equivalent function usable in tests.

2010-02-22 Thread Mikołaj Zalewski
2010/2/22 Dmitry Timoshkov dmi...@codeweavers.com: Mikołaj Zalewski miko...@zalewski.pl wrote: +/* strcmpW is avaiable for tests compiled under Wine, but not in standalone + * builds under Windows, so we reimplement it under a different name. */ +static inline int winetest_strcmpW( const

Re: includes: Fix IsEqualPropertyKey definitions

2010-02-22 Thread Alexandre Julliard
Maarten Lankhorst m.b.lankho...@gmail.com writes: @@ -48,7 +48,11 @@ #endif #ifndef IsEqualPropertyKey +#ifdef __cplusplus #define IsEqualPropertyKey(a,b) (((a).pid == (b).pid) IsEqualIID((a).fmtid,(b).fmtid)) +#else +#define IsEqualPropertyKey(a,b) (((a)-pid == (b)-pid)

Re: [PATCH] shell32: Allow copy operation to overwrite an existing write protected file + tests

2010-02-22 Thread Alexandre Julliard
Christian Costa titan.co...@wanadoo.fr writes: @@ -627,6 +627,14 @@ static DWORD SHNotifyCopyFileW(LPCWSTR src, LPCWSTR dest, BOOL bFailIfExists) TRACE((%s %s %s)\n, debugstr_w(src), debugstr_w(dest), bFailIfExists ? failIfExists : ); +if (PathFileExistsW(dest)) +

Re: [PATCH 3/5] d3dx9_36: Implement stubbed ID3DXConstantTable interface

2010-02-22 Thread Henri Verbeet
On 21 February 2010 22:30, Christian Costa titan.co...@wanadoo.fr wrote: +if (IsEqualGUID(riid, IID_IUnknown) || +IsEqualGUID(riid, IID_ID3DXConstantTable)) ID3DXConstantTable extends ID3DXBuffer. +ERR(Interface %s not found\n, debugstr_guid(riid)); Not implementing a random

Re: [PATCH 4/5] d3dx9_36: Implement D3DXFindShaderComment (based on code from Luis Busquets)

2010-02-22 Thread Henri Verbeet
I think tests would be appropriate.

Re: [PATCH 4/5] d3dx9_36: Implement D3DXFindShaderComment (based on code from Luis Busquets)

2010-02-22 Thread Christian Costa
Sure. Having tests is better but in this case there is no chance for regression. I usually write tests for my personnal pursose or to compare behaviour against real windows or native dll. I don't think lack of tests in this case should prevent integration. Tests can be added later if needed.

Re: [PATCH 4/5] d3dx9_36: Implement D3DXFindShaderComment (based on code from Luis Busquets)

2010-02-22 Thread Henri Verbeet
On 22 February 2010 13:43, Christian Costa titan.co...@wanadoo.fr wrote: Sure. Having tests is better but in this case there is no chance for regression. I usually write tests for my personnal pursose or to compare behaviour against real windows or native dll. You write tests to prove your

Re: [PATCH 4/5] d3dx9_36: Implement D3DXFindShaderComment (based on code from Luis Busquets)

2010-02-22 Thread Christian Costa
Wrt the correctness of the behaviour about these parameters, it seems the documentation is not something we should rely on. That's why it's better to write test. So until there is an app which rely on this particular behaviour, this does not worth the trouble. It's not from me. This is

Re: [PATCH 4/5] d3dx9_36: Implement D3DXFindShaderComment (based on code from Luis Busquets)

2010-02-22 Thread Henri Verbeet
On 22 February 2010 15:15, Christian Costa titan.co...@wanadoo.fr wrote: More generally I would say that over-reviewing is not a good thing. It does not bring much in term of quality and slow down development. Thrust is the base of a community. Sure, but that trust has to be justified. I can

Trouble compiling Wine tests with Cygwin

2010-02-22 Thread Alex Villací­s Lasso
I have a VB6 application that requires oledb32 to convert from DBTYPE_VARIANT to DBTYPE_BYTES, and fails with builtin oledb32 since conversions to DBTYPE_BYTES are unimplemented. I set up a VirtualBox VM with Windows XP and the latest Cygwin in order to add a oledb32 test to show me what

Re: includes: Fix IsEqualPropertyKey definitions

2010-02-22 Thread Maarten Lankhorst
Hi AJ, 2010/2/22 Alexandre Julliard julli...@winehq.org: Maarten Lankhorst m.b.lankho...@gmail.com writes: @@ -48,7 +48,11 @@  #endif  #ifndef IsEqualPropertyKey +#ifdef __cplusplus  #define IsEqualPropertyKey(a,b) (((a).pid == (b).pid) IsEqualIID((a).fmtid,(b).fmtid)) +#else

Re: includes: Fix IsEqualPropertyKey definitions

2010-02-22 Thread Alexandre Julliard
Maarten Lankhorst m.b.lankho...@gmail.com writes: This doesn't match the PSDK. Where does this come from? In C, REFPROPERTYKEY becomes PROPERTYKEY*, in C++ it's PROPERTYKEY , so IsEqualPropertyKey will need a different macro. The psdk doesn't define this properly for the C case, but since I

Re: includes: Fix IsEqualPropertyKey definitions

2010-02-22 Thread Maarten Lankhorst
Hi AJ, 2010/2/22 Alexandre Julliard julli...@winehq.org: Maarten Lankhorst m.b.lankho...@gmail.com writes: This doesn't match the PSDK. Where does this come from? In C, REFPROPERTYKEY becomes PROPERTYKEY*, in C++ it's PROPERTYKEY , so IsEqualPropertyKey will need a different macro. The psdk

Re: includes: Fix IsEqualPropertyKey definitions

2010-02-22 Thread Alexandre Julliard
Maarten Lankhorst m.b.lankho...@gmail.com writes: That doesn't match the PSDK I have, and yes it has a C definition that is different from the C++ one, but is also different from yours. Mine doesn't seem to have a C definition at all, but from the devpropdef one I assume the C version takes a

Re: Trouble compiling Wine tests with Cygwin

2010-02-22 Thread Austin English
On Mon, Feb 22, 2010 at 11:03 AM, Alex Villací­s Lasso a_villa...@palosanto.com wrote: I have a VB6 application that requires oledb32 to convert from DBTYPE_VARIANT to DBTYPE_BYTES, and fails with builtin oledb32 since conversions to DBTYPE_BYTES are unimplemented. I set up a VirtualBox VM with

Re: Trouble compiling Wine tests with Cygwin

2010-02-22 Thread Alex Villací­s Lasso
El 22/02/10 12:41, Austin English escribió: On Mon, Feb 22, 2010 at 11:03 AM, Alex Villací­s Lasso a_villa...@palosanto.com wrote: I have a VB6 application that requires oledb32 to convert from DBTYPE_VARIANT to DBTYPE_BYTES, and fails with builtin oledb32 since conversions to DBTYPE_BYTES

[PATCH] server: Fix console create function

2010-02-22 Thread XueFeng Chang
Check create_event return value. --- server/console.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/server/console.c b/server/console.c index a79dc67..3758a71 100644 --- a/server/console.c +++ b/server/console.c @@ -294,7 +294,7 @@ static struct object

Re: [PATCH] shlwapi/tests: skip SHCreateStreamOnFileA/W configurations not supported on Win98 SE.

2010-02-22 Thread Paul Vriens
On 02/23/2010 01:28 AM, Reece Dunn wrote: +if (ret == E_INVALIDARG) /* Win98 SE */ { +skip(Not supported\n); +return; +} Hi Reece, Could you turn that (and the other one) into a win_skip() ? -- Cheers, Paul.

Re: dns-sd dll implementation -- Needs work

2010-02-22 Thread Dmitry Timoshkov
C.W. Betts computer...@hotmail.com wrote: What do you guys think? it's a wrapper for multicast DNS, aka Bonjour®. Looks like this is not a part of Windows or PSDK, so Wine shouldn't include it either. -- Dmitry.

Re: [PATCH] shlwapi/tests: skip SHCreateStreamOnFileA/W configurations not supported on Win98 SE.

2010-02-22 Thread Reece Dunn
On 23 February 2010 07:33, Paul Vriens paul.vriens.w...@gmail.com wrote: On 02/23/2010 01:28 AM, Reece Dunn wrote: +    if (ret == E_INVALIDARG) /* Win98 SE */ { +        skip(Not supported\n); +        return; +    } Hi Reece, Could you turn that (and the other one) into a win_skip() ?

Re: [PATCH 4/5] d3dx9_36: Implement D3DXFindShaderComment (based on code from Luis Busquets)

2010-02-22 Thread Christian Costa
Henri Verbeet a écrit : On 22 February 2010 15:15, Christian Costa titan.co...@wanadoo.fr wrote: More generally I would say that over-reviewing is not a good thing. It does not bring much in term of quality and slow down development. Thrust is the base of a community. Sure, but that