Re: [PATCH 1/4] dlls/ntdll: Add handle count to NtQuerySystemInformation

2011-06-20 Thread Vitaliy Margolen
On 06/20/2011 09:36 PM, Austin Lund wrote: On 20 June 2011 23:52, Vitaliy Margolen wrote: +SERVER_START_REQ( create_snapshot ) You not checking if this call succeeded or not. I've tried to include all the checks in the new patch. +while (sizeof(ULONG) + size

Re: [PATCH 3/6] dinput: SetActionMap setting the device buffer.

2011-06-20 Thread Vitaliy Margolen
On 06/20/2011 04:06 PM, Lucas Fialho Zawacki wrote: @@ -1391,9 +1391,24 @@ HRESULT WINAPI IDirectInputDevice8AImpl_SetActionMap(LPDIRECTINPUTDEVICE8A iface LPCSTR lpszUserName, DWORD dwFlags

Re: Disablin xinput2 for now?

2011-06-20 Thread Vitaliy Margolen
On 06/20/2011 08:36 AM, Alexandre Julliard wrote: Vitaliy Margolen writes: On 06/20/2011 02:55 AM, Marcus Meissner wrote: As this might be a X.org issue, should we packagers turn off xinput2 support for the time being? If that will fix all dinput regressions - absolutely. Or add a registry

Re: Disablin xinput2 for now?

2011-06-20 Thread Vitaliy Margolen
On 06/20/2011 02:31 PM, Austin English wrote: On Mon, Jun 20, 2011 at 09:36, Alexandre Julliard wrote: Vitaliy Margolen writes: On 06/20/2011 02:55 AM, Marcus Meissner wrote: As this might be a X.org issue, should we packagers turn off xinput2 support for the time being? If that will fix

Re: [PATCH 1/4] dlls/ntdll: Add handle count to NtQuerySystemInformation

2011-06-20 Thread Austin Lund
On 20 June 2011 23:52, Vitaliy Margolen wrote: >> +            SERVER_START_REQ( create_snapshot ) > > You not checking if this call succeeded or not. I've tried to include all the checks in the new patch. > >> +                while (sizeof(ULONG) + >> sizeof(SYSTEM_HANDLE_ENTRY)*shi->Count >=

Re: Regression today?

2011-06-20 Thread Austin English
On Mon, Jun 20, 2011 at 21:00, Susan Cragin wrote: > I think a regression was introduced today. I got the following trying to run > NatSpeak 11.0 with today's git. > wine-1.3.22-255-g4c0c0d3 > Should I do a regression test and file a bug, or is it obvious from this? > Or is it me -- something to

Regression today?

2011-06-20 Thread Susan Cragin
I think a regression was introduced today. I got the following trying to run NatSpeak 11.0 with today's git. wine-1.3.22-255-g4c0c0d3 Should I do a regression test and file a bug, or is it obvious from this? Or is it me -- something to do with my new Oneiric Ocelot? Or the new 3.0 kernel? susa

Re: [PATCH 1/3] user32/tests: Added client-side raw input function tests (try 5)

2011-06-20 Thread Marvin
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 http://testbot.winehq.org/JobDetails.pl?Key=11852 Your paranoid android

Re: Disablin xinput2 for now?

2011-06-20 Thread Austin English
On Mon, Jun 20, 2011 at 09:36, Alexandre Julliard wrote: > Vitaliy Margolen writes: > >> On 06/20/2011 02:55 AM, Marcus Meissner wrote: >>> As this might be a X.org issue, should we packagers turn off >>> xinput2 support for the time being? >> >> If that will fix all dinput regressions - absolute

Re: shdocvw: Add ITaskbarList2 Interface (try 2)

2011-06-20 Thread Michael Stefaniuc
Hello André, On 06/20/2011 10:01 PM, André Hentschel wrote: > I should mention that i never added a interface before, neither one > that extends an existing one. I want to stub out ITaskbarList3, so i > thought i should first start with ITaskbarList2 and see what i'm doing > right or wrong. this i

Re: [PATCH 2/3] server: Added server-side raw input function implementations (try 4)

2011-06-20 Thread Vincas Miliūnas
On 06/20/2011 05:24 PM, Vitaliy Margolen wrote: > On 06/20/2011 02:55 AM, Vincas Miliūnas wrote: >> (see the message in the first part of the patch) >> +struct raw_input >> +{ >> +struct listentry; >> +unsigned short usage_page; >> +unsigned short usage; >> +BOOL proce

Re: shdocvw: Add ITaskbarList2 Interface

2011-06-20 Thread André Hentschel
Am 20.06.2011 17:50, schrieb Henri Verbeet: > 2011/6/18 André Hentschel : >> -static HRESULT STDMETHODCALLTYPE taskbar_list_QueryInterface(ITaskbarList >> *iface, REFIID riid, void **object) >> +static HRESULT STDMETHODCALLTYPE taskbar_list_QueryInterface(ITaskbarList2 >> *iface, REFIID riid, LPV

Re: shdocvw: Add ITaskbarList2 Interface

2011-06-20 Thread Henri Verbeet
2011/6/18 André Hentschel : > -static HRESULT STDMETHODCALLTYPE taskbar_list_QueryInterface(ITaskbarList > *iface, REFIID riid, void **object) > +static HRESULT STDMETHODCALLTYPE taskbar_list_QueryInterface(ITaskbarList2 > *iface, REFIID riid, LPVOID *ppv) >  { > -    TRACE("iface %p, riid %s, ob

Re: Disablin xinput2 for now?

2011-06-20 Thread Alexandre Julliard
Vitaliy Margolen writes: > On 06/20/2011 02:55 AM, Marcus Meissner wrote: >> As this might be a X.org issue, should we packagers turn off >> xinput2 support for the time being? > > If that will fix all dinput regressions - absolutely. Or add a > registry option to disable xinput2. It will of cou

Re: [PATCH 2/3] server: Added server-side raw input function implementations (try 4)

2011-06-20 Thread Vitaliy Margolen
On 06/20/2011 02:55 AM, Vincas Miliūnas wrote: (see the message in the first part of the patch) +struct raw_input +{ +struct listentry; +unsigned short usage_page; +unsigned short usage; +BOOL processed; +BOOL retrieved; +RAWINPUT raw; + +

Re: [PATCH 1/3] user32/tests: Added client-side raw input function tests (try 4)

2011-06-20 Thread Vitaliy Margolen
On 06/20/2011 02:55 AM, Vincas Miliūnas wrote: Fixed function name typo, compiler warnings, code style issues in a fragment of new code and incorrect usage of wcslen in the tests instead of lstrlenW after previous changes. My apologies :). +#define DEADBEEF 0xdeadbeef Please don't do this. J

Re: [PATCH 1/4] dlls/ntdll: Add handle count to NtQuerySystemInformation

2011-06-20 Thread Vitaliy Margolen
On 06/20/2011 05:28 AM, Austin Lund wrote: Thanks for the review. Does this patch address these concerns correctly? No quite there yet: +SERVER_START_REQ( create_snapshot ) You not checking if this call succeeded or not. +while (sizeof(ULONG) + sizeof(SYSTEM_HAN

Re: Disablin xinput2 for now?

2011-06-20 Thread Vitaliy Margolen
On 06/20/2011 02:55 AM, Marcus Meissner wrote: As this might be a X.org issue, should we packagers turn off xinput2 support for the time being? If that will fix all dinput regressions - absolutely. Or add a registry option to disable xinput2. Vitaliy.

Re: [PATCH 2/2] d3dx9/line: Implemented tests for ID3DXLine's Begin and End methods.

2011-06-20 Thread Matteo Bruni
2011/6/18 Charles Welton : > --- >  dlls/d3dx9_36/tests/line.c |   23 +++ >  1 files changed, 23 insertions(+), 0 deletions(-) > > +srand(time(NULL)); > +random = ((FLOAT) rand())/RAND_MAX; /* Random value between 0.0 and 1.0. > We're using this later. */ I didn't chec

Re: GSoC: dinput8 Action Mapping

2011-06-20 Thread Lucas Zawacki
> Because callback doesn't have any string parameters it would be an easy > thing to do. I've made an EnumDevicesBySemantics version using A -> W calls, but it turned out to be trickier than I thought because the callback function is passed the device interfaces and then the A callback gets W inte

Re: [PATCH 2/4] server: Add function to count only valid handles in a process

2011-06-20 Thread Austin Lund
On 20 June 2011 15:12, Vitaliy Margolen wrote: > On 06/19/2011 08:15 PM, Austin Lund wrote: >> >> [PATCH 2/4] server: Add function to count only valid handles in a process > > This makes no sense. All handles are valid inside wineserver. When looking at bug 9484 and trying to implement this funct

Disablin xinput2 for now?

2011-06-20 Thread Joerg-Cyril.Hoehle
Hi, >There seem to be regressions in xinput2 support I too have experienced various mouse trouble since approx. 1.3.18 on MacOS, using XQuartz 2.5.x or 2.6.1 -- which is much more recent than any of my Ubuntu systems (Lucid). As I've currently little time for regressions, I haven't yet turned that

Re: [PATCH 1/4] dlls/ntdll: Add handle count to NtQuerySystemInformation

2011-06-20 Thread Austin Lund
On 20 June 2011 15:08, Vitaliy Margolen wrote: > On 06/19/2011 08:15 PM, Austin Lund wrote: >> >> +                    ret = wine_server_call( req ); >> +                    shi->Count += reply->handles; >> +                    len = sizeof(SYSTEM_HANDLE_ENTRY)*shi->Count + >> sizeof(ULONG); >> +

Re: [1/2] d3dx9/test: Add UpdateSemantics test. (try 3)

2011-06-20 Thread Stefan Dösinger
These two patches look OK to me. On Friday 17 June 2011 14:19:56 Michael Mc Donnell wrote: > Removed a superfluous test. > Added NULL check before releasing mesh. > Changed struct declaration style. > Changed the struct declaration style again. > Added GetNumBytesPerVertex test > Added GetDeclarat

Re: [PATCH 1/3] user32/tests: Added client-side raw input function tests (try 4)

2011-06-20 Thread Marvin
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 http://testbot.winehq.org/JobDetails.pl?Key=11841 Your paranoid android

Disablin xinput2 for now?

2011-06-20 Thread Marcus Meissner
Hi, There seem to be regressions in xinput2 support, http://bugs.winehq.org/show_bug.cgi?id=27028 As this might be a X.org issue, should we packagers turn off xinput2 support for the time being? Ciao, Marcus