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

Re: [PATCH 3/3] ddraw/tests: Extend the CreateSurface(NULL test), use own function
by Marvin Dec. 7, 2011

Dec. 7, 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=15825 Your paranoid android. === W2KPROSP4 (32 bit dsurface) === dsurface.c:4626: Test failed: SetSurfaceDesc failed, hr 0x80070057. dsurface.c:4630: Test failed: SetSurfaceDesc failed, hr 0x80070057. === W7PRO (32 bit dsurface) === dsurface.c:4708: Test failed: CreateSurface failed, hr 0x88760233. dsurface: unhandled exception c0000005 at 0042E584 === W7PROX64 (32 bit dsurface) === dsurface.c:4708: Test failed: CreateSurface failed, hr 0x88760233. dsurface: unhandled exception c0000005 at 0042E584 === TEST64_W7SP1 (32 bit dsurface) === dsurface.c:4708: Test failed: CreateSurface failed, hr 0x88760233. dsurface: unhandled exception c0000005 at 0042E584 === W7PROX64 (64 bit dsurface) === dsurface.c:4708: Test failed: CreateSurface failed, hr 0x88760233. Timeout === TEST64_W7SP1 (64 bit dsurface) === dsurface.c:4708: Test failed: CreateSurface failed, hr 0x88760233. Timeout

1 0

Re: [PATCH 2/3] ddraw: SetSurfaceDesc can set width, height and pitch under certain conditions
by Marvin Dec. 7, 2011

Dec. 7, 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=15824 Your paranoid android. === W2KPROSP4 (32 bit dsurface) === dsurface.c:4628: Test failed: SetSurfaceDesc failed, hr 0x80070057. dsurface.c:4632: Test failed: SetSurfaceDesc failed, hr 0x80070057. === W7PRO (32 bit dsurface) === dsurface.c:4710: Test failed: CreateSurface failed, hr 0x88760233. dsurface: unhandled exception c0000005 at 0042E5E4 === W7PROX64 (32 bit dsurface) === dsurface.c:4710: Test failed: CreateSurface failed, hr 0x88760233. dsurface: unhandled exception c0000005 at 0042E5E4 === TEST64_W7SP1 (32 bit dsurface) === dsurface.c:4710: Test failed: CreateSurface failed, hr 0x88760233. dsurface: unhandled exception c0000005 at 0042E5E4 === W7PROX64 (64 bit dsurface) === dsurface.c:4710: Test failed: CreateSurface failed, hr 0x88760233. Timeout === TEST64_W7SP1 (64 bit dsurface) === dsurface.c:4710: Test failed: CreateSurface failed, hr 0x88760233. Timeout

1 0

Re: [PATCH 1/3] wininet: Use test.winehq.org instead of crossover.codeweavers.com in test_async_HttpSendRequestEx
by Marvin Dec. 7, 2011

