- wine-devel - winehq.org

Re: [2/2] kernel32: Implement InitOnceExecuteOnce() for Linux platform
by Marvin 24 Jul '12

24 Jul '12

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://winetestbot.dolphin/JobDetails.pl?Key=55 Your paranoid android. === debiantesting (build) === Patch failed to apply

1 0

Re: [1/2] kernel32: Some tests for blocking initialization with InitOnceBeginInitialize()
by Marvin 24 Jul '12

24 Jul '12

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://winetestbot.dolphin/JobDetails.pl?Key=54 Your paranoid android. === debiantesting (build) === Patch failed to apply

1 0

What's the right way to implement _wtoi_l?
by Alex Henrie 24 Jul '12

24 Jul '12

Dear Wine developers, Bug 24389 documents an application crash due to the unimplemented function msvcr90._wtoi_l. I have been looking into how to resolve this problem. I see three workable options: 1. Implement msvcr90._wtoi_l by copying the code from ntdll._wtoi to msvcrt, ignoring the locale parameter for now. 2. Forward msvcr90._wtoi_l to ntdll._wtoi via msvcr90.spec, ignoring the locale parameter for now. 3. Implement msvcr90._wtoi_l as a stub function in msvcrt that always returns 0. Any of these options would fix bug 24389. Which one should I work on? -Alex

3 3

Re: msvcp90: Explicitly cast to signed char
by André Hentschel 24 Jul '12

24 Jul '12

Am 24.07.2012 21:30, schrieb André Hentschel: > --- > dlls/msvcp90/ios.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) This has been obsoleted due to Piotrs changes, thx. -- Best Regards, André Hentschel

1 0

Re: [4/5] gdiplus: Add some tests for GdipGetPropertyItemSize and GdipGetPropertyItem. Take 2.
by Marvin 24 Jul '12

24 Jul '12

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://winetestbot.dolphin/JobDetails.pl?Key=44 Your paranoid android. === debiantesting (build) === Patch failed to apply

1 0

Re: [1/5] windowscodecs: Implement MetadataHandler_GetValue. Take 2.
by Marvin 24 Jul '12

24 Jul '12

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://winetestbot.dolphin/JobDetails.pl?Key=43 Your paranoid android. === debiantesting (build) === Patch failed to apply

1 0

Re: [PATCH 2/3] mshtml: Added IHTMLDOMAttribute::get_specified implementation
by Marvin 24 Jul '12

24 Jul '12

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://winetestbot.dolphin/JobDetails.pl?Key=41 Your paranoid android. === windowsxp (32 bit dom) === Can't copy TestLauncher to VM: :7094

1 0

Re: [01/18] windowscodecs: Do not assume that vtable is the first element of the object.
by Vincent Povirk 24 Jul '12

24 Jul '12

I disagree with many of the choices in this series (for example, I think casting an interface pointer to an inherited interface is fine, and replacing that with QueryInterface just adds unnecessary additional logic), but I don't think any semantics have been changed and overall it's a good cleanup.

3 4

Re: gdiplus: Store device resolution in the graphics object.
by Vincent Povirk 24 Jul '12

24 Jul '12

Looks good, thanks.

1 0

Re: windowscodecs: Implement MetadataHandler_Load.
by Vincent Povirk 24 Jul '12

24 Jul '12

Looks good, thanks.

1 0

Re: [PATCH 6/6] jscript: Use prototype for buitin String properties
by Marvin 24 Jul '12

24 Jul '12

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=20282 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 === 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: [PATCH 5/6] jscript: Use prototype for buitin RegExp properties
by Marvin 24 Jul '12

24 Jul '12

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=20281 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 === 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: [PATCH 4/6] jscript: Use prototype for buitin Number properties
by Marvin 24 Jul '12

24 Jul '12

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=20280 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 === 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: [2/2] kernel32: Implement InitOnceExecuteOnce() for Linux platform (resend)
by Alexandre Julliard 24 Jul '12

24 Jul '12

