Re: [Fwd: 64bit wine]

2009-11-10 Thread Marcus Meissner
On Wed, Nov 11, 2009 at 11:05:49AM +1100, Ben Klein wrote: > 2009/11/11 Michael Stefaniuc : > > For fedora it will not be possible to do a special 32bit compile on the > > x86_64 > > arch. > > Wait ... no gcc -m32 on Fedora x86_64? > > > Hence the only way is to have the default x86_32 version a

Re: [PATCH 02/16] comctl32: Add basic structure for IImageList interface

2009-11-10 Thread Owen Rudge
is wrong because it changes offsets... but it also breaks those comments. (Can you put the vtable at the end?) In comctl32 v6, IImageList and HIMAGELIST are the same thing. See, for instance, http://msdn.microsoft.com/en-us/library/bb762185%28VS.85%29.aspx: "The IImageList pointer type, suc

re: [PATCH 02/16] comctl32: Add basic structure for IImageList interface

2009-11-10 Thread Dan Kegel
As Nikolay pointed out, the change > /* the ones with offsets at the end are the same as in Windows */ > struct _IMAGELIST > { >+const struct IImageListVtbl *lpVtbl;/* IImageList vtable */ >+LONGref;/* reference count */ >+ > DWORD magic;

Re: [PATCH 02/16] comctl32: Add basic structure for IImageList interface

2009-11-10 Thread Nikolay Sivov
On Wed, Nov 11, 2009 at 12:11 AM, Owen Rudge wrote: > --- >  dlls/comctl32/imagelist.c |  322 > - >  dlls/comctl32/imagelist.h |    3 + >  2 files changed, 319 insertions(+), 6 deletions(-) > After that: --- diff --git a/dlls/comctl32/imagelist.h b/dlls

Re: Cpp Check Nov. 7th

2009-11-10 Thread chris ahrendt
Alexandre Julliard wrote: > chris ahrendt writes: > >> Well its been a few since I ran CPP Check and so I ran it today and got the >> following: >> >> New CPP Check Errors for the Wine Code: >> >> [/home/cahrendt/wine-git/dlls/comctl32/imagelist.c:1908]: (error) >> Uninitialized variable: bmi_

RE: [PATCH 16/16] comctl32/tests: Remove todo_wine from SHGetImageList tests

2009-11-10 Thread Owen Rudge
> This patch suggest that one of the previous patches in this series makes > the tests succeed or actually fail as they are inside a todo_wine(). The previous patch is the one that makes the tests succeed. I split the todo_wine removal from the tests due to the modifications being in separate p

Re: [Fwd: 64bit wine]

2009-11-10 Thread Ben Klein
2009/11/11 Michael Stefaniuc : > For fedora it will not be possible to do a special 32bit compile on the x86_64 > arch. Wait ... no gcc -m32 on Fedora x86_64? > Hence the only way is to have the default x86_32 version and a x86_64 > version to be parallel installable. While it may be possible to

Re: Wine 64-bit in Fedora ...

2009-11-10 Thread Ben Klein
2009/11/10 Michael Stefaniuc : > Uwe Bonnes wrote: >> Did nobody of the packages test what they did? > It is easy to miss as it requires: > - Fresh install of Wine (via PackageKit or yum install) and > - Old 32-bit WINEPREFIX. Except that it's obvious that the packagers are not people who USE wine

Re: avifile bugfixes

2009-11-10 Thread Detlef Riekenberg
On Di, 2009-11-10 at 00:43 +0100, Julius Schwartzenberg wrote: > I will add tests to show this and submit the patches then. I created a test for avifil32 in June 2008, but it was not comitted. ( http://www.winehq.org/pipermail/wine-patches/2008-June/056310.html ) The bug 14085 was created to docu

Re: setupapi: add stub and some tests for SetupGetInfFileListW [resend]

2009-11-10 Thread Henri Verbeet
2009/11/10 Stefan Leichter : > When reviewing the patch i found two line containing only spaces. But this are > to my understanding no "whitespace errors". Can you please tell me what else > you have seen ? > That counts as trailing spaces. Note that git apply warns about white space errors by defa

Re: setupapi: add stub and some tests for SetupGetInfFileListW [resend]

2009-11-10 Thread Stefan Leichter
Am Tuesday 10 November 2009 10:09:22 schrieb Henri Verbeet: > 2009/11/9 Stefan Leichter : > > my crystal ball is still not working, therefore resending unchanged > > Try http://source.winehq.org/patches/ > > And indeed, the patch doesn't apply: > > diff -urw wine-git/dlls/setupapi/install.c > wine-

Re: [PATCH 16/16] comctl32/tests: Remove todo_wine from SHGetImageList tests

2009-11-10 Thread Paul Vriens
On 11/10/2009 07:12 PM, Owen Rudge wrote: --- dlls/comctl32/tests/imagelist.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) Hi Owen, This patch suggest that one of the previous patches in this series makes the tests succeed or actually fail as they are inside a todo_wine(). -

Re: setupapi: add stub and some tests for SetupGetInfFileListW [resend]

2009-11-10 Thread Alexandre Julliard
Stefan Leichter writes: > Am Tuesday 10 November 2009 10:09:22 schrieb Henri Verbeet: >> 2009/11/9 Stefan Leichter : >> > my crystal ball is still not working, therefore resending unchanged >> >> Try http://source.winehq.org/patches/ >> >> And indeed, the patch doesn't apply: >> >> diff -urw wine

Re: rsaenh/tests(3/5): Add round-trip test of RSA

2009-11-10 Thread Paul Vriens
On 11/05/2009 07:11 PM, Juan Lang wrote: --Juan Hi Juan, This one introduces some test failures on Win9x and WinMe: http://test.winehq.org/data/tests/rsaenh:rsaenh.html Could you have a look? -- Cheers, Paul.

Re: [8/9] widl: Add support for varargs functions.

2009-11-10 Thread Paul Vriens
On 11/07/2009 03:55 PM, Rob Shearman wrote: --- dlls/rpcrt4/tests/server.c |3 ++ dlls/rpcrt4/tests/server.idl |2 + tools/widl/header.c | 68 ++--- tools/widl/parser.l |1 + tools/widl/parser.y | 12 +--

Re: [2/3] oledb32: Add test for marshalling IDBProperties.

2009-11-10 Thread Paul Vriens
On 11/07/2009 01:29 PM, Rob Shearman wrote: --- dlls/oledb32/tests/Makefile.in |5 +- dlls/oledb32/tests/marshal.c | 272 2 files changed, 275 insertions(+), 2 deletions(-) create mode 100644 dlls/oledb32/tests/marshal.c Hi Rob, The test r

Re: setupapi: add stub and some tests for SetupGetInfFileListW [resend]

2009-11-10 Thread Stefan Leichter
Am Tuesday 10 November 2009 10:09:22 schrieb Henri Verbeet: > 2009/11/9 Stefan Leichter : > > my crystal ball is still not working, therefore resending unchanged > > Try http://source.winehq.org/patches/ > > And indeed, the patch doesn't apply: > > diff -urw wine-git/dlls/setupapi/install.c > wine-

Re: Infinite loop with translation DLL

2009-11-10 Thread Marcus Meissner
On Mon, Nov 09, 2009 at 04:53:31PM +0100, Uwe Bonnes wrote: > > "Henri" == Henri Verbeet writes: > > Henri> 2009/11/9 Uwe Bonnes : > >> Can anybody help with the flaw in my implementation? Or did anything > >> change in wine? > >> > Henri> I'm not sure if anything changed

[Fwd: 64bit wine]

2009-11-10 Thread Michael Stefaniuc
--- Begin Message --- Hi there, thanks for reporting the bug regarding the 64bit wine in bugzilla. I am quite aware of the problem as I get requests about trouble myself. Could you please give some more insight into the way stuff is developing in the wine area regarding future use of 64bit versio

Re: [PATCH] server, ntoskrnl: Implement support for IRP_MJ_CREATE requests.

2009-11-10 Thread Charles Davis
Vitaliy Margolen wrote: > Charles Davis wrote: >> Also, add stub handlers in mountmgr. >> >> This patch fixes an issue with the previous one where wine would hang if >> ntoskrnl got a STATUS_BUFFER_OVERFLOW back from thes server. >> --- > >> +int suspend_thread( struct thread *thread ); >> +int re

Re: mciwave: Rework MCI notification system.

2009-11-10 Thread Paul Vriens
On 11/10/2009 12:56 PM, joerg-cyril.hoe...@t-systems.com wrote: Hi, same patch as yesterday, with more spaces and volatile and mciSave fix thrown out. I was about to resend [PATCH 2] winmm: MCI Close all causes one notification per open driver. which I rebased as well when I realised that it's

Re: Cpp Check Nov. 7th

2009-11-10 Thread Alexandre Julliard
chris ahrendt writes: > Well its been a few since I ran CPP Check and so I ran it today and got the > following: > > New CPP Check Errors for the Wine Code: > > [/home/cahrendt/wine-git/dlls/comctl32/imagelist.c:1908]: (error) > Uninitialized variable: bmi_buf > [/home/cahrendt/wine-git/dlls/n

Re: Wine 64-bit in Fedora ...

2009-11-10 Thread Michael Stefaniuc
Uwe Bonnes wrote: >> "Michael" == Michael Stefaniuc writes: > > Michael> Fedora includes now a 64-bit Wine package (at least in F11) > Michael> which of course breaks existing Wine setups. "yum install wine" > Michael> will install the 64-bit version... Looks like we had our >

Re: Wine 64-bit in Fedora ...

2009-11-10 Thread Damjan Jovanovic
On Tue, Nov 10, 2009 at 1:48 PM, Uwe Bonnes wrote: >> "Michael" == Michael Stefaniuc writes: > >    Michael> Fedora includes now a 64-bit Wine package (at least in F11) >    Michael> which of course breaks existing Wine setups. "yum install wine" >    Michael> will install the 64-bit version.

Re: Wine 64-bit in Fedora ...

2009-11-10 Thread Uwe Bonnes
> "Michael" == Michael Stefaniuc writes: Michael> Fedora includes now a 64-bit Wine package (at least in F11) Michael> which of course breaks existing Wine setups. "yum install wine" Michael> will install the 64-bit version... Looks like we had our Michael> WineConf too late f

Re: Wine 64-bit in Fedora ...

2009-11-10 Thread Michael Stefaniuc
Austin English wrote: > On Mon, Nov 9, 2009 at 5:54 PM, Michael Stefaniuc wrote: >> Fedora includes now a 64-bit Wine package (at least in F11) which of course >> breaks existing Wine setups. "yum install wine" will install the 64-bit >> version... Looks like we had our WineConf too late for this.

Re: setupapi: add stub and some tests for SetupGetInfFileListW [resend]

2009-11-10 Thread Henri Verbeet
2009/11/9 Stefan Leichter : > my crystal ball is still not working, therefore resending unchanged > Try http://source.winehq.org/patches/ And indeed, the patch doesn't apply: diff -urw wine-git/dlls/setupapi/install.c wine-build/dlls/setupapi/install.c --- a/wine-git/dlls/setupapi/install.c +++ b