January 2013 - Wine-Devel - list.winehq.org

Re: windowscodecs: Implement GetThumbnail in the ICO frame decoder.
by Hans Leidekker Jan. 30, 2013

Jan. 30, 2013

On Wed, 2013-01-30 at 18:02 +0800, Dmitry Timoshkov wrote: > diff --git a/dlls/windowscodecs/icoformat.c b/dlls/windowscodecs/icoformat.c > index 7aec245..da5a733 100644 > --- a/dlls/windowscodecs/icoformat.c > +++ b/dlls/windowscodecs/icoformat.c > @@ -200,8 +200,12 @@ static HRESULT WINAPI IcoFrameDecode_GetColorContexts(IWICBitmapFrameDecode *ifa > static HRESULT WINAPI IcoFrameDecode_GetThumbnail(IWICBitmapFrameDecode *iface, > IWICBitmapSource **ppIThumbnail) > { > - FIXME("(%p,%p)\n", iface, ppIThumbnail); > - return E_NOTIMPL; > + TRACE("(%p,%p)\n", iface, ppIThumbnail); > + > + if (!ppIThumbnail) return E_INVALIDARG; > + > + *ppIThumbnail = NULL; > + return WINCODEC_ERR_CODECNOTHUMBNAIL; > } The test added by 6395af1ae7b0cc5f2f1f82796502e2a605bc5a6b says otherwise, GetThumbnail is supported for ICO frames.

2 3

Re: [2/6] windowscodecs: Implement ComponentFactory::CreateEncoderPropertyBag
by Alexandre Julliard Jan. 29, 2013

Jan. 29, 2013

Ludger Sprenker <Ludger(a)sprenker.net> writes: > Previous "[2/5]" should have been "[3/6]" ... It doesn't build on Mac: gcc -m32 -c -I. -I. -I../../include -I../../include -D__WINESRC__ -DENTRY_PREFIX=WIC_ -DPROXY_DELEGATION -DWINE_REGISTER_DLL -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wstrict-prototypes -Wwrite-strings -gstabs+ -fno-omit-frame-pointer -Wpointer-arith -I/sw/include/freetype2 -I/sw/include -I/sw/include -g -O2 -o icnsformat.o icnsformat.c icnsformat.c: In function 'IcnsEncoder_CreateNewFrame': icnsformat.c:666: warning: passing argument 1 of 'CreatePropertyBag2' from incompatible pointer type icnsformat.c:666: error: too few arguments to function 'CreatePropertyBag2' -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: gdiplus: Avoid not necessary memory allocation for BITMAPINFO.
by Vincent Povirk Jan. 29, 2013

Jan. 29, 2013

Did you try running the tests in valgrind with this change? You're essentially reverting 0ecd8daa16e4aa3df157a1175c611cdffc66d6fe which I think was done to make valgrind happy.

1 0

Re: kernel32: dstlen < 0 causes error in codepage conversion
by Alexandre Julliard Jan. 29, 2013

Jan. 29, 2013

Alex Henrie <alexhenrie24(a)gmail.com> writes: > This is a pretty straightforward patch to resolve a couple of test > failures marked todo_wine. Hopefully it's not controversial, but if it > is, I am willing to make any reasonable modifications that are > communicated to me. If it was that easy it would have been done already. There are places in Wine that depend on being able to use negative sizes, so it would need a code review of all callers before we can make that change. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: winealsa: Have the MIDI recorder wait in poll(), not snd_seq_event_input().
by Johannes Kroll Jan. 29, 2013

Jan. 29, 2013

On Thu, 24 Jan 2013 10:33:58 +0100 <Joerg-Cyril.Hoehle(a)t-systems.com> wrote: > Compare with wineoss which uses poll <= 0 already... > > Johannes Kroll mentioned that MIDIIn takes a long time to end. > Indeed, our testsuite ran 3 times into the "kill after 5s" timeout due to this bug. > Now the tests execute much faster. Just a quick note to confirm that this patch does fix the waiting problem for me. Thanks a lot! - Johannes

1 0

Re: riched20: ME_StrDup() is no longer used so remove it.
by Huw Davies Jan. 29, 2013

Jan. 29, 2013

On Tue, Jan 29, 2013 at 07:05:13AM +0100, Francois Gouget wrote: > --- > dlls/riched20/editor.h | 1 - > dlls/riched20/string.c | 5 ----- > 2 files changed, 6 deletions(-) I'm about to send in a patch that uses this again, so please hold off on this one. Huw.

1 0

Re: winemenubuilder: Try to wait for creating icon files (try 2).
by Sergey Guralnik Jan. 29, 2013

Jan. 29, 2013

On 2013-01-23 16:34, Sergey Guralnik wrote: > It supersedes next patch: > > From 4ca51cf16c5be2bcf34dddb7f33fe043c1d777c4 Mon Sep 17 00:00:00 > 2001 > From: Sergey Guralnik <serhio(a)etersoft.ru> > Date: Tue, 22 Jan 2013 15:58:20 +0200 > Subject: winemenubuilder: Use only existing file for icon creating. > > The logic is the same as before - wait for parent process termination > if > required icon file doesn't exist yet. But now used easier way to do > this. Is something wrong with this patch? It's in queue about a week without any feedback. -- Sergey

1 0

Re: [PATCH] winebuild: Use Clang to assemble if found. (try 5)
by Alexandre Julliard Jan. 28, 2013

Jan. 28, 2013

Charles Davis <cdavis5x(a)gmail.com> writes: > Try 5: Don't use strstr(3). > > The configure script invokes the C compiler to figure out if .cfi > pseudo-ops are supported, on the assumption that it will invoke the > system assembler. Winebuild, on the other hand, invokes the system > assembler directly. This caused a problem on Mac OS hosts when compiling > with Clang: configure would detect that .cfi ops were supported, because > Clang's integrated assembler supports them; however, when winebuild then > emitted assembly with .cfi ops in them, the system assembler would then > choke. This patch makes winebuild invoke Clang's assembler if it is > found in the path. It's causing warnings on my Mac: clang: warning: /usr/bin/clang: 'linker' input unused when '-c' is present clang: warning: /usr/bin/clang: 'linker' input unused when '-c' is present -- Alexandre Julliard julliard(a)winehq.org

1 0

Why 0.1.0.0 as local socket address instead of 127.0.0.1?
by Francois Gouget Jan. 27, 2013

Jan. 27, 2013

The ws2_32:sock fails on some of my machines with the following error: sock.c:4794: Test failed: Local socket address is different 0.1.0.0 != 127.0.0.1 Does anyone know why we GetAcceptExSockaddrs() would return 0.1.0.0 instead of 127.0.0.1? Looking at the Windows 7 winetest results it does not appear to be correlated with either the SP1 or IE9. For instance got an error on Windows 7 + SP1 + IE9 (32 and 64-bit): http://test.winehq.org/data/b233d2054e64e216a5b3868f301bfc934b019b4b/win7_f… http://test.winehq.org/data/b233d2054e64e216a5b3868f301bfc934b019b4b/win7_f… But not here (Windows 7 + SP1 + IE9 64-bit): http://test.winehq.org/data/83846cd131aac33228bf08aefae0bf0ea93d4273/win7_f… I also got it failing at least once on a Windows 2000 SP4 VM: http://test.winehq.org/data/b233d2054e64e216a5b3868f301bfc934b019b4b/2000_f… But never saw it fail on my Windows XP or 2003 VMs. -- Francois Gouget <fgouget(a)codeweavers.com>

2 1

FOSDEM logistics
by Jeremy White Jan. 27, 2013

Jan. 27, 2013

Hi folks, Alright, this is it - we're going to try to get together at FOSDEM starting this Friday. If you are planning on being there, you should make sure to subscribe to the wineconf mailing list: http://www.winehq.org/mailman/listinfo/wineconf All further logistics emails and discussions will take place there. So if you're not subscribed, you can't complain we didn't try to include you in the planning. We plan on informally assembling in the lobby / bar of the Best Western Hotel Royal Centre (http://www.royalcentre.be/) on Friday, most likely starting in late afternoon. I don't have any plans for Saturday; I'll likely go harangue the Xorg guys, and maybe catch the key note. We may try to informally gather as well. However, there is no plan for Saturday dinner (sorry, no sponsored dinner by CodeWeavers this year). I have no idea how FOSDEM usually works on Saturday night, so I figured we'd just play it be year. Advice is welcome. Our devroom starts Sunday at 9:00 am. We're in building K, which you can see on this map: http://www.ulb.ac.be/campus/solbosch/plan-K.html Main FOSDEM entrance / registration is in building H on that map. One bit of unsolicited advice - when I naively copy/paste the address on the FOSDEM page into Google maps for directions, I end up with the wrong location. If you're at the Best Western, it looks like Tram 94 goes to the Legrand stop in 16 minutes, and then a 10 minute walk carries you the rest of the way there. Looking forward to seeing you all on Friday! Cheers, Jeremy

1 0

Re: wined3d: Rebind texture before checking for its content in check_fbo_compat().
by Stefan Dösinger Jan. 27, 2013

Jan. 27, 2013

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Am 2013-01-26 22:32, schrieb Matteo Bruni: > + > gl_info->fbo_ops.glFramebufferTexture2D(GL_FRAMEBUFFER, > GL_COLOR_ATTACHMENT0, GL_TEXTURE_2D, tex, 0); Do you really have to re-attach it to the FBO? Or does glBindTexture(GL_TEXTURE_2D, tex) work as well? -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQIcBAEBAgAGBQJRBFTlAAoJEN0/YqbEcdMw7bcQAIfUM81xBr3zIpcEYek3gMso jXPq49+8Dp8Y3xiJGU/nFD8yLH9Y24SXpyageueCMWZR2jnEQcXBL0l76MmL5ZYK Y/L1ytRwrOvdXP07pJ8p3ZXxmA6hl82fuNxMvYhlSYP8PVCoobhaH7MwGp9WPpGF Ij20eOOR3GQnDjb6M2qHHfP9s+mNREu5NrvZ76NCf/lURXLJyDECuMz+6TbSl4CM qv0xCKdtqxm13gTZio3ftKc3QZZUSstiYjUYQmFh5NXQe5oIr9j2/iueU2K0zA/M CqswN4divbFFbh7QwCIDaubwfGidRfiwIG3lCCiNpfIvnowQouELIoFKjIipQ1AL ihIIS7iyekasAxkErVZ3E8O8IJCKYbTC+tAXBfK3niKmSRfdUB9cmfo5nVZw4Xjb 3/Zz84WnjMlIznyBGsKyKX+VLKlqzXY5zCyboHuEWPxTXdS0FAbwHzz06Kv0zIqc Pe4X6dJOUz6EPnx2gkbtFygGFbAa8haWhiIcY01H1bMSa7NnZPU3QgQXimrvXaKK SbQofZfhbNSVv0dL1kxqUYeLPb5hKXqMRHO7yi/GeyejTie9onzjxrjt4zn4rxe3 9IEZ0ymUYs499ahUnW/DAe0yFJvX+EXiPIiWwLBGVpx/Dzduk0WADOLBaN/YwrEz 1gxkuQjF2tbVSTkP41yr =xi8z -----END PGP SIGNATURE-----

3 3

Implement D3DXSHEvalConeLight
by Nozomi Kodama Jan. 26, 2013

Jan. 26, 2013

Hello what is the problem with the patch http://source.winehq.org/patches/data/93498 Nozomi

1 0

Re: [PATCH] gdi32: Adjust a fixed-pitch full-width character has double advance of a half-width character. (try 5)
by Alexandre Julliard Jan. 26, 2013

Jan. 26, 2013

Akihiro Sagawa <sagawa.aki(a)gmail.com> writes: > Hi, > > This patch enhances full-width character glyph advance when we choose > fixed-pitch font. > > Windows 2000 (or XP for Japanese) and later adjust fixed-pitch > full-width character glyph advances so that two half-width characters' > advance equals to one full-width character's advance. Because CJK > fixed-pitch font, such as MS Gothic, is designed as full-width character > (e.g. Kanji) has a double advance of half-width character (e.g. ASCII) > and many developpers expect that. > > Please refer Microsoft document KB 417434 [1] and white paper [2] for > details. > > [1] ... http://support.microsoft.com/kb/417434/ja (written in Japanese) > [2] ... http://msdn.microsoft.com/ja-jp/library/cc979401.aspx (written in Japanese) > > try 5: > - Add patch description above. > - Fix avgAdvance's rounding method. (as in get_outline_text_metrics) > - Refine a commit message, "Ensure" to "Adjust" and "double width" to > "double advance". > > try 4: > - Fix full-width character detection that causes test failure. > > try 3: > - Improve full-width character detection. > - Support fake italic transformation. > - Choose non-linked fonts in the test case. > - Add more trace messages. > > try 2: > - Add rotated transformation tests. > > Regards, > Akihiro Sagawa > > --- > dlls/gdi32/freetype.c | 37 ++++++++++++++++++-- > dlls/gdi32/tests/font.c | 87 +++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 121 insertions(+), 3 deletions(-) This is still failing here: ../../../tools/runtest -q -P wine -M gdi32.dll -T ../../.. -p gdi32_test.exe.so font.c && touch font.ok font.c:2549: Tests skipped: Font Droid Arabic Naskh doesn't contain 'x', skipping the test font.c:2549: Tests skipped: Font Droid Sans Armenian doesn't contain 'x', skipping the test font.c:2549: Tests skipped: Font Droid Sans Ethiopic doesn't contain 'x', skipping the test font.c:2549: Tests skipped: Font Droid Sans Georgian doesn't contain 'x', skipping the test font.c:2549: Tests skipped: Font Droid Sans Hebrew doesn't contain 'x', skipping the test font.c:2549: Tests skipped: Font Droid Sans Hebrew doesn't contain 'x', skipping the test font.c:3008: Tests skipped: Unable to retrieve first and last glyphs from cmap font.c:2549: Tests skipped: Font Droid Sans Japanese doesn't contain 'x', skipping the test font.c:2549: Tests skipped: Font Droid Sans Thai doesn't contain 'x', skipping the test font.c:2549: Tests skipped: Font OpenSymbol doesn't contain 'x', skipping the test font.c:2549: Tests skipped: Font rsfs10 doesn't contain 'x', skipping the test font.c:3756: Test failed: expected 19, got 0 (@IPAGothic:128) make[1]: *** [font.ok] Error 1 -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [PATCH] winebuild: Use Clang to assemble if found. (try 4)
by Alexandre Julliard Jan. 25, 2013

Jan. 25, 2013

Charles Davis <cdavis5x(a)gmail.com> writes: > @@ -346,13 +352,26 @@ struct strarray *get_as_command(void) > struct strarray *args = strarray_init(); > > if (!as_command) > + as_command = find_tool( "clang", NULL ); > + > + if (!as_command) > { > static const char * const commands[] = { "gas", "as", NULL }; > as_command = find_tool( "as", commands ); > } > + > + if (!as_command) > + fatal_error( "cannot find suitable assembler\n" ); > + > strarray_add_one( args, as_command ); > > - if (force_pointer_size) > + if (strstr( as_command, "clang" )) It's looking better, but strstr is not appropriate here. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 2/2] jscript: Fixed a leak in regexp_match function
by Marcus Meissner Jan. 25, 2013

Jan. 25, 2013

On Fri, Jan 25, 2013 at 04:26:59PM +0100, Piotr Caban wrote: > --- > dlls/jscript/regexp.c | 11 ++++++++--- > 1 file changed, 8 insertions(+), 3 deletions(-) > > > diff --git a/dlls/jscript/regexp.c b/dlls/jscript/regexp.c > index 0d66551..fa2668c 100644 > --- a/dlls/jscript/regexp.c > +++ b/dlls/jscript/regexp.c > @@ -3466,10 +3466,15 @@ static HRESULT regexp_match(script_ctx_t *ctx, jsdisp_t *dispex, jsstr_t *str, B > break; > > if(ret_size == i) { > - if(ret) > - ret = heap_realloc(ret, (ret_size <<= 1) * sizeof(match_result_t)); > - else > + if(ret) { > + match_result_t *old_ret = ret; > + > + ret = heap_realloc(old_ret, (ret_size <<= 1) * sizeof(match_result_t)); > + if(!ret) > + heap_free(old_ret); Should be probably if (ret) heap_free(old_ret); here. > + }else { > ret = heap_alloc((ret_size=4) * sizeof(match_result_t)); > + } > if(!ret) { > hres = E_OUTOFMEMORY; > break; > >

2 1

Re: [1/5] comctl32/tests: more tests for icon spacing calculation
by Alexandre Julliard Jan. 25, 2013

Jan. 25, 2013

Daniel Jelinski <djelinski1(a)gmail.com> writes: > +#ifdef _WIN64 > + /* NOTE: -1 is not treated the same as (DWORD)-1 by 64bit listview */ > + ret = SendMessage(hwnd, LVM_SETICONSPACING, 0, (DWORD)-1); > +todo_wine { > + expect(100, LOWORD(ret)); > + expect(0xFFFF, HIWORD(ret)); > +} > + ret = SendMessage(hwnd, LVM_SETICONSPACING, 0, 0xBAADF00DDEADBEEFLL); Please avoid ifdefs and long long constants. -- Alexandre Julliard julliard(a)winehq.org

3 2

Wine Wiki needs your help!
by Dimi Paun Jan. 25, 2013

Jan. 25, 2013

Folks, As we all know, through the years spam has been an ongoing problem. We dealt with it OK, as we valued the openness of our wiki. That's good. However, all this churn has accumulated a lot of garbage on the system, as orphaned pages and dummy users. These have accumulated to such an extent that we are reaching system limits. I need your help to clean them out. First, we have to separate the spam from the real content. I've created a Google Spreadsheet for pages: https://docs.google.com/a/lattica.com/spreadsheet/ccc?key=0AmY-Kp_Ihu3idFNE… and one for users: https://docs.google.com/a/lattica.com/spreadsheet/ccc?key=0AmY-Kp_Ihu3idGdP… We need to mark spam as "Spam" and what's good as "OK" in the "Spam" column. I'll take that data and apply it to the wiki. Please let me know if we can do this any simpler or if there are any problems. Thanks! -- Dimi Paun Lattica, Inc.

8 18

Re: [1/6] windowscodecs: Add tests for IPropertyBag2
by Vincent Povirk Jan. 24, 2013

Jan. 24, 2013

Other than the 6/6, this series looks good to me. Thanks for breaking this up. It makes it much easier to review.

1 0

Re: [6/6] windowscodecs: Implement IPropertyBag2::Read
by Vincent Povirk Jan. 24, 2013

Jan. 24, 2013

+ if (idx > -1) + { + VariantInit(pvarValue+i); + res = VariantCopy(pvarValue+i, This->values+idx); + phrError[i] = res; + if (FAILED(res)) + break; + } This looks wrong to me. We have an array of error values to set for each read attempt, so it seems like each read should be independent - one failure shouldn't cause us to give up on the others, nor to free the values we already read successfully. If this is really how Windows works, we need a test for it.

1 0

Re: [5/6] windowscodecs: Implement IPropertyBag2::Write
by Vincent Povirk Jan. 24, 2013

Jan. 24, 2013

if (This->properties[i].pstrName) { HeapFree(GetProcessHeap(), 0, This->properties[i].pstrName); + VariantClear( This->values+i ); /* Not initialized if pstrName not initialized */ } ... { This->properties = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(PROPBAG2)*count); - - if (!This->properties) + This->values = HeapAlloc(GetProcessHeap(), 0, sizeof(VARIANT)*count); + + if (!This->properties || !This->values) res = E_OUTOFMEMORY; else for (i=0; i < count; i++) Well, that explains why you had the NULL check in the previous patch. If you allocate values with HEAP_ZERO_MEMORY, you don't have to use VariantInit, and you could simplify your Release code (at least in terms of the effort required to verify that it is correct). I think the way it is now is technically correct though. + { + if (pPropBag[i].pstrName) + FIXME("Application tried to set the unknown option %s.\n", + debugstr_w(pPropBag[i].pstrName)); + /* FIXME: Function is not atomar on error, but MSDN does not say anything about it + * (no reset of items between 0 and i-1) */ + return E_FAIL; + } Bit of an indentation glitch there - the return statement isn't (and shouldn't be) in the if statement.

1 0

Re: [4/6] windowscodecs: Implement IPropertyBag2::GetPropertyInfo
by Vincent Povirk Jan. 24, 2013

Jan. 24, 2013

+ if (This->properties[i].pstrName) + { + HeapFree(GetProcessHeap(), 0, This->properties[i].pstrName); + } HeapFree already checks for NULL, so there's no need for an if statement here.

1 0

Re: [PATCH] ieframe/tests: Don't convert to long path name until the file exists
by Marvin Jan. 24, 2013

Jan. 24, 2013

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=24128 Your paranoid android. === WNT4WSSP6 (32 bit webbrowser) === Timeout === W2KPROSP4 (32 bit webbrowser) === webbrowser.c:2607: Test failed: expected GetOverridesKeyPath webbrowser.c:2612: Test failed: expected Invoke_SETSECURELOCKICON webbrowser.c:2613: Test failed: expected Invoke_FILEDOWNLOAD webbrowser.c:3046: Test failed: doc_disp == NULL webbrowser: unhandled exception c0000005 at 00402F21

1 0

Re: ws2_32: Use assignment instead of memcpy to copy structs.
by Michael Stefaniuc Jan. 24, 2013

Jan. 24, 2013

Forgot to add the credit for this patch series as I didn't wrote this coccinelle script: Found using the memcpy-assign.cocci script submitted for inclusion into the Linux kernel: https://systeme.lip6.fr/pipermail/cocci/2013-January/000121.html bye michael On 01/24/2013 02:21 PM, Michael Stefaniuc wrote: > --- > dlls/ws2_32/socket.c | 2 +- > dlls/ws2_32/tests/sock.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/dlls/ws2_32/socket.c b/dlls/ws2_32/socket.c > index c390828..6fd1e34 100644 > --- a/dlls/ws2_32/socket.c > +++ b/dlls/ws2_32/socket.c > @@ -2183,7 +2183,7 @@ static BOOL interface_bind( SOCKET s, int fd, struct sockaddr *addr ) > > if (setsockopt(fd, IPPROTO_IP, IP_UNICAST_IF, &ifindex, sizeof(ifindex)) != 0) > goto cleanup; /* Failed to suggest egress interface */ > - memcpy(&specific_interface_filter, &generic_interface_filter, sizeof(generic_interface_filter)); > + specific_interface_filter = generic_interface_filter; > specific_interface_filter.iface_rule.k = adapter->Index; > filter_prog.len = sizeof(generic_interface_filter)/sizeof(struct sock_filter); > filter_prog.filter = (struct sock_filter *) &specific_interface_filter; > diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c > index 56a64bc..4b4b99a 100644 > --- a/dlls/ws2_32/tests/sock.c > +++ b/dlls/ws2_32/tests/sock.c > @@ -2725,7 +2725,7 @@ static void test_getsockname(void) > return; > } > > - memcpy(&sa_get, &sa_set, sizeof(sa_set)); > + sa_get = sa_set; > if (getsockname(sock, (struct sockaddr*) &sa_get, &sa_get_len) == 0) > ok(0, "getsockname on unbound socket should fail\n"); > else {

1 0

Re: ieframe: Fixed a race in tests
by Marvin Jan. 24, 2013

Jan. 24, 2013

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=24121 Your paranoid android. === WNT4WSSP6 (32 bit webbrowser) === Timeout === W2KPROSP4 (32 bit webbrowser) === webbrowser.c:2607: Test failed: expected GetOverridesKeyPath webbrowser.c:2612: Test failed: expected Invoke_SETSECURELOCKICON webbrowser.c:2613: Test failed: expected Invoke_FILEDOWNLOAD webbrowser.c:3046: Test failed: doc_disp == NULL webbrowser: unhandled exception c0000005 at 00402F21

1 0

Re: [PATCH 2/2] vbscript: Added GetObject tests
by Marvin Jan. 24, 2013

Jan. 24, 2013

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=24115 Your paranoid android. === W2KPROSP4 (32 bit createobj) === createobj.c:999: Test failed: ParseScriptText failed: 8007007e createobj.c:1000: Test failed: expected QI_IObjectWithSite createobj.c:1001: Test failed: expected SetSite createobj.c:1002: Test failed: expected reportSuccess createobj.c:1008: Test failed: hres = 8007007e createobj.c:1009: Test failed: expected QI_IObjectWithSite createobj.c:1010: Test failed: expected SetSite createobj.c:1015: Test failed: ParseScriptText failed: 8007007e createobj.c:1016: Test failed: expected QI_IObjectWithSite createobj.c:1017: Test failed: expected reportSuccess

1 0