Re: [PATCH] dsound: Use event based threads

2012-12-03 Thread Maarten Lankhorst
Op 03-12-12 17:59, Andrew Eikum schreef: > (...) I think the best solution is to have dsound call into the system > openal to perform multichannel mixing and 3D processing. (...) > > Andrew ... Must print this out on poster format, and frame this above my bed.. ~Maarten

Re: [PATCH] dsound: Use event based threads

2012-12-03 Thread Maarten Lankhorst
Op 03-12-12 14:59, joerg-cyril.hoe...@t-systems.com schreef: > Hi, > > Maarten Lankhorst queried: >> Bump, anything wrong with this patch? > Here's my 0.0$ ... (standard DSound disclaimer here...) > > Using mmdevapi's events in DSound is basically TRT. > Then, the DSound mixer will be synchronized

Re: winepulse test data review

2012-12-03 Thread Maarten Lankhorst
Op 03-12-12 15:42, joerg-cyril.hoe...@t-systems.com schreef: > Hi, > > Maarten Lankhorst kindly posted mmdevapi test results for render and capture > gathered using his winepulse driver: > http://www.winehq.org/pipermail/wine-devel/2012-October/097602.html > >> render.c:1199: padding 1250 position

Re: [PATCH 1/4] kernel32: Support UTF-7 in MultiByteToWideChar.

2012-12-03 Thread Alex Henrie
2012/12/2 Dmitry Timoshkov : > Why don't you put it in libs/wine where other unicode conversion routines > are implemented? Before I started this project I asked where to put the functions: http://www.winehq.org/pipermail/wine-devel/2012-January/093705.html I received no reply, so I put them in l

Re: [3/3] gdiplus: Bitmaps created from a 32-bit dib should be PARGB.

2012-12-03 Thread Vincent Povirk
> test failure I.. might have forgotten this test was in a different file from the other one I was working on when I went to run it on my Windows VM. In any case, sorry about the noise, please disregard this last one.

Re: [3/3] gdiplus: Bitmaps created from a 32-bit dib should be PARGB.

2012-12-03 Thread Dmitry Timoshkov
Vincent Povirk wrote: > case 32: > -format = PixelFormat32bppRGB; > +if (bm.bmBits) > +format = PixelFormat32bppPARGB; > +else > +format = PixelFormat32bppRGB; Besides the test failure, to my eyes format = bm.bmBits ?

Re: [3/3] gdiplus: Bitmaps created from a 32-bit dib should be PARGB.

2012-12-03 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=23202 Your paranoid android

Re: [PATCH 4/6] server: Store and return security attributes with extended file attributes (try 5).

2012-12-03 Thread Erich E. Hoover
On Sun, Dec 2, 2012 at 5:28 PM, Anssi Hannula wrote: > ... > I don't see the need for an uppercase "WINEACL" in the attribute name, > especially since all the examples in e.g. attr(5) are lowercase. > Moreover, [1] recommends using application specific namespaces in the > attribute name, so IMHO a

Re: [PATCH] dsound: Use event based threads

2012-12-03 Thread Andrew Eikum
On Mon, Dec 03, 2012 at 02:59:07PM +0100, joerg-cyril.hoe...@t-systems.com wrote: > BTW, I still believe that mixing and resampling would find their best place > in mmdevapi, not DSound. > Perhaps, but that's provably not what Windows does. Given a format with: dwChannelMask = (CHANNEL_BACK_

Re: kernel32: Implement SetFileValidData. (try 2)

2012-12-03 Thread Alexandre Julliard
Hans Leidekker writes: > QuickBooks 2008 calls this function. > --- > configure.ac |1 + > dlls/kernel32/file.c | 12 +++- > dlls/kernel32/tests/file.c | 136 > > dlls/ntdll/file.c | 28 + > include/ddk/n

winepulse test data review

2012-12-03 Thread Joerg-Cyril.Hoehle
Hi, Maarten Lankhorst kindly posted mmdevapi test results for render and capture gathered using his winepulse driver: http://www.winehq.org/pipermail/wine-devel/2012-October/097602.html >render.c:1199: padding 1250 position 51/21250 slept 470ms iteration 0 I've run your data through some MS-E

[PATCH] dsound: Use event based threads

2012-12-03 Thread Joerg-Cyril.Hoehle
Hi, Maarten Lankhorst queried: >Bump, anything wrong with this patch? Here's my 0.0$ ... (standard DSound disclaimer here...) Using mmdevapi's events in DSound is basically TRT. Then, the DSound mixer will be synchronized with mmdevapi writes. >+ /* ALSA is retarded, add a timeout.

Re: [PATCH 3/3] shell32/tests: Add tests for DoEnvironmentSubstA/W

2012-12-03 Thread Alexandre Julliard
Detlef Riekenberg writes: > +/* Not all variables are supported before vista */ > +if (broken(*bufferA == '%')) > +trace("%d: %s not supported\n", i, bufferA); > +else > +{ > +if (i < 3) todo_wine > +ok(HIWORD(res) && (LOWORD(res

Re: wshom: Basic support for REG_SZ values in RegRead()

2012-12-03 Thread Jacek Caban
Hi Nikolay, On 12/03/12 12:29, Nikolay Sivov wrote: > +/* skip '\' after root name */ > +if (name[len] == '\\') len++; It looks like if name[len] != '\\', then name should be considered invalid. > + > +val = strlenW(name); > +while (name[--val] != '\\' && val) > +; > +

Re: wshom: Basic support for REG_SZ values in RegRead()

2012-12-03 Thread Nikolay Sivov
On 12/3/2012 15:35, Marvin wrote: 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

Re: wshom: Basic support for REG_SZ values in RegRead()

2012-12-03 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=23189 Your paranoid android