February 2011 - Wine-Devel - list.winehq.org

Re: rpcrt4/tests: remove a dead assignment (LLVM/Clang)
by Marvin Feb. 10, 2011

Feb. 10, 2011

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=9120 Your paranoid android. === WNT4WSSP6 (32 bit ndr_marshall) === ndr_marshall: unhandled exception c0000005 at 004081C6

2 1

Re: ntdll: Increase buffer size for debugging functions
by Alexandre Julliard Feb. 9, 2011

Feb. 9, 2011

Maarten Lankhorst <m.b.lankhorst(a)gmail.com> writes: > Since warn+debugstr takes user data increase and basically does > WARN("%s\n", msg); the buffer size is too small, increasing the size > makes the lord of the rings online launcher print messages to WARN > without crashing That's just hiding the bug. -- Alexandre Julliard julliard(a)winehq.org

1 0

error message for ddraw
by David Adam Feb. 9, 2011

Feb. 9, 2011

When I test the following test rc = IDirectDraw_SetCooperativeLevel(lpDD, NULL, DDSCL_FULLSCREEN | DDSCL_EXCLUSIVE | DDSCL_CREATEDEVICEWINDOW); ok(rc==DDERR_INVALIDPARAMS, "Expected DDERR_INVALIDPARAMS, received %x\n", rc); in my vista box, I receive the following failing message ddrawmodes: test failed: expected DDERR_INVALIDPARAMS, received 8876035a. But I don't know what is this message. Does anyone know what means this message? Thanks in advance A+ David

2 1

Try to Implement my first Stub function - AbortPrinter() - (try 2).
by Loïc Maury Feb. 9, 2011

Feb. 9, 2011

Hello, After the various comments, I have modified the patch. What do you think ? Thank You Loïc

4 8

Patch 21026 superseded by patch 71118.
by Max TenEyck Woodbury Feb. 9, 2011

Feb. 9, 2011

No text.

1 0

Re: [PATCH 2/5] d3dx9_36: Implemented ID3DXConstantTable_SetIntArray and ID3DXConstantTable_SetInt
by Matteo Bruni Feb. 9, 2011

Feb. 9, 2011

> + /* We need the for loop, since we need to convert the integer to a float */ > + for (i = 0; i < count && i < desc.RegisterCount; i++) > + { > + row[0] = (float)n[i]; The cast is unneeded, and the comment above is too.

2 2

Re: d3dx9_36: make sure return values are used (LLVM/Clang)
by Matteo Bruni Feb. 9, 2011

Feb. 9, 2011

hr = IDirect3DTexture9_GetLevelDesc(texture, 0, &desc); + ok(hr == D3D_OK, "D3DXCreateTexture returned %#x, expected %#x\n", hr, D3D_OK); Copy-paste error, it should be GetLevelDesc in the ok() message. :)

1 0

Re: user32: Add message tests for SetParent().
by Marvin Feb. 9, 2011

Feb. 9, 2011

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=9096 Your paranoid android. === W7PROX64 (64 bit msg) === Timeout

1 0

Re: d3dx9_36: Implement a few tests for D3DXCreateBox
by Stefan Dösinger Feb. 9, 2011

Feb. 9, 2011

Am Mittwoch 09 Februar 2011, 07:34:21 schrieb David Adam: > + wnd = CreateWindow("static", "d3dx9_test", 0, 0, 0, 0, 0, NULL, NULL, NULL, NULL); The class "static" doesn't work on Windows 7, I am afraid you'll need a proper window class. And I think a CreateWindow failure should count as test failure, it's a stupid reason to skip the entire test. (The other d3dx9 tests may have the same issue)

1 0

Re: user32: Add a test for reparenting a WS_POPUP window to a WS_CHILD parent. Take 2.
by Alexandre Julliard Feb. 9, 2011

Feb. 9, 2011