Dec. 7, 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=15809 Your paranoid android. === WXPPROSP3 (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12002 === WVISTAADM (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === W2K8SE (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === W7PRO (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === W7PROX64 (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === TEST64_W7SP1 (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === W7PROX64 (64 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set

2 1

Re: [PATCH 3/3] wininet: Added more connection failure tests
by Marvin Dec. 7, 2011

Dec. 7, 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=15810 Your paranoid android. === W2K3R2SESP2 (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === WVISTAADM (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === W2K8SE (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === W7PRO (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set === W7PROX64 (32 bit http) === http.c:2873: Test failed: HttpSendRequest failed: 12057 http.c:2878: Test failed: expected secure flag to be set http.c:2957: Test failed: HttpSendRequest failed: 12057 http.c:2962: Test failed: expected secure flag to be set

1 0

Re: [PATCH 5/6] jscript: Use bytecode for post-increment expression
by Marvin Dec. 7, 2011

Dec. 7, 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=15795 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

1 0

Re: mshtml: Don't use WINAPI for assembly functions to avoid linker warnings on mingw
by Francois Gouget Dec. 7, 2011

Dec. 7, 2011

> #ifdef __i386__ > > #define DEFINE_WRAPPER_FUNC(n, off, x) \ > HRESULT wrapper_func_##n(IUnknown*); \ [...] > #elif defined(__x86_64__) > > #define DEFINE_WRAPPER_FUNC(n, x, off) \ > HRESULT WINAPI wrapper_func_##n(IUnknown*); \ [...] > #else > > #define DEFINE_WRAPPER_FUNC(n, x, off) \ > static HRESULT WINAPI wrapper_func_##n(IUnknown *iface) { \ [...] Is it normal that these functions are still marked as WINAPI in the 64bit and other platform cases? Also, is there a reason not to mark these functions as static in the Intel cases? Because they are coded in assembly? -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ The software said it requires Win95 or better, so I installed Linux.

2 1

Re: riched20: fix memory leak
by Alexandre Julliard Dec. 7, 2011

Dec. 7, 2011

Pierre Schweitzer <pierre(a)reactos.org> writes: > @@ -102,6 +102,7 @@ ME_UndoItem *ME_AddUndoItem(ME_TextEditor *editor, ME_DIType type, const ME_Disp > } > default: > assert(0 == "AddUndoItem, unsupported item type"); > + ME_DestroyDisplayItem(pItem); > return NULL; It's not useful to free memory in unreachable code paths. -- Alexandre Julliard julliard(a)winehq.org

1 0

dosdevices/c not present.. how is it created?
by Roger Cruz Dec. 7, 2011

Dec. 7, 2011

I'm getting the following error when running Wine. err:process:init_windows_dirs directory L"C:\\windows" could not be created, error 2 err:process:init_windows_dirs directory L"C:\\windows\\system32" could not be created, error 3 Using the debugger, I think I found where the problem is: 2140 int fd = open( unix_name, O_RDONLY | O_DIRECTORY ); (gdb) p unix_name $25 = 0x4040b2e8 "/data/winehome/dosdevices/c:" (gdb) n 2141 if (fd != -1) (gdb) p fd $26 = -1 (gdb) where #0 find_file_in_dir (unix_name=0x4040b2e8 "/data/winehome/dosdevices/c:", pos=29, name=0x4040b2c6, length=7, check_case=0, is_win_dir=0x0) at /home/rcruz/sandbox/wine-dev-branch/dlls/ntdll/directory.c:2141 #1 0x80236cf4 in lookup_unix_name (name=0x4040b2c6, name_len=0, buffer=0xbebc45e4, unix_len=295, pos=28, disposition=2, check_case=0 '\0') at /home/rcruz/sandbox/wine-dev-branch/dlls/ntdll/directory.c:2759 #2 0x80237ad4 in wine_nt_to_unix_file_name (nameW=0xbebc48f8, unix_name_ret=0xbebc4844, disposition=2, check_case=0 '\0') at /home/rcruz/sandbox/wine-dev-branch/dlls/ntdll/directory.c:2981 #3 0x80237004 in nt_to_unix_file_name_attr (attr=0xbebc4900, unix_name_ret=0xbebc4844, disposition=2) at /home/rcruz/sandbox/wine-dev-branch/dlls/ntdll/directory.c:2821 #4 0x8023b744 in FILE_CreateFile (handle=0xbebc48ec, access=2147483648, attr=0xbebc4900, io=0xbebc48f0, alloc_size=0x0, attributes=128, sharing=1, disposition=2, options=33, ea_buffer=0x0, ea_length=0) at /home/rcruz/sandbox/wine-dev-branch/dlls/ntdll/file.c:125 #5 0x8023bd40 in NtCreateFile (handle=0xbebc48ec, access=2147483648, attr=0xbebc4900, io=0xbebc48f0, alloc_size=0x0, attributes=128, sharing=1, disposition=2, options=33, ea_buffer=0x0, ea_length=0) at /home/rcruz/sandbox/wine-dev-branch/dlls/ntdll/file.c:271 #6 0x8045d210 in ?? () export WINEPREFIX=/data/winehome When I look in my $WINEPREFIX directory, I can see that the c: directory is not there # ls /data/winehome system.reg dosdevices drive_c user.reg # ls /data/winehome/dosdevices # How does that c: directory get created and by whom? I have deleted the $WINEPREFIX several times already to no avail. Thanks Roger R. Cruz # ls /data/winehome system.reg dosdevices drive_c user.reg # ls /data/winehome/dosdevices #

2 2

Re: cmd: Use FormatMessage() for better internationalization support.
by Frédéric Delanoy Dec. 6, 2011

Dec. 6, 2011

On Tue, Dec 6, 2011 at 17:40, Francois Gouget <fgouget(a)free.fr> wrote: > This makes it possible to reorder the format string placeholders in message translations. > Also add a WCMD_format_string() function to mirror the WCMD_output*() ones and so all resources can use the same formatting style. > This also simplifies the padding code for the 'dir /w' command. > + static const WCHAR empty[] = {'\0'}; There's already a suitable nullW string available

1 0

Re: [2/2] msi/tests: Try harder to remove a shortcut file.
by Marvin Dec. 6, 2011

Dec. 6, 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=15780 Your paranoid android. === WNT4WSSP6 (32 bit install) === install.c:5994: Test failed: Per-user icon file isn't where it's expected (C:\WINNT\Profiles\Default User\Application Data\Microsoft\Installer\{7DF88A49-996F-4EC8-A022-BF956F9B2CBB}\testicon)

1 0