March 2005 - Wine-Devel - list.winehq.org

Re: USER32: Sysparams unit tests fixes and additions
by Paul Vriens March 1, 2005

March 1, 2005

On Tue, 2005-03-01 at 05:53, jchevrier(a)nexicom.net wrote: > Changelog: > > * Created SPI_{GET,SET}KEYBOARDPREF and SPI_{GET,SET}SCREENREADER unit tests > * Removed a trace that shouldn't have been there > * Add return checking to SPI_{GET,SET}WHEELSCROLLLINES and > SPI_{GET,SET}MENUSHOWDELAY unit tests so they no longer run on Windows 95 > where they aren't supported > * Changed minimum hovertime in SPI_{GET,SET}MOUSEHOVERTIME to 10 as Windows XP > defaults to 10 any value below that Hi, the extra trace was my doing :-(. I've added this and after that I realized that SPI_{GET|SET} doesn't clean the LastError after a correct call. That's why I think it's necessary to do a SetLastError(0) before every check of GetLastError. That's the reason I had these failures on win98 and SPI_GETICONTITLEWRAP. The Error was still set by one of the previous called functions (I think a ..W call, the ..A call was fine). That still leaves us with the questions, whether it would be good to check for ERROR_CALL_NOT_IMPLEMENTED after each call? Cheers, Paul Vriens.

2 1

Re: [wined3d] texture interface separation.
by Oliver Stieber March 1, 2005

March 1, 2005

> wrote: > > > > Hi, > > > > This patch start to separate the interfaces > in > > > > wine3d3. Complete separation requires > > > queryInterface > > > > support which I'll send through soon, this > patch > > > has > > > > no dependencies but provides the final > interface > > > > needed by later revisions. > > > > > > > Hi Oliver, > > > > > > this patch couldn't be applied cleanly to CVS. > Just > > > to let you know, I > > > didn't check further. > > > > > > I'm wondering if AJ will have many comments on > the > > > patches. I hope not > > > as your work is going to make a lot of people > happy. > > It's a diff against CVS. But I think you need a diff > against CVS + your > previous patch. Or am I missing something ? > The patches aren't incremental, that way one can fail but it shouldn't affect the rest being merged. I have another couple of discrete patches to send in then the patches become dependant on some of the interface changes, but hopefully not the underlying functionality changes. I also don't have enough HDD space to keep incremental versions and check that they compile/work especially with all the interface/include changes. If anyone knows of a good tool for managing patches let me know (not cerversa!), there's a surprising lack of tools available. at the moment I'm using the following... #!/bin/bash #given a file, replace the first path with a new path #diff the two files stripedpath=$(echo $1 | grep -o "/.*") kdiff3 $1 "$2/$stripedpath"& kwrite $1 & kwrite "$2/$stripedpath"& -- diffdueo clean/wine/.... working Very 21st century. A tool for merging would be good too, e.g. merge patch1.patch patch2.patch > patch_merged.patch. if one doesn't exist I'll write it and put it on my site. Send instant messages to your online friends http://uk.messenger.yahoo.com

2 6

Re: [PATCH] Added 2 functions to shell32
by Jacek Caban March 1, 2005

March 1, 2005

> I doubt shell32 function on windows does this check, also original windows shell32 > function is a lot more complicated and longer, I've compared the results between > real and my code and they are correct. I wrote a small test for this function showing that your implementation is almost correct. It failes in two cases: 1. If a path ends with '\' and is compared with one that doesn't, then it's length is less than the other one, but both are the same paths. 2. If a path doesn't end with '\' and is compared with a longer one that has the same first len chars, but is not the same, ex: c:\test c:\test123 Changelog: Added test for PathIsEqualOrSubFolder Index: dlls/shell32/tests/shellpath.c =================================================================== RCS file: /home/wine/wine/dlls/shell32/tests/shellpath.c,v retrieving revision 1.6 diff -u -p -r1.6 shellpath.c --- dlls/shell32/tests/shellpath.c 10 Feb 2005 19:19:35 -0000 1.6 +++ dlls/shell32/tests/shellpath.c 28 Feb 2005 21:19:31 -0000 @@ -871,6 +871,55 @@ static void testNonExistentPath(void) userShellFolders); } +typedef BOOL (WINAPI*fnPathIsEqualOrSubFolder)(LPCWSTR path1, LPCWSTR path2); + +static void test_PathIsEqualOrSubFolder() +{ + fnPathIsEqualOrSubFolder pPathIsEqualOrSubFolder; + BOOL ret; + static const WCHAR path1[] = {'c',':','\\','t','e','s','t',0}; + static const WCHAR path2[] = {'c',':','\\','t','e','s','t','\\','t','e','s','t',0}; + static const WCHAR path3[] = {'c',':','\\','t','e','s','t','1','2','3',0}; + static const WCHAR path4[] = {'c',':','\\','t','e','s','t','\\',0}; + static const WCHAR path5[] = {'c',':','\\','t','e','s','t','\\','t','e','s','t','2', + '\\','.','.','\\','t','e','s','t',0}; + + pPathIsEqualOrSubFolder = GetProcAddress(hShell32, (LPSTR)755); + if(!pPathIsEqualOrSubFolder) + return; + +#define TEST_PATH(p1, p2, r) \ + ret = pPathIsEqualOrSubFolder(p1, p2); \ + ok(ret == r, "PathIsEqualOrSubFolder(" #p1 " " #p2 ") failed, expected: %x\n", r); + + TEST_PATH(path1, path1, TRUE); + TEST_PATH(path1, path2, TRUE); + TEST_PATH(path1, path3, FALSE); + TEST_PATH(path1, path4, TRUE); + TEST_PATH(path2, path1, FALSE); + TEST_PATH(path2, path2, TRUE); + TEST_PATH(path2, path3, FALSE); + TEST_PATH(path2, path4, FALSE); + TEST_PATH(path3, path1, FALSE); + TEST_PATH(path3, path2, FALSE); + TEST_PATH(path3, path3, TRUE); + TEST_PATH(path3, path4, FALSE); + TEST_PATH(path4, path1, TRUE); + TEST_PATH(path4, path2, TRUE); + TEST_PATH(path4, path3, FALSE); + TEST_PATH(path4, path4, TRUE); + TEST_PATH(path5, path1, FALSE); + TEST_PATH(path5, path2, FALSE); + TEST_PATH(path5, path3, FALSE); + TEST_PATH(path5, path4, FALSE); + TEST_PATH(path5, path5, TRUE); + TEST_PATH(NULL, path1, FALSE); + TEST_PATH(path1, NULL, FALSE); + TEST_PATH(NULL, NULL, FALSE); + +#undef TEST_PATH +} + START_TEST(shellpath) { if (!init()) return; @@ -892,5 +941,6 @@ START_TEST(shellpath) testWinDir(); testSystemDir(); testNonExistentPath(); + test_PathIsEqualOrSubFolder(); } }

3 2

New uninstaller
by Jonathan Ernst March 1, 2005

March 1, 2005

As requested, I'm trying to make a new uninstaller using a dialog instead of a window. It's my first windows program and I have something working but there are still some issues. If someone could help with some of the following problems, I'd be very thanfull. 1) Strange problem with UpdateWindow uninstaller.c, line 264 I implemented a search/filter field and I want to repaint the window after each input in the EDITTEXT control. However, when I call UpdateWindow on this line, the value returned is 1 so that a WM_PAINT message should be sent right ? But strangly no message is sent and I have to make something else (like moving the dialog) to force the window to repaint and filter the entries. The funny thing is that UpdateWindow on line 279 works just fine. I guess it's a bug in my code and not in Wine so I'd be glad if you could help me. 2) Improve filter code Currently I filter the entries that match exactly the filter string. I would like to at least make a case insensitive strstr on the string. What's the best way to do it (See uninstaller.c, line 235) 3) Stringtables a)Is there a way to load localized strings without having to guess beforehand the size of the string in each language ? b)Is there a way to not call LoadString for each string ? c)Where is the best place to load every string of a stringtable ? in main(), just before calling DialogBox() ? 4) Please tell me if you see things that are bad in the attached code, so that I'll fix it before submitting a patch. Thanks you for your help