Dmitry Timoshkov <dmitry(a)codeweavers.com> writes: > This test is supposed to show the root of the problem with the test > application attached to the bug 13951. It appears that the check performed > by server/window.c,is_top_level_window() is not correct (which leads to > a SetWindowPos() failure to set foreground window), and probably should be > replaced by simple (win->style & (WS_CHILD | WS_POPUP)) != WS_CHILD. Maybe, but the test doesn't really demonstrate that. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: mshtml/tests: make sure return values are used (LLVM/Clang)
by Marvin Feb. 9, 2011

Feb. 9, 2011

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=9073 Your paranoid android. === WXPPROSP3 (32 bit dom) === No test summary line found

1 0

Re: secur32/tests: make sure a return value is used (LLVM/Clang)
by Marvin Feb. 9, 2011

Feb. 9, 2011

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=9068 Your paranoid android. === WNT4WSSP6 (32 bit ntlm) === ntlm.c:1231: Test failed: AcquireCredentialsHande() returned SEC_E_INVALID_TOKEN ntlm.c:423: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:433: Test failed: expected SEC_E_INTERNAL_ERROR or SEC_I_CONTINUE_NEEDED, got SEC_E_NO_IMPERSONATION ntlm.c:445: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:457: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:885: Test failed: cbSecurityTrailer should be 16 but is 0 ntlm.c:423: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:433: Test failed: expected SEC_E_INTERNAL_ERROR or SEC_I_CONTINUE_NEEDED, got SEC_E_NO_IMPERSONATION ntlm.c:445: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:457: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:885: Test failed: cbSecurityTrailer should be 16 but is 0 ntlm.c:423: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:433: Test failed: expected SEC_E_INTERNAL_ERROR or SEC_I_CONTINUE_NEEDED, got SEC_E_NO_IMPERSONATION ntlm.c:445: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:457: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:885: Test failed: cbSecurityTrailer should be 16 but is 0 ntlm.c:423: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:433: Test failed: expected SEC_E_INTERNAL_ERROR or SEC_I_CONTINUE_NEEDED, got SEC_E_NO_IMPERSONATION ntlm.c:445: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:457: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:885: Test failed: cbSecurityTrailer should be 16 but is 0 ntlm.c:423: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:433: Test failed: expected SEC_E_INTERNAL_ERROR or SEC_I_CONTINUE_NEEDED, got SEC_E_NO_IMPERSONATION ntlm.c:445: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:457: Test failed: expected SEC_E_BUFFER_TOO_SMALL, got SEC_E_INSUFFICIENT_MEMORY ntlm.c:1031: Test failed: MakeSignature returned SEC_E_INVALID_TOKEN, not SEC_E_OK. ntlm.c:1061: Test failed: Crypt trailer not as expected. ntlm.c:1075: Test failed: Crypt message not as expected. ntlm.c:1097: Test failed: DecryptMessage returned SEC_E_MESSAGE_ALTERED, not SEC_E_OK. ntlm.c:1099: Test failed: Failed to decrypt message correctly. ntlm.c:1127: Test failed: MakeSignature returned SEC_E_INVALID_TOKEN, not SEC_E_OK. ntlm.c:1135: Test failed: VerifySignature returned SEC_E_INVALID_TOKEN, not SEC_E_OK ntlm.c:1139: Test failed: EncryptMessage returned SEC_E_INVALID_TOKEN, not SEC_E_OK. ntlm.c:1157: Test failed: Crypt message not as expected. ntlm.c:1176: Test failed: DecryptMessage returned SEC_E_INVALID_TOKEN, not SEC_E_OK.

1 0

Re: comctl32/tests: Use a function instead of a macro for CheckSize test
by Vitaliy Margolen Feb. 9, 2011

Feb. 9, 2011

