September 2009 - Wine-Devel

Re: New testcase for creating nested storage objects
by Nikolay Sivov Sept. 18, 2009

Sept. 18, 2009

Markus Stockhausen wrote: > Hi, > > attached you will find a new testcase that checks for differences > between Wine and Windows when handling reference counters for nested > storage objects. > > Best regards. > > Markus > Hi, Markus. Tests passes on XP, fails on current Wine. You should mark failing tests with todo_wine. Like that: > c1 = IStorage_AddRef(stg); > todo_wine ok(c1 == 2, "creating internal storage added > references to ancestor\n"); > c1 = IStorage_AddRef(stg); > IStorage_Release(stg2); > c2 = IStorage_AddRef(stg) - 1; > todo_wine ok(c1 == c2, "releasing internal storage > removed references to ancestor\n"); otherwise it will fail on Wine. Also give a more appropriate patch name and don't forget to add a prefix something like this: ole32/tests: Test for nested storages reference counting. P.S. git diff patch is preferable here. Thanks for your work.

2 2

glean and Piglit -- OpenGL driver testing
by Saulius Krasuckas Sept. 18, 2009

Sept. 18, 2009

Today I saw two similar projects related to OpenGL: [1]: > glean is a suite of tools for evaluating the quality of an OpenGL > implementation and diagnosing any problems that are discovered. glean > also has the ability to compare two OpenGL implementations and highlight > the differences between them. It seems be having win32 port also. [2]: > Piglit is a collection of automated tests for OpenGL implementations. > > The goal of Piglit is to help improve the quality of open source OpenGL > drivers by providing developers with a simple means to perform > regression tests. > > Current status is that the framework is working (though rough at the > edges). It contains the Glean tests, some tests adapted from Mesa as > well as some specific regression tests for certain bugs. HTML summaries > can be generated (see below), including the ability to compare different > test runs. Could these be of any use for our graphic guys -- Stefan and co.? Then there is PerceptualDiff utility I found some time ago [3]. Guessed, could it also usefull for finding visual regressions of Wine? Probably not, as it seems to be used for testing video codecs (but I may be wrong): > PerceptualDiff is an image comparison utility that makes use of a > computational model of the human visual system to compare two images. > > So why would I use a program to tell me if two images are similar if I > can tell the difference myself by eyeballing it? > > Well the utility of this program really shines in the context of QA of > rendering algorithms. > > During regression testing of a renderer, hundreds of images are > generated from an older version of the renderer and are compared with a > newer version of the renderer. This program drastically reduces the > number of false positives (failures that are not actually failures) > caused by differences in random number generation, OS or machine > architecture differences. Also, you do not want a human looking at > hundreds of images when you can get the computer to do it for you > nightly on a cron job. [1] http://glean.sourceforge.net/whatis.html [2] http://people.freedesktop.org/~nh/piglit/ [3] http://pdiff.sourceforge.net/

4 5

Re: oledb32/tests: Add CanConvert tests.
by Paul Vriens Sept. 18, 2009

Sept. 18, 2009

On 09/17/2009 03:35 PM, Huw Davies wrote: > --- > dlls/oledb32/tests/Makefile.in | 2 +- > dlls/oledb32/tests/convert.c | 76 ++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 77 insertions(+), 1 deletions(-) > > > ------------------------------------------------------------------------ > > Hi Huw, This introduces issues on (older) W2K systems and below. Do we need to add some kind of skip variable or just quit (with a win_skip) if the first one fails? We could also check the conversion 0000 -> 0087 as that seems to be common on all as a failure. If this conversion fails, bail out? Thoughts? -- Cheers, Paul.

1 0

Regression
by chris ahrendt Sept. 17, 2009

Sept. 17, 2009

b5b58e423d32666581f69046c74bb19902c35c2b is first bad commit commit b5b58e423d32666581f69046c74bb19902c35c2b Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Sep 17 12:35:24 2009 +0200 d3d8: Add a separate function for cube texture initialization. :040000 040000 8cd1752ed711799ebfb77127de399d527add5415 963c6467c8b4367b8a7bd1c1e4eece07937c13d6 M dlls [cahrendt(a)stinky wine-git]$ Chris