7 13

Regression in Rebar
by Maxime Bellengé March 1, 2005

March 1, 2005

Hello This patch is causing a regression in emule, the rebar simply does not display. a+ Maxime Modified files: dlls/comctl32 : rebar.c Log message: Robert Shearman <rob(a)codeweavers.com> - Make row number be zero-based. - Improvements to dumping functions to not dump out fields that may not have been filled in.

2 4

Tools for identifying what a DLL provides?
by David D. Hagood March 1, 2005

March 1, 2005

I have a DLL from a manufacturer that I don't have the source for, nor do I have the header or the import library for. Since Wine developers frequently have to reverse engineer what a given DLL provides, does anybody have a good recommendation for a tool that can scan the DLL and report what functions it exports, and their arg types and return types?

4 4

Re: [wined3d] texture interface separation.
by Oliver Stieber March 1, 2005

March 1, 2005

2 1

Re: Wine cabinet.dll FDI Conformace Test Patch
by Daniel Kegel March 1, 2005

March 1, 2005

Andreas wrote: > It's an unspoken rule that preprocessor directives (#ifdef, ...) > always start at the very beginning of a line to make sure one recognizes > immediately which parts of the code are being processed and which ones aren't. Good point. I should have caught that style violation when I reviewed their code for them this weekend. > AFAICS you're not using wine test suite infrastructure, e.g. ok() etc. > > Using printf() directly strikes me as being quite bad, too... I think they're using the wine test infrastructure properly everywhere. The only exception is the disabled self-test in tvfs.c, but I think that's fine. It's not even built, it was just used while developing the test harness. > Apart from that it appears to be very good. Cool! - Dan

