June 2007 - Wine-devel - list.winehq.org

Re: [advapi32/tests] Add some service related tests
by Paul Vriens 27 Jun '07

27 Jun '07

On 6/26/07, Paul Vriens <paul.vriens.wine(a)gmail.com> wrote: > Hi, > > While playing with native crypt32/wintrust I added a service (CryptSvc) to the > registry (seems needed for native and XP+). With a full trace I get some crashes > which made me start to write some tests for services. (Rolf Kalbermatter was > already busy with services stuff, but said he doesn't have the time currently). > > Changelog > Add some service related tests > > Cheers, > > Paul. > Ignore this one. I'll come up with something else as I've started fixing things first. Cheers, Paul.

1 1

Re: [PATCH try2 2/2] winemenubuilder: Wait for application icons to be created instead of adding a RunOnce entry that may never run.
by Hans Leidekker 27 Jun '07

27 Jun '07

On Wednesday 27 June 2007, Misha Koshelev wrote: + /* Now watch the directory for our path to be created */ + if ((*handles = FindFirstChangeNotificationW(dir, subtree, FILE_NOTIFY_CHANGE_FILE_NAME)) == + INVALID_HANDLE_VALUE) I'm not sure if file change notifications are the right solution here but since they are only supported on Linux you should at least keep the run once key as a fallback strategy. -Hans

4 3

Re: [3/3] gdi32: improved PolyDraw
by Alexandre Julliard 27 Jun '07

27 Jun '07

"Evan Stade" <estade(a)gmail.com> writes: > + for( i = dc->path.numEntriesUsed-1; i >= 0; i-- ){ > + if(dc->path.pFlags[i] == PT_MOVETO){ > + lastmove = dc->path.pPoints[i]; > + break; > + } > + } > + if(i < 0){ > + lastmove.x = 0; > + lastmove.y = 0; > + } This can't work, i is unsigned. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: msi [2/6]: Add the IWineMsiRemotePackage interface
by Alexandre Julliard 27 Jun '07

27 Jun '07

"James Hawkins" <truiken(a)gmail.com> writes: > +interface IWineMsiRemotePackage : IUnknown > +{ > + HRESULT SetMsiHandle( [in] ULONG_PTR handle ); > + HRESULT GetActiveDatabase( [out] ULONG_PTR handle ); > + HRESULT GetProperty( [in] BSTR *property, [out] BSTR *value, [out] ULONG_PTR size ); Why are you using ULONG_PTR instead of the proper pointer types for out parameters? -- Alexandre Julliard julliard(a)winehq.org

1 0

GDI/GDI+ creating paths
by Evan Stade 27 Jun '07

27 Jun '07

Hi, I was talking with Julliard about the implementation of paths in GDI+. He suggested that the right way to do it was to build paths directly (i.e. not calling into GDI). I'm not sure why calling into GDI is undesirable. There are several reasons I'd like to use GDI paths: * GDI and GDI+ paths seem to behave exactly the same way * this implementation of GDI+ calls into GDI for drawing, so it seems natural to rely on GDI to create paths as well. * unless we take rendering completely out of the hands of GDI, we need GDI to draw paths for us because of line joins. It seems strange to me to draw shapes and draw paths with GDI, but not to create paths with GDI. * path.c in gdi32 is 2000 lines, and re-implementing it seems overkill I don't see any drawbacks of calling into GDI to make paths, other than a slight performance decrease introduced by an extra level of indirection. Can anyone suggest another reason why this approach is flawed? -- Evan Stade

2 2

Re: [SPAM] Problem with patch "dsound: Simplify mixing by removing remixing support, and fix its waveout breakage"
by Alex Villacís Lasso 27 Jun '07

27 Jun '07

Chris Rankin escribió: > Hi, > > This patch has killed the sound on World of Warcraft. Instead, I now get silence and a stream of > errors like this: > > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > err:dsound:DSOUND_CalcPlayPosition Bad length in CalcPlayPosition! > > The following change makes the sound work again without error messages, but there is still a brief > burst of sound/noise before the music starts properly: > > $ cvswine diff dlls/dsound/buffer.c > Index: dlls/dsound/buffer.c > =================================================================== > RCS file: /home/wine/wine/dlls/dsound/buffer.c,v > retrieving revision 1.70 > diff -u -r1.70 buffer.c > --- dlls/dsound/buffer.c 22 Jun 2007 12:37:22 -0000 1.70 > +++ dlls/dsound/buffer.c 25 Jun 2007 19:08:34 -0000 > @@ -442,15 +442,15 @@ > pmix *= This->pwfx->nBlockAlign; > > TRACE("this back-offset=%d\n", pmix); > - > +#if 0 > /* sanity */ > if(pmix > This->buflen){ > ERR("Bad length in CalcPlayPosition!\n"); > return 0; > } > - > +#endif > /* subtract from our last mixed position */ > - if (bplay < pmix) bplay += This->buflen; /* wraparound */ > + while (bplay < pmix) bplay += This->buflen; /* wraparound */ > bplay -= pmix; > > /* check for lead-in */ > > > Cheers, > Chris > > > > > > > ___________________________________________________________ > New Yahoo! Mail is the ultimate force in competitive emailing. Find out more at the Yahoo! Mail Championships. Plus: play games and win prizes. > http://uk.rd.yahoo.com/evt=44106/*http://mail.yahoo.net/uk > > You might try the patch with the title "DSOUND: recover from mismatch between time spanned by primary buffer being longer than the one spanned by secondary buffer", which I sent along with some other patches today. It looks somewhat similar to the one you sent. Alex Villacís Lasso -- perl -e '$x=2.4;print sprintf("%.0f + %.0f = %.0f\n",$x,$x,$x+$x);'

2 1

Writing tests in Wine.
by Anatoly Lyutin 26 Jun '07

26 Jun '07

Hi all! When I analyzed Wine test codes today I do not understand how to interpret result of tests.That is why I refer to community. I can not find any full documentation for writing tests at WineHq. I have some points which I do not understand: 1. Why some tests are running in Win not fall if expected and obtained messages are not coincide. ex (user32:msg:test_mdi_messages:Win2k3): msg.c:2716:mdi frame: 00090362, 0086, 00000000, 00000000 msg.c:2716:mdi frame: 00090362, 0002, 00000000, 00000000 msg.c:2716:mdi frame: 00090362, 0082, 00000000, 00000000 msg.c:3302:expected 0004 - actual 0004 msg.c:3302:expected 0046 - actual 0046 msg.c:3302:expected 8003 - actual 0047 msg.c:3302:expected 0086 - actual 0047 msg.c:3302:expected 0047 - actual 0047 msg.c:3302:expected 0086 - actual 0086 msg.c:3302:expected 0006 - actual 0002 msg.c:3302:expected 001c - actual 0002 msg.c:3302:expected 8003 - actual 0002 msg.c:3302:expected 8001 - actual 0002 msg.c:3302:expected 0002 - actual 0002 msg.c:3302:expected 0082 - actual 0082 Why does not this fail?(ok_sequence(WmDestroyMDIframeSeq, "Destroy MDI frame window", FALSE);) 2. How I can write test for different OS Windows? Some tests run successfully in win2k3 but fail in XP, that is why this test can not be validated by HQ. 3.What does it mean :{ HCBT_SETFOCUS, hook }? What does this hook in some tests make fail in Win2k3 if it was included? Thank you for attention. -- Best regards Anatoly Lyutin.

2 1

Re: winefile: Use the msvcrt provided _tsplitpath instead of reimplementing it.
by Alexandre Julliard 26 Jun '07

26 Jun '07

Michael Stefaniuc <mstefani(a)redhat.de> writes: > Actually i wanted to just remove the wide character constants out of > splitpath.c but then i have seen that the file can be removed completely. > I checked the history but didn't find a clue why a separate > implementation might have been usefull. splitpath.c appeared with the > initial import of the winefile source code. Winefile cannot import msvcrt because it has support for accessing Unix files too. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: please help me with filling a bug concerning WM_ERASEBKGND
by Juan Lang 26 Jun '07

26 Jun '07

Hi, if you've done a reasonable job searching for a dup and haven't found one, go ahead and file a new bug. Bugs can be closed or linked where appropriate. > is this http://bugs.winehq.org/show_bug.cgi?id=6006 ? > Il try to compile latest git with patches from that bug before filling > new one. I think these may very well be related, but (IMO) they should be different bugs. A bug should describe the problem, not the fix. If a patch for one fixes the other, great! --Juan ____________________________________________________________________________________ Got a little couch potato? Check out fun summer activities for kids. http://search.yahoo.com/search?fr=oni_on_mail&p=summer+activities+for+kids&…

1 0

Slackware 11: After Wine update applications crash if WINEDEBUG not set
by Victor 26 Jun '07

26 Jun '07

Hello! I didn't get an answer for this in wine-user, so I've decided to ask this question here. After upgrading wine (from 0.9.37 to 0.9.38 or from 0.9.38 to 0.9.39 - I'm not sure) many graphical applications started crashing if WINEDEBUG+=all or WINEDEBUG+=relay not set. Among those applications are Postal 1, Kiosaky's Cashflow 101, .kkreiger and probably more. I'm not sure if this is a bug in Wine or a problem in my system, because uninstalling and downgrading (even to 0.9.34) didn't help - problem now exists in other versions as well. Applications crash in a "windows way" - with access violation at address 0x7XXXXXXX. Here is a beginning of output produced by trying to start .kkreiger (avaliable at http://www.theprodukkt.com/) --begin-- Unhandled page fault on read access to 0xffffffff at address 0x7d66c9ce (thread 0009), starting debugger... Unhandled exception: page fault on read access to 0xffffffff in 32-bit code ?0x7d66c9ce). Register dump: CS:0073 SS:007b DS:007b ES:007b FS:0033 GS:003b EIP:7d66c9ce ESP:0033e8a8 EBP:00009f80 EFLAGS:00210246( - 00 -RIZP1) EAX:0033e8c8 EBX:7d68a000 ECX:00000000 EDX:00000000 ESI:01000000 EDI:00001f80 Stack dump: 0x0033e8a8: 0033e8c8 00000000 00000000 b7fd6ff8 0x0033e8b8: 7d72aaa9 7d72aaa9 0033ec18 00001f80 0x0033e8c8: 00000000 00000000 00000000 00000000 0x0033e8d8: 00000000 00000000 00000000 00000000 0x0033e8e8: 00000000 00000000 00000000 00000000 0x0033e8f8: 00000000 00000000 00000000 00000000 Backtrace: =>1 0x7d66c9ce (0x00009f80) 2 0x00000000 (0x00000000) 0x7d66c9ce: --end-- But it works fine if started as WINEDEBUG=+relay wine wine pno001.exe &>/dev/null This application worked fine before and didn't need those flags. Kyosaky's Cashflow 101 crashes with a windows message box which says "Unhandled exceptionL c0000005 At address: 7ce2f9ce". but works fine if started with WINEDEBUG=+relay same thing with Postal 1. There are also problems with an Alien vs Predator 2 game and some other apps that enumerate D3D display modes: if program has a listbox where videomodes can be selected, this box will contain no strings (alien vs. predator, ragnarok online, some other apps), if WINEDEBUG+=all is not set. Can these problems be connected with those crashes? System configuration: Slackware 11 linux with 2.6.17.13 kernel on AMD Sempron 2800+ processor with GeForce 7100GS and 512Mb ram. Any pointers how can I figure out what's wrong? Thanks. With best regards, Victor Eremin.

2 3