July 2014 - Wine-Devel - list.winehq.org

Re: [PATCH 2/4] vbscript: Implemented Sgn(try 5)
by Marvin July 1, 2014

July 1, 2014

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 https://testbot.winehq.org/JobDetails.pl?Key=7802 Your paranoid android. === w2000pro (32 bit) === No test summary line found === wxppro (32 bit) === No test summary line found === wvistau64 (32 bit) === No test summary line found === w2008s64 (32 bit) === No test summary line found === w7u (32 bit) === No test summary line found === w8 (32 bit) === No test summary line found === wvistau64 (64 bit) === No test summary line found === w2008s64 (64 bit) === No test summary line found

1 0

Re: [PATCH 1/4] vbscript: Implemented VarType(try 5)
by Marvin July 1, 2014

July 1, 2014

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 https://testbot.winehq.org/JobDetails.pl?Key=7801 Your paranoid android. === w2000pro (32 bit) === No test summary line found === wxppro (32 bit) === No test summary line found === wvistau64 (32 bit) === No test summary line found === w2008s64 (32 bit) === No test summary line found === w7u (32 bit) === No test summary line found === w8 (32 bit) === No test summary line found === wvistau64 (64 bit) === No test summary line found === w2008s64 (64 bit) === No test summary line found

1 0

Re: shlwapi/tests: Add tests for PathIsRelativeA/W.
by Alexandre Julliard July 1, 2014

July 1, 2014

Zhenbo Li <litimetal(a)gmail.com> writes: > +void test_PathIsRelativeA(void) > +{ > + BOOL ret; > + int i, num; > + static const char *flag[2] = {"FALSE", "TRUE"}; This is not a good idea. There's no guarantee that the return value is always 0 or 1. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 3/3] winecoreaudio.drv: Use AUHAL API instead of AudioQueue
by Ken Thomases July 1, 2014

July 1, 2014

Hi, I'm concerned about the use of the AudioConverter API within the capture callback (ca_capture_cb). I have it embedded in my memory that it's not permitted to call a converter on the device thread. I can't find a hard and fast declaration of that in the documentation, although Technical Note TN2091: Device input using the HAL Output Audio Unit does say this: <https://developer.apple.com/library/mac/technotes/tn2091/_index.html> "If sample rate conversion is needed, it can be accomplished by buffering the input and converting the data on a separate thread with another AudioConverter." If what you're doing were allowed, it would seem they would have mentioned that and not suggested the use of "a separate thread". It also seems that the HAL Output Audio Unit would just do this itself and not have the restriction on the sample rate in the first place. I could have sworn that the API would actually generate errors if you tried to use it on the AUHAL's thread, but I may be thinking of the AUConverter audio unit, instead. Ah, after more searching, I believe it was for the AUConverter audio unit. There's an error, kAudioUnitErr_CannotDoInCurrentContext, that is generated if you try to use such a unit or if you try to make the AudioOutputUnit do the conversion itself. Besides my specific recollection that sample rate conversion is not allowed, there's the general requirement to avoid slow operations in those callbacks. The call to AudioConverterFillComplexBuffer() may be such a slow operation. So could the calls to malloc() in your ca_capture_cb() and feed_cb(). Could you not move the sample rate conversion to the application thread, at the point where the application calls in to get the captured data (or metadata about it)? In AudioClient_IsFormatSupported(), is it really the case that any format for which a stream description can be constructed is supported? Is there a need to try setting the format on an AudioConverter to see if it will accept it? There are still two mentions of AudioQueue in lines like: WARN("AudioQueue doesn't support per-channel volume control\n"); -Ken

2 1

Re: [PATCH 3/5] ddraw/tests: Get rid of struct vertex.
by Marvin July 1, 2014

July 1, 2014

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 https://testbot.winehq.org/JobDetails.pl?Key=7787 Your paranoid android. === w8 (32 bit visual) === visual.c:2633: Test failed: unexpected colortable on offscreen surface

2 1

Re: [PATCH] riched20: Don't create too many IRichEditOle interface for a RichEdit. (try 4)
by Huw Davies July 1, 2014

July 1, 2014

On 30 Jun 2014, at 17:08, Jactry Zeng wrote: > --- > dlls/riched20/editor.c | 13 +++++++++++-- > dlls/riched20/editstr.h | 1 + > dlls/riched20/tests/richole.c | 19 ++++++++++++++++++- > 3 files changed, 30 insertions(+), 3 deletions(-) Looks good to me. Huw.

1 0

Re: [PATCH 5/5] ddraw/tests: Get rid of struct nvertex.
by Marvin July 1, 2014

July 1, 2014

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 https://testbot.winehq.org/JobDetails.pl?Key=7789 Your paranoid android. === w8 (32 bit visual) === visual.c:2631: Test failed: unexpected colortable on offscreen surface

1 0

Re: [PATCH 4/5] ddraw/tests: Get rid of struct tvertex.
by Marvin July 1, 2014

July 1, 2014

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 https://testbot.winehq.org/JobDetails.pl?Key=7788 Your paranoid android. === w8 (32 bit visual) === visual.c:2632: Test failed: unexpected colortable on offscreen surface

1 0