2 1

Re: winedbg questions
by Pouech Eric DMI AEI CAEN March 1, 2005

March 1, 2005

> I'm trying to use winedbg to run the direct sound regression > tests and have two questions: > > Execution stops when starting the first dsound.dll thread. Is > this normal? Is the stack trace with no symbol information > normal? no, because of this in your trace Couldn't initiate DbgHelp the only reason this fails is because dbghelp cannot get to the wine-loader in the running programm try to set the env var WINELOADER to the {pk}loader (full path) you're using and check again A+

1 0

winedbg questions
by Robert Reif March 1, 2005

March 1, 2005

I'm trying to use winedbg to run the direct sound regression tests and have two questions: Execution stops when starting the first dsound.dll thread. Is this normal? Is the stack trace with no symbol information normal? Typing c continues to the exception I am interested in but the stack trace has no symbolic information. This is with a modified dsound.dll that does zeroing of memory before freeing. An unmodified dsound.dll stops at the start but does not get the second exception. This is with a standard RH9 with up to date erratas and this mornings cvs. typescript log attached. Why is winedbg stopping at the first thread creation? Why is there no symbolic stack information? [wine(a)dell tests]$ WINEDEBUG=+wave,+dsound,+tid winedbg "dsound_test.exe.so dsound" WineDbg starting on pid 0xa Couldn't initiate DbgHelp 000b:trace:wave:OSS_WaveInit () 000b:trace:wave:OSS_WaveOutInit (0x41765960) /dev/dsp 000b:trace:wave:OSS_OpenDevice (0x41765960,1,(nil),0,-1,-1,ffffffff) ... 000b:warn:wave:OSS_RawOpenDevice Couldn't open /dev/dsp5 (No such device) 000b:trace:wave:OSS_widMessage (0, DRVM_INIT, 00000000, 00000000, 00000000); 000b:trace:wave:OSS_widMessage (0, WIDM_GETNUMDEVS, 00000000, 00000000, 00000000); 000b:trace:wave:OSS_wodMessage (0, DRVM_INIT, 00000000, 00000000, 00000000); 000b:trace:wave:OSS_wodMessage (0, WODM_GETNUMDEVS, 00000000, 00000000, 00000000); 000b:trace:dsound:DllMain (0x40870000 0x1 0x1) 000b:trace:dsound:DllMain DLL_PROCESS_ATTACH 000f:trace:dsound:DllMain (0x40870000 0x2 (nil)) 000f:trace:dsound:DllMain DLL_THREAD_ATTACH In 32 bit mode. 0x40471f77: pushl %esi Wine-dbg>bt Backtrace: =>1 0x40471f77 (0x4085fff4) 2 0x400298f5 (0x00000000) Wine-dbg>c dsound.c:167:Testing IDirectSound 000b:trace:dsound:DSOUND_DllGetClassObject ({47d4d946-62e8-11cf-93bc-444553540000},{00000001-0000-0000-c000-000000000046},0x4085fe30) 000b:trace:dsound:DSCF_AddRef (0x408a586c) ref was 1 000b:trace:dsound:DSCF_CreateInstance (0x408a586c)->((nil),{279afa83-4981-11ce-a521-0020af0be560},0x4085fe6c) 000b:trace:dsound:DSOUND_Create ((null),0x4085fe6c,(nil)) ... 000b:trace:wave:wodOpen OSS_OpenDevice returned this format: 22050x8x2 000b:trace:wave:wodOpen got 32 2048 byte fragments (46 ms/fragment) 000b:trace:wave:wodOpen fd=16 fragstotal=32 fragsize=2048 BufferSize=65536 001a:trace:wave:wodPlayer_ProcessMessages calling ExitThread() 001a:trace:dsound:DllMain (0x40870000 0x3 (nil)) 001a:trace:dsound:DllMain DLL_THREAD_DETACH First chance exception: page fault on read access to 0x00000000 in 32-bit code (0x401a0d88). In 32 bit mode. Register dump: CS:0023 SS:002b DS:002b ES:002b FS:0033 GS:005f EIP:401a0d88 ESP:419d4528 EBP:419d4540 EFLAGS:00210293( - 00 RISA1C) EAX:00000008 EBX:401d63c4 ECX:00000001 EDX:402f0038 ESI:00000000 EDI:00010000 Stack dump: 0x419d4528: 00000000 00000000 00000000 401d63c4 0x419d4538: 402f0000 00010000 419d456c 401a1819 0x419d4548: 402f0000 00010000 419d455c 00000000 0x419d4558: 402f0000 00000000 405036c8 00010000 0x419d4568: 00000000 419d4584 40457b3a 402f0000 0x419d4578: 00000002 00010000 405036c8 419d45b0 Backtrace: =>1 0x401a0d88 (0x419d4540) 2 0x401a1819 (0x419d456c) 3 0x40457b3a (0x419d4584) 4 0x404561d7 (0x419d45b0) 5 0x404562bd (0x419d45d0) 6 0x404918f5 (0x419d45e4) 7 0x4045ace9 (0x419d45fc) 8 0x4045b07b (0x419d4608) 9 0x4043a0e8 (0x419d4638) 10 0x401a2c9a (0x419d4650) 11 0x401a3dd3 (0x419d46c0) 12 0x401a41a3 (0x419d46e8) 13 0x40486c8f (0x419d47b8) 14 0x401ba077 (0x419d4ff4) 15 0x401116c7 (0x00000000) 0x401a0d88: movl 0x0(%esi),%eax Wine-dbg>q WineDbg terminated on pid 0xa

1 0