Nikolay Sivov <nsivov(a)codeweavers.com> writes: > +DWORD NTAPI RtlRunOnceExecuteOnce(PRTL_RUN_ONCE initonce, PRTL_RUN_ONCE_INIT_FN callback, void *parameter, void **ctxt) > +{ > +#ifdef __linux__ > + if (!use_futexes()) return FALSE; > + > + for (;;) > + { > + DWORD_PTR val = (DWORD_PTR)interlocked_cmpxchg_ptr( &initonce->Ptr, (void*)1, NULL ); > + switch (val & 0x3) > + { > + case 0: > + { > + DWORD ret = callback(initonce, parameter, ctxt); > + if (ret) > + { > + DWORD_PTR context = ctxt ? (DWORD_PTR)*ctxt : 0; > + context = (context & ~0x3) | 0x2; > + interlocked_cmpxchg_ptr( &initonce->Ptr, (void*)context, (void*)1 ); > + } > + else > + interlocked_cmpxchg_ptr( &initonce->Ptr, NULL, (void*)1 ); > + futex_wake( (int *)&initonce->Ptr, 1 ); > + return ret; > + } > + case 1: > + /* blocked by another thread */ > + futex_wait( (int *)&initonce->Ptr, 1, NULL ); > + break; > + case 2: > + /* already initialized */ > + return TRUE; > + } > + } It's supposed to return an NTSTATUS, not a boolean. Also you have to always return the context, and you should try to do a generic implementation first. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 3/6] jscript: Use prototype for buitin Date properties
by Marvin 24 Jul '12

24 Jul '12

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=20279 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 === 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: [PATCH 3/3] apphelp/tests: Add tests for ApphelpCheckShellObject
by Alexandre Julliard 24 Jul '12

24 Jul '12

Detlef Riekenberg <wine.dev(a)web.de> writes: > + flags = 0xdeadbeeffeedfaceULL; Don't use long long constants. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 2/6] jscript: Use prototype for buitin Boolean properties
by Marvin 24 Jul '12

24 Jul '12

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=20278 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 === 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: [PATCH 1/6] jscript: Use prototype for buitin Array properties
by Marvin 24 Jul '12

24 Jul '12

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=20277 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 === 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: [01/18] windowscodecs: Do not assume that vtable is the first element of the object.
by Michael Stefaniuc 24 Jul '12

24 Jul '12

Hello Dmitry, On 07/24/2012 08:01 AM, Dmitry Timoshkov wrote: > The patches in this series do not depend on each other, numeration is just > for a convenience. Patches do basically the same job for different objects, > so I decided to not invent a new subject for every separate kind of object. > --- > dlls/windowscodecs/bmpdecode.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/dlls/windowscodecs/bmpdecode.c b/dlls/windowscodecs/bmpdecode.c > index 6da6a3c..1a1ee0a 100644 > --- a/dlls/windowscodecs/bmpdecode.c > +++ b/dlls/windowscodecs/bmpdecode.c > @@ -94,6 +94,8 @@ static inline BmpDecoder *impl_from_IWICBitmapFrameDecode(IWICBitmapFrameDecode > static HRESULT WINAPI BmpFrameDecode_QueryInterface(IWICBitmapFrameDecode *iface, REFIID iid, > void **ppv) > { > + BmpDecoder *This = impl_from_IWICBitmapFrameDecode(iface); > + > TRACE("(%p,%s,%p)\n", iface, debugstr_guid(iid), ppv); > > if (!ppv) return E_INVALIDARG; > @@ -102,7 +104,7 @@ static HRESULT WINAPI BmpFrameDecode_QueryInterface(IWICBitmapFrameDecode *iface > IsEqualIID(&IID_IWICBitmapSource, iid) || > IsEqualIID(&IID_IWICBitmapFrameDecode, iid)) > { > - *ppv = iface; > + *ppv = &This->IWICBitmapFrameDecode_iface; > } > else > { this part of the change is gratuitous. As long as there is only one interface implementation in the object there is no need for impl_from_Foo(). bye michael

2 1

Re: [12/18] windowscodecs: Do not assume that vtable is the first element of the object, avoid not necessary casts.
by Michael Stefaniuc 24 Jul '12

24 Jul '12

Hello Dmitry, On 07/24/2012 08:12 AM, Dmitry Timoshkov wrote: > --- > dlls/windowscodecs/jpegformat.c | 23 +++++++++++++---------- > 1 file changed, 13 insertions(+), 10 deletions(-) > > diff --git a/dlls/windowscodecs/jpegformat.c b/dlls/windowscodecs/jpegformat.c > index 53d7ec4..2c77d4f 100644 > --- a/dlls/windowscodecs/jpegformat.c > +++ b/dlls/windowscodecs/jpegformat.c > @@ -469,6 +470,8 @@ static const IWICBitmapDecoderVtbl JpegDecoder_Vtbl = { > static HRESULT WINAPI JpegDecoder_Frame_QueryInterface(IWICBitmapFrameDecode *iface, REFIID iid, > void **ppv) > { > + JpegDecoder *This = impl_from_IWICBitmapFrameDecode(iface); > + > TRACE("(%p,%s,%p)\n", iface, debugstr_guid(iid), ppv); > > if (!ppv) return E_INVALIDARG; > @@ -477,7 +480,7 @@ static HRESULT WINAPI JpegDecoder_Frame_QueryInterface(IWICBitmapFrameDecode *if > IsEqualIID(&IID_IWICBitmapSource, iid) || > IsEqualIID(&IID_IWICBitmapFrameDecode, iid)) > { > - *ppv = iface; > + *ppv = &This->IWICBitmapFrameDecode_iface; > } > else > { same here: not needed. bye michael

1 0

Re: [2/2] kernel32: Implement InitOnceExecuteOnce() for Linux platform
by Marvin 24 Jul '12

24 Jul '12

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=20269 Your paranoid android. === WVISTAADM (32 bit sync) === sync.c:1231: Test failed: got 0 === W7PRO (32 bit sync) === sync.c:1231: Test failed: got 0 === W7PROX64 (32 bit sync) === sync.c:1231: Test failed: got 0 === TEST64_W7SP1 (32 bit sync) === sync.c:1231: Test failed: got 0 === W7PROX64 (64 bit sync) === sync.c:1231: Test failed: got 0 === TEST64_W7SP1 (64 bit sync) === sync.c:1231: Test failed: got 0

1 0

Re: [1/2] kernel32: Some tests for blocking initialization with InitOnceBeginInitialize()
by Marvin 24 Jul '12

24 Jul '12

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=20268 Your paranoid android. === WVISTAADM (32 bit sync) === sync.c:1225: Test failed: got 0 === W7PRO (32 bit sync) === sync.c:1225: Test failed: got 0 === W7PROX64 (32 bit sync) === sync.c:1225: Test failed: got 0 === TEST64_W7SP1 (32 bit sync) === sync.c:1225: Test failed: got 0 === W7PROX64 (64 bit sync) === sync.c:1225: Test failed: got 0 === TEST64_W7SP1 (64 bit sync) === sync.c:1225: Test failed: got 0

1 0

re: Have there been any problems with Wine on GCC 4.7?
by Dan Kegel 24 Jul '12

24 Jul '12

Groveling around, I found a few possibilities: backtraces broken on gcc 4.7 http://bugs.winehq.org/show_bug.cgi?id=26791 ICE when compiling: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=641056 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=640864 https://bugs.launchpad.net/ubuntu/+source/gcc-4.6/+bug/842012 needs-new-prelink-to-avoid-crash-with -g option: http://bugs.winehq.org/show_bug.cgi?id=30862 steam overlays: http://bugs.winehq.org/show_bug.cgi?id=22053 enabling -fomit-frame-pointer by default breaks Wine: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46468 gcc shouldn't assume that the stack is aligned: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40838 wine's conformance test kernel32/thread.ok always fails/crashes on ubuntu 11.10: https://bugs.launchpad.net/ubuntu/+source/wine1.3/+bug/872063 http://wiki.winehq.org/GccVersions might be a place to record this kind of stuff, but it's outdated. There are some known fixes, possibly related to the above problems, e.g. http://bugs.winehq.org/show_bug.cgi?id=27692 http://bugs.winehq.org/show_bug.cgi?id=28753 http://bugs.winehq.org/show_bug.cgi?id=23137 And then there's debian's "wine uses an old gcc" bug: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=654742

1 0

Re: kernel32: Reimplement GetLogicalDrives using the mountmgr
by Alexandre Julliard 23 Jul '12

23 Jul '12

Alexandre Goujon <ale.goujon(a)gmail.com> writes: > Previous approach was patch #88280 > --- > dlls/kernel32/volume.c | 18 +++--------------- > 1 file changed, 3 insertions(+), 15 deletions(-) > > diff --git a/dlls/kernel32/volume.c b/dlls/kernel32/volume.c > index 8377e41..57453bb 100644 > --- a/dlls/kernel32/volume.c > +++ b/dlls/kernel32/volume.c > @@ -1435,27 +1435,15 @@ DWORD WINAPI QueryDosDeviceA( LPCSTR devname, LPSTR target, DWORD bufsize ) > */ > DWORD WINAPI GetLogicalDrives(void) > { > - const char *config_dir = wine_get_config_dir(); > - struct stat st; > - char *buffer, *dev; > DWORD ret = 0; > + WCHAR drive; > int i; > > - if (!(buffer = HeapAlloc( GetProcessHeap(), 0, strlen(config_dir) + sizeof("/dosdevices/a:") ))) > - { > - SetLastError( ERROR_NOT_ENOUGH_MEMORY ); > - return 0; > - } > - strcpy( buffer, config_dir ); > - strcat( buffer, "/dosdevices/a:" ); > - dev = buffer + strlen(buffer) - 2; > - > for (i = 0; i < 26; i++) > { > - *dev = 'a' + i; > - if (!stat( buffer, &st )) ret |= (1 << i); > + drive = 'A'+i; > + if (get_mountmgr_drive_type(&drive) != DRIVE_UNKNOWN) ret |= (1 << i); > } That's very inefficient. There are better way of enumerating drives, check for instance QueryDosDevice. -- Alexandre Julliard julliard(a)winehq.org

2 4

Query MediaType of SD-Cards using IOCTL_GET_DISK_DRIVE_GEOMETRY
by Eduard Hasenleithner 23 Jul '12

23 Jul '12

Hi I'm trying to use the TANITA Bodyvision software (http://www.tanita.co.uk/index.php?id=158) under wine. The software is supposed to read its data from an SD-Card. But under wine it cannot find the SD card. I managed to find out what it takes so that the software finds the SD card. Please see the attached small sample to see how the software iterates the drive letters, and how it determines a eligibile media using (data.MediaType == RemovableMedia). The reasons why this does not work are manyfold. Here are some events which happen when inserting the card. * The drive is reported using udisks. This creates a symlink in dosdevices to the raw block device /dev/sdf * The partition /dev/sdf1 is mounted automatically in /media/XXXX-YYYY * /dev/sdf and /dev/sdf1 are not user read/writable. * wine creates a drive letter symlink for /dev/sdf * no symlink is created for /media/XXXX-YYYY * When you open wine explorer, the drive does not show up in the drive list This means that the drive letter assigned for the sd card will not be returned in GetLogicalDrives(). When the symlink to the mountpoint is added manually, following output is produced: DbgPrint says: Drive f fixme:ntdll:server_ioctl_file Unsupported ioctl 70000 (device=7 access=0 func=0 method=0) DbgPrint says: DeviceIoControl failed When get_dos_device is made to unconditionally return STATUS_BAD_DEVICE_TYPE, the ioctl is properly delivered to harddisk_ioctl DeviceIoControl succeeds: DbgPrint says: Drive f DbgPrint says: Removable: 0 But still, the device is reported as non-removable. The reason for this is probably that udisks reports /dev/sdf as removable, but not its partition /dev/sdf1. Is it worth for working on patches for these issues, or is this completely impossible with the current architecture? Eduard

3 7

Re: [1/2] msvcrt: added wmemcpy_s and wmemmove_s
by Alexandre Julliard 23 Jul '12

23 Jul '12

Dan Kegel <dank(a)kegel.com> writes: > Next patch will expose them for msvcr100. > > From 357c45971e1315e8f6cd1f9feac65cf96a006a41 Mon Sep 17 00:00:00 2001 > From: Dan Kegel <dank(a)kegel.com> > Date: Fri, 20 Jul 2012 10:39:32 -0700 > Subject: [PATCH 1/2] msvcrt: add wmemcpy_s and wmemmove_s > > --- > dlls/msvcrt/heap.c | 29 ++++++++++++++++++++++++ > dlls/msvcrt/msvcrt.spec | 2 + > dlls/msvcrt/tests/misc.c | 54 ++++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 85 insertions(+), 0 deletions(-) You shouldn't need to put that in msvcrt. Also please check the parameters correctly like every other _s function does. -- Alexandre Julliard julliard(a)winehq.org

2 3

Re: Dmitry Timoshkov : gdiplus: GdipMeasureString operates internally in pixels but in/ out rectangles are in device units.
by Vincent Povirk 23 Jul '12

23 Jul '12

> gdiplus: GdipMeasureString operates internally in pixels but in/out rectangles are in device units. I still don't understand this patch. Your subject implies that device units and pixels are not the same thing, but my understanding is that except for metafile HDCs (which we don't handle correctly anyway, and I suspect native doesn't either), they are. You're scaling sizes based on graphics->unit. Yet the conversion in get_graphics_transform (which we were already using in GdipMeasureString to convert between world and device units, indirectly through GdipTransformPoints) already takes graphics->unit into account. So I'm concerned that the original conversion apparently wasn't sufficient, and that you've added more code instead of fixing it. And I'm still concerned about whether the sizes returned by GdipMeasureString will match the size of the strings rendered by GdipDrawString with this patch applied.

2 6

Re: [1/6] windowscodecs: Add support for generating WICBitmapPaletteTypeFixedHalftone27 palette.
by Vincent Povirk 23 Jul '12

23 Jul '12

Series looks good to me.

1 0

Re: [PATCH 4/4] mshtml: Added IHTMLFrameBase::frameBorder implementation
by Marvin 23 Jul '12

23 Jul '12

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=20256 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [PATCH 4/4] vbscript: Added select case statement tests
by Marvin 23 Jul '12

23 Jul '12

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=20253 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: atl80: Implement AtlGetVersion - try2
by Dmitry Timoshkov 23 Jul '12

23 Jul '12

Vijay Kiran Kamuju <infyquest(a)gmail.com> wrote: > +WINAPI DWORD AtlGetVersion(void* pReserved) That's a somewhat strange choice for the placement of a calling convention specifier. Also it shouldn't be too hard to avoid a hungarian style named variable, and placing '*' properly. -- Dmitry.

2 2

Re: configure.ac: Always treat char as signed char on ARM
by Alexandre Julliard 23 Jul '12

23 Jul '12

André Hentschel <nerv(a)dawncrow.de> writes: > Problem: > On ARM the default "char" is "unsigned char" for performance reasons, > Wine uses "char" expecting the x86 default (signed char). In general there's no such expectation in Wine. If there are places that expect char to be signed, they are bugs that should be fixed. -- Alexandre Julliard julliard(a)winehq.org

1 0

Noticed openwatcom.org's afxapi
by Dan Kegel 22 Jul '12

22 Jul '12

"Initial checkin of afxapi, a class library aiming to provide source-level compatibility with MFC." http://www.openwatcom.org:4000/@md=d&cd=//depot/openwatcom/bld/afxapi/inclu… Wonder how that's going.

1 0

Error When Using winedbg on 64-bit Ubuntu 12.04
by John Edmonds 22 Jul '12

22 Jul '12

I'm using 64-bit Ubuntu 12.04. I built and installed wine using the instructions from http://wiki.winehq.org/WineOn64bit. Whenever I try to debug an application using winedbg I get the following error message: > /var/chroot/usr/local/bin/winedbg ~/.wine/drive_c/windows/system32/notepad.exe err:module:LdrInitializeThunk Main exe initialization for L"C:\\windows\\system32\\notepad.exe" failed, status c0000022 WineDbg starting on pid 0023 couldn't load main module (2) Process of pid=0023 has terminated This is the output when the same command is run with WINEDEBUG=warn+all: http://pastebin.com/RjErdbt8. My source tree is synced to git commit 020a68e1350eaf7293718732c1c9203e460709c6. Has anyone had success debugging Windows applications on 64-bit Linux? Thanks, John Edmonds

3 2

Re: [PATCH 3/5] wined3d: Don't use GL_EXTCALL for WGL extension functions.
by Henri Verbeet 22 Jul '12

22 Jul '12

Actually, ignore this one, it shouldn't be needed anymore.

1 0

Re: mpr/tests: Сheck the return values of the WNetGetUniversalNameA. (try 2)
by Nikolay Sivov 20 Jul '12

20 Jul '12

On 7/20/2012 16:02, Roman Dadkov wrote: > This patch adds additional check for the returned results in the test > for the function WNetGetUniversalNameA. > The following cases are checked: > 1 transmission of an incorrect buffer size > 2 transmission of an incorrect buffer pointer > 3 transmission path to the file > Similar checks are carried out to call a function with parameter > REMOTE_NAME_INFO_LEVEL. > > + fail_size = sizeof(driveA) / sizeof(char) - 1; // buffer size is less than the path I don't know if this will be accepted as is, but anyway don't use cpp style comments in future. > + if(drive_type == DRIVE_REMOTE) > + todo_wine ok(ret == WN_NO_ERROR, "WNetGetUniversalNameW failed: %08x\n", ret); > + else > + todo_wine ok(ret == ERROR_NOT_CONNECTED, "WNetGetUniversalNameW gave wrong error: %08x\n", ret); > This is mostly matter of taste but in such cases you can use todo_wine {} and place all test call inside.

1 0

Re: mpr/tests: Сheck the return values of the WNetGetUniversalNameA. (try 2)
by Marvin 20 Jul '12

20 Jul '12

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=20185 Your paranoid android. === W2KPROSP4 (32 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca === WXPPROSP3 (32 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca === W2K3R2SESP2 (32 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca === WVISTAADM (32 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca === W7PRO (32 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca === W7PROX64 (32 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca === TEST64_W7SP1 (32 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca === W7PROX64 (64 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca === TEST64_W7SP1 (64 bit mpr) === mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca mpr.c:121: Test failed: WNetGetUniversalNameA failed: 000008ca

1 0

Re: gdiplus: Set actual resolution of a being encoded image.
by Vincent Povirk 20 Jul '12

20 Jul '12

Looks good, thanks.

1 0

Re: mpr/tests: Сheck the return values of the WNetGetUniversalNameA.
by Marvin 19 Jul '12

19 Jul '12

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=20174 Your paranoid android. === WNT4WSSP6 (32 bit mpr) === mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7 mpr.c:124: Test failed: WNetGetUniversalNameA gave wrong error: 000001e7

1 0

Re: fusion: Don't assume a specific compiler behaviour
by Ruslan Kabatsayev 19 Jul '12

19 Jul '12

Hello, > Most likely it's rotated on x86 and hardly shifted on ARM, damn compilers. No, it's not compilers, it's intel's "shr" instruction behavior: it masks bit count lower 5 bits, thus making 33 effectively 1. Corresponding ARM instruction most likely does true 33 bit shift giving you zero result. Regards, Ruslan PS I didn't look at types your patch works on, but be careful to not mess up 64 bit types this way. On Thu, Jul 19, 2012 at 11:49 PM, André Hentschel <nerv(a)dawncrow.de> wrote: > maybe it's some braindamage by me, but when i is greater or equal to 32 a shift of HighPart by e.g. 33 doesn't make much sense. For some reason this works on x86 but not on ARM. Most likely it's rotated on x86 and hardly shifted on ARM, damn compilers. > If i'm right i don't want to know how much places there are in Wine with assumptions like this, i'll most likely find them with the testsuite on ARM, but i's quite hard to get to the point where you see them... > --- > dlls/fusion/assembly.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/dlls/fusion/assembly.c b/dlls/fusion/assembly.c > index d04ba73..8e6bfcb 100644 > --- a/dlls/fusion/assembly.c > +++ b/dlls/fusion/assembly.c > @@ -506,7 +506,7 @@ static HRESULT parse_clr_tables(ASSEMBLY *assembly, ULONG offset) > for (i = 0; i < MAX_CLR_TABLES; i++) > { > if ((i < 32 && (assembly->tableshdr->MaskValid.u.LowPart >> i) & 1) || > - (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> i) & 1)) > + (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> (i-32)) & 1)) > { > assembly->numtables++; > } > @@ -522,7 +522,7 @@ static HRESULT parse_clr_tables(ASSEMBLY *assembly, ULONG offset) > for (i = 0; i < MAX_CLR_TABLES; i++) > { > if ((i < 32 && (assembly->tableshdr->MaskValid.u.LowPart >> i) & 1) || > - (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> i) & 1)) > + (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> (i-32)) & 1)) > { > assembly->tables[i].rows = assembly->numrows[offidx]; > offidx++; > @@ -534,7 +534,7 @@ static HRESULT parse_clr_tables(ASSEMBLY *assembly, ULONG offset) > for (i = 1; i < MAX_CLR_TABLES; i++) > { > if ((i < 32 && (assembly->tableshdr->MaskValid.u.LowPart >> i) & 1) || > - (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> i) & 1)) > + (i >= 32 && (assembly->tableshdr->MaskValid.u.HighPart >> (i-32)) & 1)) > { > currofs += get_table_size(assembly, previ) * assembly->numrows[offidx - 1]; > assembly->tables[i].offset = currofs; > -- > 1.7.4.1 > > > -- > > Best Regards, André Hentschel > > >

2 1

Re: wininet: Support ICU_ENCODE_PERCENT, ICU_ENCODE_SPACES_ONLY, and ICU_NO_META.
by Austin English 19 Jul '12

19 Jul '12

On Wed, Jul 18, 2012 at 6:33 PM, Alex Henrie <alexhenrie24(a)gmail.com> wrote: > Without this patch, Internet Explorer 6 produces a lot of messages like > "fixme:wininet:InternetCanonicalizeUrlW Unhandled flags 0x04000000" > > Wine's wininet.h was missing ICU_ENCODE_PERCENT, which is present in > the wininet.h from the Windows 7 SDK. > > Wine previously treated ICU_ESCAPE like ICU_DECODE. This is incorrect, > MSDN and empirical tests agree that ICU_ESCAPE should be ignored: > http://msdn.microsoft.com/en-us/library/windows/desktop/aa384342(v=vs.85).a… > https://testbot.winehq.org/JobDetails.pl?Key=20155 Why didn't you include the tests in the patch? -- -Austin

2 2

Re: [1/2]usp10: cache OUTLINETEXTMETRICW instead of just TEXTMETRICW
by Dmitry Timoshkov 19 Jul '12

19 Jul '12

Aric Stewart <aric(a)codeweavers.com> wrote: > + sc->tm = heap_alloc(size); > + GetOutlineTextMetricsW(hdc, size, sc->tm); otmSize exists for a reason, if GetOutlineTextMetrics currently doesn't check it this doesn't mean it won't at some point in future. -- Dmitry.

1 0

Re: [PATCH 4/4] ieframe: Avoid useless QI(IID_IWebBrowser2) calls in tests by passing IWebBrowser2 interface where possible
by Marvin 19 Jul '12

19 Jul '12

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=20160 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: include/winhttp.h: Make strings in WINHTTP_PROXY_INFO non-const
by Hans Leidekker 19 Jul '12

19 Jul '12

On Thu, 2012-07-19 at 07:27 +0000, Heiko Hund wrote: > The strings shouldn't be const according to the docs at [1] and > declaring them as LPWSTR removes some false warnings when > compiling code GlobalFree()ing them with mingw. Please also remove unnecessary casts from Wine.

1 0

Re: [PATCH 01/10] dsound/tests: Add IDirectSound refcount and COM aggregation tests.
by Marvin 18 Jul '12

18 Jul '12

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=20152 Your paranoid android. === WNT4WSSP6 (32 bit dsound) === dsound.c:194: Test failed: DirectMusicPerformance create failed: 80070057, expected CLASS_E_NOAGGREGATION dsound.c:196: Test failed: dso = DEADBEEF

1 0

Re: ole32: Forward BSTR functions to oleaut32
by Alexandre Julliard 18 Jul '12

18 Jul '12

Jacek Caban <jacek(a)codeweavers.com> writes: > @@ -1244,6 +1244,9 @@ static void COMPOBJ_InitProcess( void ) > static void COMPOBJ_UninitProcess( void ) > { > UnregisterClassW(wszAptWinClass, hProxyDll); > + > + if(oleaut32_instance) > + FreeLibrary(oleaut32_instance); Since oleaut32 imports ole32 you'll never get here. Using a delayed import would probably be easier. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: mshtml: Set outgoing pass-by-address variable in an error case case within confirm_safety_load.
by Jacek Caban 18 Jul '12

18 Jul '12

Hi Gerald, On 03/07/12 23:35, Gerald Pfeifer wrote: > I noticed we return in this case, without initializing this variable. > Visual inspection indicates we do not seem to access the variable in > this error case, but a) better safe than sorry, and b) GCC 4.8 currently > warns about it. > > Not sure URLPOLICY_DISALLOW is the best return value, open for better > alternatives. I generally don't think this patch makes things any better (or worse). If we return an error, the caller should not expect this value to be sane. What's the GCC 4.8 warning? Jacek

2 4

Re: d3dx9_36: Implementation of D3DXSHRotateZ
by Matteo Bruni 18 Jul '12

18 Jul '12

2012/7/18 Nozomi Kodama <nozomi.kodama(a)yahoo.com>: > Minor nitpicks, but nevertheless: > -@ stub D3DXSHRotateZ(ptr long long ptr) > +@ stdcall D3DXSHRotateZ(ptr long long ptr) ... > +FLOAT* WINAPI D3DXSHRotateZ(FLOAT *out, UINT order, FLOAT angle, CONST FLOAT *in) I guess using "float" in the spec file for the third parameter would be more appropriate (yes, it was already wrong but that's not an excuse ;) + if ( i >= order * order || ( (order > D3DXSH_MAXORDER) && ( i >= D3DXSH_MAXORDER * D3DXSH_MAXORDER ) ) ) + expected = ( i + 1.0f ) * ( i + 1.0f ); Bad indentation. In general the inner for loop in the tests feels awkward, try to rewrite it to avoid the duplicated stuff. + } + } + } You're adding a tab character here, please use only whitespaces for indentation. That said, thanks for your work on d3dx9!

1 0

Re: [PATCH 2/2] vbscript: Added LCase implementation
by Marvin 18 Jul '12

18 Jul '12

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=20130 Your paranoid android. === WINEBUILD (build) === Patch failed to apply

1 0

Re: [2/2] gdiplus: Add 8bpp to the list of supported formats.
by Vincent Povirk 18 Jul '12

18 Jul '12

> + &GUID_WICPixelFormat8bppGray, This will give you a halftone palette instead of a grayscale palette, unless you add special handling for that format.

2 2

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-devel