2 1

apply context draw buffer
by chris ahrendt Sept. 17, 2009

Sept. 17, 2009

Something broke between yesterday's git tree and todays... went to run some applications and everything is slow as molasas... seems to be getting in a infinite loop in the apply context draw buffer

1 0

Re: Add dxdiag help message box
by Henri Verbeet Sept. 17, 2009

Sept. 17, 2009

2009/9/17 Brian Nguyen <mtxcoll(a)gmail.com>: > +/* Thread-safe function for converting to wide char strings at runtime */ > +LPWSTR PrintWide(LPWSTR buf, size_t len, const char *s) > +{ > + size_t i; > + for (i = 0; i < len && *s != '\0'; i++, s++) { > + buf[i] = *s; > + } > + return buf; > +} That doesn't do what you want, you should use MultiByteToWideChar() to convert to WCHAR. However, I don't think you want to hardcode the strings in the first place, you should load them from the resources so they can be translated. Also, functions like that which are only used in the current file should be static.

2 1

Re: [1/5] mapi32: Scan registry for MAPI providers and load them
by Nikolay Sivov Sept. 17, 2009

Sept. 17, 2009

Owen Rudge wrote: > --- > dlls/mapi32/Makefile.in | 2 +- > dlls/mapi32/mapi32_main.c | 5 +- > dlls/mapi32/util.c | 148 > +++++++++++++++++++++++++++++++++++++++++++++ > dlls/mapi32/util.h | 28 +++++++++ > 4 files changed, 181 insertions(+), 2 deletions(-) > create mode 100644 dlls/mapi32/util.h Hi. Few comments. > + > +static HMODULE mapi_provider = NULL; > +static HMODULE mapi_ex_provider = NULL; > + This is redundant, static will be initialized to zero anyway. > +static void load_mapi_provider(HKEY hkeyMail, LPCWSTR valueName, HMODULE *mapi_provider) > +{ > + static const WCHAR mapi32_dll[] = {'m','a','p','i','3','2','.','d','l','l',0 }; > + > + DWORD dwType, dwLen = 0; > + LPWSTR dllPath; > + > + if (!mapi_provider) > + return; > + I don't think this check is necessary, it's your internal helper. > +cleanUp: ; Semicolumn isn't needed here.

1 0

please test inconsistent WAVEFORMAT on MS-Windows (compiler needed)
by Joerg-Cyril.Hoehle＠t-systems.com Sept. 17, 2009

Sept. 17, 2009

Hi, One MS-Windows machine passes all the attached tests, proving that MS-Windows ignores nBlockAlign and nAvgBytesPerSec. It would be helpful to get confirmation from more machines. Please build with the attached patch, execute it in interactive mode set WINETEST_INTERACTIVE=1 winetest.exe wave and listen to the sound and especially the pauses. OTOH, Wine fails at least one test: - One waveOutGetPosition test in WAVE_MAPPER fails. - My recently accepted patches (for OSS and CoreAudio) do something forbidden, that the test was written to reveal: they fix broken values in the WAVEFORMAT description, because both winmm and dsound depend on their correctness. But Wine should not touch user-supplied data. That is a todo_wine until dsound learns how to live with broken input. I'll add todo_wine before submitting these tests as a patch. Try this out with <= 1.1.29 (test with WINETEST_INTERACTIVE=1 make test) and you'll hear how Wine's winmm incorrectly computes the duration of the sound when given inconsistent format descriptions, causing silence where none should be, because MS-Windows does not care about BlockAlign and AvgBytesPerSec. With the upcoming 1.1.30, this will be corrected for OSS and CoreAudio, but not yet with the WAVE_MAPPER (ALSA is still broken). Regards, Jörg Höhle

1 0

Re: winetest: Add the ability to retrieve the dll path for com dlls from the registry.
by Paul Vriens Sept. 17, 2009

Sept. 17, 2009

On 09/14/2009 03:10 PM, Huw Davies wrote: > --- > programs/winetest/main.c | 87 ++++++++++++++++++++++++++++++++++++++------- > 1 files changed, 73 insertions(+), 14 deletions(-) > > > ------------------------------------------------------------------------ > > Hi Huw, Winetest doesn't do a complete run anymore (tested only on W2K3 and Vista) for now. It ends with a 998 error during enumeration of the test (oleaut32 being the last visible one). I'm crossbuilding a winetest as we speak but just thought I'd mention it. -- Cheers, Paul.

2 3

Re: user32: Skip STM_SETIMAGE message if the static control style doesnot match
by Dmitry Timoshkov Sept. 17, 2009

Sept. 17, 2009

"Ilya Shpigor" <shpigor(a)etersoft.ru> wrote: > This style checking is needed to the STM_SETIMAGE message processing according > to MSDN: > > http://msdn.microsoft.com/en-us/library/bb760782%28VS.85%29.aspx This change requires a test case. -- Dmitry.

1 0

re: Disabling URL attachments.
by Dan Kegel Sept. 16, 2009

Sept. 16, 2009

Done.

2 1

dstlen < 0 for MultiByteToWideChar / WideChatToMultiByte
by Michael Stefaniuc Sept. 16, 2009

Sept. 16, 2009

Hello Detlef, although Smatch could have handled the task too of finding places where Wine passes -1 as destlen to MultiByteToWideChar / WideChatToMultiByte this can be done with the C compiler too. Please see the attached patch. This solution doesn't finds more occurrences that you have found and patched already. But this is really only the stuff that the compiler can figure out at compile time. It has no chance at all to catch stuff like void foo(int bar) { int dstlen = -1; if (bar) dstlen = 10; MultiByteToWideChar(a, b, c, d, e, dstlen) } Those can't be found by a grep either and I think that's why Alexandre is reluctant to apply your patches. bye michael

2 1

Disabling URL attachments.
by Vitaliy Margolen Sept. 16, 2009

Sept. 16, 2009

Can Bugzilla admin please disable URL attachments? Number of users don't get the concept of what attachment is, and keep attaching urls to things like pastebin, or some file share servers. Instead of attaching actual files. Vitaliy.

1 0

Re: oleaut32: Fail the typelib test when LoadTypeLib fails.
by Jacek Caban Sept. 16, 2009

Sept. 16, 2009

Hi Vincent, Vincent Povirk wrote: > > As far as I know, this failure only occurs when I do something stupid > like set use_midl_tlb to 1 and don't provide a working > midl_tmarshal.tlb, but I need the test to remind me that I'm an idiot > when I do that. Silently skipping tests is unacceptable. > While you are at this, it would be better to test the exact hr value, so the test should be: ok(hr == S_OK, ...). Jacek

1 0

Status of Quartz
by Keith Muir Sept. 16, 2009

Sept. 16, 2009

what's the status of wine quartz.dll at present? reason I'm asking is because I'm trying to get a Computer based training(CBT) app Testout Navigator to run which uses Windows Media Player 9 to show videos. Using WINDEBUG with warn+dll seems to show most of the trouble is due to quartz.dll interaction with ntdll.dll Regards, Keith

1 0

Testcase for IStorage_CreateStorage
by Markus Stockhausen Sept. 16, 2009

Sept. 16, 2009

Hi, as I'm still working on a resolution to bug #9575 I have digged into several directions. My current opinion is that IStorage_CreateStorage for some unkown reason increments the reference pointer of the associated IStorage object. In native Windows this is not the case. My 37th try in writing a Wine testcase resulted in the following code that hopefully shows the difference between the two environments. I put it somewhere in the storage32 tests to get it running. No warranty for side effects. static const WCHAR yyy[] = { 'x','.','t','x','t',0 }; r = StgOpenStorage( yyy, NULL, STGM_TRANSACTED | STGM_SHARE_EXCLUSIVE | STGM_READWRITE , NULL, 0, &stg); ok(r==S_OK, "StgCreateStorage failed\n"); if(stg) { static const WCHAR xxx[] = { 'D','a','t','a','S','p','a','c','e','I','n','f','o',0 }; IStorage_CreateStorage(stg,xxx,STGM_READWRITE | STGM_SHARE_EXCLUSIVE,0,0,&stg2); if (stg2) { r = IStorage_AddRef(stg); ok(r == 2, "WINE ERROR: refcount should be 2\n"); IStorage_Release(stg2); while ( r ) r=IStorage_Release(stg); } else { ok( 1 == 2, "IStorage::CreateStorage failed\n"); IStorage_Release(stg); } } If I got everything in the right place this test should fail on Wine wheres it will be ok in Windows. Before doing further research I hope someone of the experts can check if the test really reveals the source of the above mentioned bug. Thanks in advance.

1 0

now OSS driver emulation -- and other drivers too -- works as well as OSS HW mode; block size error gone?
by Joerg-Cyril.Hoehle＠t-systems.com Sept. 16, 2009

Sept. 16, 2009

Hi, if your app worked well in OSS HW driver mode but not in emulation mode, the following patch may help you. For me, it fixed the following: - wrong playback speed in some apps (bug #19901); - spurious errors about block size: err:dsound:DSOUND_MixInBuffer length not a multiple of block size, len = 1579, block size = 2 - an assertion failure and crash (bug #12349): mixer.c:489: DSOUND_MixInBuffer: Assertion `dsb->buf_mixpos + len <= dsb->tmp_buffer_len' failed. As the MacOS CoreAudio driver produces sound effects much like OSS emulation, a similar fix to wineCoreAudio.drv will certainly cure sound on the Mac, lifting some apps' rating from garbage to gold! diff --git a/dlls/wineoss.drv/audio.c b/dlls/wineoss.drv/audio.c index ca79cd5..9ef0d5b 100644 --- a/dlls/wineoss.drv/audio.c +++ b/dlls/wineoss.drv/audio.c @@ -2071,6 +2071,13 @@ static DWORD wodOpen(WORD wDevID, LPWAVEOPENDESC lpDesc, DWORD dwFlags) wwo->waveFormat.Format.nSamplesPerSec) / wwo->waveFormat.Format.nChannels; } + if (wwo->waveFormat.Format.nBlockAlign != wwo->waveFormat.Format.nChannels * + wwo->waveFormat.Format.wBitsPerSample / 8) { + WARN("Fixing BlockAlign\n"); + wwo->waveFormat.Format.nBlockAlign = wwo->waveFormat.Format.nChannels * + wwo->waveFormat.Format.wBitsPerSample / 8; + lpDesc->lpFormat->nBlockAlign=wwo->waveFormat.Format.nBlockAlign; /* see DirectSoundDevice_Create */ + } /* Read output space info for future reference */ if (ioctl(wwo->ossdev.fd, SNDCTL_DSP_GETOSPACE, &info) < 0) { ERR("ioctl(%s, SNDCTL_DSP_GETOSPACE) failed (%s)\n", wwo->ossdev.dev_name, strerror(errno)); The problem is that Wine depends on invariants about BlockAlign and AvgBytesPerSec but does not enforce them when presented incorrect data. MSDN says http://msdn.microsoft.com/en-us/library/bb669161(VS.85).aspx "If wFormatTag is WAVE_FORMAT_PCM or WAVE_FORMAT_EXTENSIBLE, nBlockAlign must be equal to the product of nChannels and wBitsPerSample divided by 8 (bits per byte). If wFormatTag is WAVE_FORMAT_PCM, nAvgBytesPerSec should be equal to the product of nSamplesPerSec and nBlockAlign." In the source base up to now, Wine only corrects the format in the OSS hardware mode. That's why OSS HW works well. It's too early to submit this as a patch as there are a few open issues: - Does the bogus data originate from the app or is some other function in Wine returning this data? A trace snippet from bug #19901 shows inconsistent data in the following call to SetFormat following device creation: PrimaryBufferImpl_Create (formattag=0x0001,chans=2,samplerate=44100,bytespersec=176400,blockalign=4,bitspersamp=16,cbSize=0) DSOUND_PrimarySetFormat (formattag=0x0001,chans=1,samplerate=22050,bytespersec=88200,blockalign=4,bitspersamp=16,cbSize=0) - Is there some central place in dsound for enforcing the invariant instead of identical code in the half dozen OSS/ALSA/CoreAudio/Jack etc. drivers? - Is the modification of the WAVEOPENDESC structure allowed, as it seems to be a user-supplied structure, which should be handled read-only? - Should the sound driver refuse to open when submitted such incorrect data? Possibly not. MSDN says about SetFormat http://msdn.microsoft.com/en-us/library/bb206041(VS.85).aspx "The method succeeds even if the hardware does not support the requested format; DirectSound sets the buffer to the closest supported format. To determine whether this has happened, an application can call the GetFormat ..." Thus, DS may set a format different from what was supplied. Presumably, BlockAlign and AvgBytesPerSec play a minor role, as they can be derived from the other values. Experimentally, I had wineoss.drv/audio.c:supportedformat() refuse inconsistent formats. Wine nevertheless played the app's sound at bogus speed after scanning all wineacm/mp3/xyz drivers (which it would not do otherwise), taking an execution path I've not yet analysed. - How does MS-Windows behave in the presence of bad data? Additional tests are needed. - I've not yet fixed the ALSA driver, but I expect similar improvements, at least in the emulation case, because the sound errors are the same in OSS and ALSA emulation modes. This is exciting news about progress in Wine sound issues! Bugzilla has errors about block size as old as 2005 still open. Regards, Jörg Höhle

2 4

Down to an average of 1% failures on my Windows boxes
by Paul Vriens Sept. 16, 2009

Sept. 16, 2009

Hi, Just wanting to let you know that I'm down to 1% average failures on my Windows boxes (W95/W98/NT4/W2K/XP/W2K3/Vista). Short list: Sep-2009 : 1% Jun-2009 : 2% May-2009 : 3% Mar-2009 : 4% Feb-2009 : 5% So I doubt we get to zero before WineConf ;) -- Cheers, Paul.

1 0

CertGetPublicKeyLength() test failures on Win7
by Greg Geldorp Sept. 15, 2009

Sept. 15, 2009

Hi Juan, I see 3 test failures related to CertGetPublicKeyLength() on Win7, crypt32/tests/cert.c lines 3146/3160/3165. All 3 failures are the same, CertGetPublicKeyLength() returns 0 with last error 0x80090004 (NTE_BAD_LEN). After looking at it for a while, I'm inclined to change the condition to ok(ret == 56 || (ok == 0 && GetLastError() == NTE_BAD_LEN), ...). Do you agree that's the correct fix or do I need to dig deeper? Thanks, Ge.

2 4

Wine at Ohloh
by Saulius Krasuckas Sept. 15, 2009

Sept. 15, 2009

FYI, http://www.ohloh.net/p/wine/analyses/latest I wonder how do they find 27k lines of C++ code in Wine tree...

4 4

Re: [PATCH 2/2] dpwsockx: Implementation of Open callback
by Vitaliy Margolen Sept. 15, 2009

Sept. 15, 2009

Ismael Barros wrote: > --- > +static DWORD WINAPI udp_listener_thread( LPVOID lpParameter ) > +{ > + for ( ;; ) > + { ..... > + } > +} You do not have exit from the loop. > + /* Launch thread */ > + listener->handle = CreateThread( NULL, 0, udp_listener_thread, > + listener, 0, NULL ); I haven't found the place where you closing this handle. > + /* Launch thread */ > + listener->handle = CreateThread( NULL, 0, > + ( is_tcp > + ? tcp_listener_thread > + : udp_listener_thread ), > + listener, 0, NULL ); Same here. Vitaliy.

3 7

re: [3/4] msi: GetDriveType and GetVolumeInformation require a path terminated by a slash.
by Dan Kegel Sept. 15, 2009

Sept. 15, 2009

Don't changes like this need corresponding conformance tests? Likewise with "[4/4] msi: Don't append the cabinet file name in find_published_source."

2 1

How I create wine app launcher icons on the Mac
by Joerg-Cyril.Hoehle＠t-systems.com Sept. 15, 2009

Sept. 15, 2009

Hi, On Linux, I have been using .desktop files that freedesktop defines to keep a directory full of icons. They all launch Wine applications. Often enough, these desktop files are initially create by Wine in ~/.local/share/applications/wine/..., along with the icon files in ~/.local/share/icons/. Here's how I now have app launcher icons on the Mac: sips -i ~/.local/share/icons/foo.png --out foo.png # sips -i converts and *additionaly* adds icon resource to file # Alas, sips does not understand .xpm cat sample.command > foo.png # > overwrites contents but keeps resource fork mv foo.png ~/Desktop/foo.command # .command is the finder's suffix for executable shell scripts chmod a+x foo.command sample.command contains: #!/bin/sh cd /User/me/xyz && WINEDEBUG=-gsm WINEPREFIX=$HOME/.wine exec wine xyz.exe Afterwards you edit foo.command and change the directory, environment variables and executable names to meet your needs. Be careful to use an editor that keeps the resource fork. "Opening" this icon will launch a Terminal where all logs will go. Optionally you may use Cmd-I(nformation) to disable the display of the .command suffix in the finder (unless forced globally). Wine's X11 window opens above the Terminal window. I wonder whether this functionality could be somehow integrated into the git (winemenubuilder?), or whether developers believe this should be left to "surrounding" projects like Darwine. What do you think? Jörg Höhle PS: You can wrap up the whole thing into a single shell script using the cat <<EOF syntax.

2 3

Re: [1/2] comctl32: Implement highlighting (marquee) selection support in listview
by Nikolay Sivov Sept. 14, 2009

Sept. 14, 2009

Owen Rudge wrote: > These patches add support for selecting multiple items in a > multi-selection listview using the mouse (by dragging and highlighting). > > --- > dlls/comctl32/listview.c | 172 > ++++++++++++++++++++++++++++++++++++++++++---- > 1 files changed, 157 insertions(+), 15 deletions(-) Hi, Owen. Some comments. --- @@ -201,6 +201,7 @@ typedef struct tagITEM_INFO LPARAM lParam; INT iIndent; ITEM_ID *id; + UINT orig_state; /* state before highlighting operation begins */ } ITEM_INFO; --- Why do you need this restoring code? If you're starting selection box with some items selected they should be deselected if not Shift pressed and leaved untouched if Shift used. I don't see a reason to restore it - when you want to toggle selected state moving rectangle with Ctrl pressed you could just: LISTVIEW_GetItemState() and place inverted state with following LISTVIEW_SetItemState(). Also such large cycles shouldn't be used if you could manage not to use them. Currently we're not doing right sometimes with ownerdata lists which leads to huge response times (see http://bugs.winehq.org/show_bug.cgi?id=14303) Anyway this part: --- + /* Take a snapshot of all the item states */ + for (i = 0; i < infoPtr->nItemCount; i++) + { + hdpaSubItems = DPA_GetPtr(infoPtr->hdpaItems, i); + lpItem = DPA_GetPtr(hdpaSubItems, 0); + + if (lpItem) + lpItem->orig_state = lpItem->state; + } --- is wrong. You can't do it that way cause selected state could also be cached at parent side with corresponding callback bit set (look at LISTVIEW_GetItemT()). P.S. No need to split in two patches here, fist is useless if we can't see rectangle.

2 2

Re: [PATCH 1/1] winecoreaudio: make supported format test more restrictive
by Ken Thomases Sept. 14, 2009

Sept. 14, 2009

On Sep 14, 2009, at 6:21 AM, <Joerg-Cyril.Hoehle(a)t-systems.com> <Joerg-Cyril.Hoehle(a)t-systems.com > wrote: > supportedformat() has been in OSS' audio.c since 2004, so the > coreaudio driver was forked even before that time?!? Well, the CoreAudio driver was based on the ALSA driver. So, if ALSA was copied from the OSS driver before that change was made, then effectively, yes, the CoreAudio driver was forked before that time. -Ken

1 0