On 02/08/2011 04:45 PM, EG Galano wrote: > This fixes the warnings: > comctl32/tests/tab.c:525: warning: logical `&&' with non-zero constant > will always evaluate as true > comctl32/tests/tab.c:545: warning: logical `&&' with non-zero constant > will always evaluate as true > comctl32/tests/tab.c:568: warning: logical `&&' with non-zero constant > will always evaluate as true > comctl32/tests/tab.c:1015: warning: logical `&&' with non-zero > constant will always evaluate as true > > for me on Debian gcc 4.3.2 Time to file bug for the compiler. Yes macro looks bad and that's not the way I would have done it now, but there is not a single place where "non-zero constant" is used directly with logical AND. Also what Nikolay said. Vitaliy.

1 0

Re: comctl32/tests: Use a function instead of a macro for CheckSize test
by Nikolay Sivov Feb. 8, 2011

Feb. 8, 2011

On 2/9/2011 02:45, EG Galano wrote: Main problem with that - you will lost ok() context in test output, line numbers will be always the same. Look like it's resolved in other places - a thin macro to pass a line number to a function and ok_(, <linenumber>) test inside.

1 0

Re: iphlpapi: Changed return value of NotifyAddrChange() stub and added test cases. (try 2)
by Juan Lang Feb. 8, 2011

Feb. 8, 2011

Hi Janne, + handle = NULL; + ZeroMemory(&overlapped, sizeof(overlapped)); + overlapped.hEvent = CreateEvent(NULL, FALSE, FALSE, NULL); + ret = gNotifyAddrChange(&handle, &overlapped); + ok(ret == ERROR_IO_PENDING, "NotifyAddrChange returned %d, expected ERROR_IO_PENDING\n", ret); + ok(handle != INVALID_HANDLE_VALUE, "NotifyAddrChange returned invalid file handle\n"); This test is safe, but it depends on the caller setting handle to some value other than INVALID_HANDLE_VALUE. I think it might be safer to have NotifyAddrChange set the handle value instead, and to mark the test todo_wine. I'm worried that some other app might assume that the call to NotifyAddrChange succeeded and attempt to do a wait on an invalid handle value, resulting in a crash. Thanks, --Juan

2 1

Re: po: Update French translation
by Paul Vriens Feb. 8, 2011

Feb. 8, 2011

On 02/08/2011 05:28 PM, Frédéric Delanoy wrote: > #: avifil32.rc:27 > +#, fuzzy > msgid "Waveform: %s" > msgstr "Waveform : %s" > > #: avifil32.rc:28 > +#, fuzzy > msgid "Waveform" > msgstr "Waveform" Hi, Are you adding the 'fuzzy' on purpose? I think that the fuzzy ones are ignored for translations: http://source.winehq.org/git/wine.git/?a=commitdiff;h=1898f22ca5d2d79750e98… -- Cheers, Paul.

2 1

Re: [PATCH 3/5] d3dx9_36: Implemented ID3DXConstantTable_SetVector and ID3DXConstantTable_SetVectorArray
by Matteo Bruni Feb. 8, 2011

Feb. 8, 2011

> + if (is_vertex_shader(This->desc.Version)) > + IDirect3DDevice9_SetVertexShaderConstantF(device, desc.RegisterIndex + i, (float *)fvector, > + desc.RegisterCount); You should probably just use 1 instead of desc.RegisterCount in SetVertexShaderConstantF, as you are already in a loop for each vector. Also, why are you casting a (float *) to a (float *)? Especially after creating a float array just for that. However, making you use a temporary array to convert struct D3DXVECTOR4 to (float *) was somewhat my bad suggestion I believe. My concern was that the structure maybe didn't have its fields packed (with some padding between the fields of the structure for alignment reasons) and just casting the struct pointer to (float *) would not give correct results. Looking at the definition of D3DXVECTOR4 (and assuming it is correct), it appears to not be the case, so you can simply cast it to (float *). Then you can remove the for loop altogether and just set min(count, desc.RegisterCount) registers in the Set{Vertex/Pixel}ShaderConstantF call. Sorry for not being clear enough / giving wrong directions. Please add a "(try n)" suffix to the email subject when you're sending new versions of a patch.

1 0

Re: user32: Add a test for reparenting a WS_POPUP window to a WS_CHILD parent. Take 2.
by Marvin Feb. 8, 2011

Feb. 8, 2011

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=9039 Your paranoid android. === W7PROX64 (64 bit win) === win.c:3480: Test failed: SetForegroundWindow() failed

2 1

Re: gdi32/tests: Use virtual screen size instead of primary monitor screen size.
by Marvin Feb. 8, 2011

Feb. 8, 2011

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=9028 Your paranoid android. === WNT4WSSP6 (32 bit clipping) === clipping.c:438: Test failed: expected 0,0-0,0, got 0,0-800,600

3 2

Re: gdi32/dib.c: Write true bitmap height into info and copy scanlines in the correct order.
by Marvin Feb. 8, 2011

Feb. 8, 2011

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=9030 Your paranoid android. === WNT4WSSP6 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4342 tests executed (0 marked as todo, 2 failures), 1 skipped. === W2KPROSP4 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4357 tests executed (0 marked as todo, 2 failures), 0 skipped. === WXPPROSP3 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W2K3R2SESP2 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === WVISTAADM (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W2K8SE (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W7PRO (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W7PROX64 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W7PROX64 (64 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped.

1 0

Re: gdi32/dib.c: Write true bitmap height into info and copy scanlines in the correct order.
by Marvin Feb. 8, 2011

Feb. 8, 2011

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=9029 Your paranoid android. === WNT4WSSP6 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4342 tests executed (0 marked as todo, 2 failures), 1 skipped. === W2KPROSP4 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4357 tests executed (0 marked as todo, 2 failures), 0 skipped. === WXPPROSP3 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W2K3R2SESP2 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === WVISTAADM (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W2K8SE (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W7PRO (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W7PROX64 (32 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped. === W7PROX64 (64 bit bitmap) === bitmap.c:3099: Test failed: Height should be -2.bitmap.c:3106: Test failed: Height should be -2.bitmap: 4358 tests executed (0 marked as todo, 2 failures), 0 skipped.

1 0

Re: mstask/tests: make sure return values are used (LLVM/Clang)
by Marvin Feb. 8, 2011

Feb. 8, 2011

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=9011 Your paranoid android. === WNT4WSSP6 (32 bit task_trigger) === task_trigger.c:165: Test failed: Failed to setup test_task

2 1

Re: gdi32: Text output of records EMR_EXTTEXTOUTA/W depend on field iGraphicsMode
by Austin English Feb. 8, 2011

Feb. 8, 2011

On Mon, Feb 7, 2011 at 15:49, Alexander Kochetkov <al.kochet(a)gmail.com> wrote: > Fix for bug #22996, and likely for bug #16557. > > BTW, I think, what the same problem exist with EMR_POLYTEXTOUTA/W records. Forgot the patch. -- -Austin

1 0

Re: inetcomm/tests: make sure to use return values (LLVM/Clang)
by Marvin Feb. 8, 2011

Feb. 8, 2011

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=9009 Your paranoid android. === WXPPROSP3 (32 bit mimeole) === No test summary line found

1 0

Re: comctl32/tests: Fixed variable type, removed sign comparison warning.
by Nikolay Sivov Feb. 7, 2011

Feb. 7, 2011

On 2/8/2011 00:26, Marko Nikolic wrote: > --- > dlls/comctl32/tests/trackbar.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/dlls/comctl32/tests/trackbar.c b/dlls/comctl32/tests/trackbar.c > index ac67d50..714a321 100644 > --- a/dlls/comctl32/tests/trackbar.c > +++ b/dlls/comctl32/tests/trackbar.c > @@ -953,7 +953,7 @@ static void test_ignore_selection(HWND hWndTrackbar){ > static void test_initial_state(void) > { > HWND hWnd; > - DWORD ret; > + HRESULT ret; > > hWnd = create_trackbar(0, hWndParent); This is wrong, did you mean LRESULT?

1 0