September 2006 - Wine-Devel

Re: secur32: Add a warning if ntlm_auth is not found or in a wrongversion.
by Dmitry Timoshkov Sept. 17, 2006

Sept. 17, 2006

"Kai Blin" <kai.blin(a)gmail.com> wrote: > + else > + { > + WARN("ntlm_auth not found, or in an outdated version.\n" \ > + "Make sure you are using ntlm_auth =< 3.x and it is in your path.\n"); > + } If the message is supposed to be user visible (otherwise why to bother with wording?) it would be better to use either FIXME or MESSAGE. Also "=<" is not really understandable in this context. Did you mean that a version 3.x or better is required (then >= 3.x), or <= 3.x (then the word "outdated" is misleading)? -- Dmitry.

2 1

How to report code errors?
by Andrew Talbot Sept. 16, 2006

Sept. 16, 2006

How should one report minor code errors that don't necessarily produce known bad behaviour? To give a concrete example, in msacm32:driver.c acmDriverPriority(), formal parameter dwPriority is of type DWORD, which is unsigned, but it contains a line that reads: if (dwPriority < 0) dwPriority = -1; ... which can never happen. Bugzilla seems inappropriate, so does wine-devel. If Coverity has abandoned us, would it be worth having a web/wiki page for posting any such code anomalies, so that the knowledgable might consider fixing them? Thanks, -- Andy.

3 4

Already Error while installing MS Office 2003
by Roland Kaeser Sept. 16, 2006

Sept. 16, 2006

Hello Just tried to test install office 2003 on wine 0.9.21. But getting already errors. I read in C't (german it magazine) that office 2003 works on codeweavers. Can somebody declare me this? Does codeweavers voluntary not provide the corrections to wine to sell their product? Looks very strange to me. Roland Debug Output while installing office (breaks down when it trieds to start installation of the files (after setup configuration and serial number request) --------------------------------------------------------------------------------------------------------- fixme:imm:ImmDisableIME (-1): stub fixme:advapi:CheckTokenMembership ((nil) 0x190388 0x346d3c) stub! err:ole:CoUninitialize Mismatched CoUninitialize err:ole:CoUninitialize Mismatched CoUninitialize fixme:advapi:CheckTokenMembership ((nil) 0x1ad6f8 0x7d3683b8) stub! err:msidb:WHERE_VerifyCondition Couldn't find column L"Lock" fixme:msi:msi_unimplemented_action_stub MigrateFeatureStates -> 67 ignored L"Upgrade" table values err:msidb:WHERE_VerifyCondition Couldn't find column L"Lock" err:msi:msi_dialog_set_font No font entry for L"52" err:msi:msi_dialog_set_font No font entry for L"9" err:msi:msi_dialog_set_font No font entry for L"52" fixme:msi:ControlEvent_SpawnWaitDialog Doing Nothing fixme:msi:ControlEvent_SpawnWaitDialog Doing Nothing err:msi:msi_dialog_set_font No font entry for L"100" err:heap:HEAP_ValidateInUseArena Heap 0x110000: in-use arena 0x1ed990 next block has PREV_FREE flag fixme:msi:msi_unimplemented_action_stub MigrateFeatureStates -> 67 ignored L"Upgrade" table values err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\1031\\OFREADME.HTM") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\1031\\OFREADME.HTM") fixme:msi:ACTION_HandleStandardAction unhandled standard action L"SetODBCFolders" fixme:msi:deformat_component component key L"HTM_Editor_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"HTM_Editor_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"Core_IDE_Resource_DLLs_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"Core_IDE_Resource_DLLs_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"Core_IDE_Cmd_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"Core_IDE_Cmd_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"VS_SCC_Core_TeamCore_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"VS_SCC_Core_TeamCore_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"CSS_Package_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"CSS_Package_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"_VS_Debugging_SDM2_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"_VS_Debugging_SDM2_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"_VS_Debugging_VSDebug_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"_VS_Debugging_VSDebug_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"WbBrwsrPckgRsrcDLLDEUX86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"WbBrwsrPckgRsrcDLLDEUX86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"URL_Font_Color_Picker_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" fixme:msi:deformat_component component key L"URL_Font_Color_Picker_DEU_X86.3643236F_FC70_11D3_A536_0090278A1BB8" err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\CSSPKGUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\CSSPKGUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\HTMEDUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\HTMEDUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\htmdlgsUI.dll") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\htmdlgsUI.dll") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\vsdebugui.dll") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\vsdebugui.dll") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\MSDBGUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\MSDBGUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\MSENVUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\MSENVUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\vsbrowseUI.dll") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\vsbrowseUI.dll") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\VisualStudioTeamCoreUI.dll") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\VisualStudioTeamCoreUI.dll") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\CMDDEFUI.DLL") err:msi:deformat_file Unable to get ShortPath size (L"c:\\Programme\\Microsoft Office\\OFFICE11\\VS Runtime\\1031\\CMDDEFUI.DLL") err:msi:ITERATE_Actions Execution halted, action L"SKURED" returned 1603 err:msi:ITERATE_Actions Execution halted, action L"ExecuteAction" returned 1603 fixme:msi:msi_dialog_set_control_condition Unhandled action L"Default" fixme:msi:msi_dialog_set_control_condition Unhandled action L"Default"

10 14

Re: winex11.drv: cleanup better in X11DRV_WineGL_InitOpenglInfo
by Jesse Allen Sept. 16, 2006

Sept. 16, 2006

On 9/14/06, Phil Costin <philcostin(a)hotmail.com> wrote: > This patch fixes many games for me, what is the reason it was not applied? > > Regards > Phil Costin > > I don't know myself either. I was just going to wait. I want to add a little more to say about it. It's funny that the patch was only meant to affect the opengl/wgl code and the crashing apps are all d3d ones. However winex11 will now change the context on it's own when doing the opengl query and not set it back. What I noticed a couple weeks ago and now is that winex11 does usually gets inited twice, therefore it will change the current context twice. One at the start of the program, and another later on -- maybe on creation of a window or something. By that time wined3d is probably inited too, and making glXMakeCurrent calls too since it's based on opengl. So perhaps winex11 is causing wined3d to trip up at some point by changing the context. So if InitOpenglInfo just sets things back, then wined3d won't go boom. >From comments in wined3d, I can tell there isn't any "context manager". So what wined3d does at most is very simple management of contextes. We probably need a manager to coordinate things like this better between the two libraries. But that will likely be a bigger patch and to do so... I don't know, we need a fix for all the broken d3d apps now. Jesse

2 1

Re: X11DRV: fix fbconfig regression
by Roderick Colenbrander Sept. 16, 2006

Sept. 16, 2006

Ati doesn't advertise GLX_SGIX_fbconfig. They only advertise a GLX client version of 1.3. Because the Xserver only supports 1.2, the reported version is also 1.2 (only the client version number is 1.3, the rest not). Roderick > Roderick Colenbrander wrote: > > The issue is that ATI's drivers don't advertise GLX_SGIX_fbconfig (which > really all opengl drivers do) but on the other hand they advertise GLX > client version 1.3 which promoted GLX_SGIX_fbconfig to core functionality. So > ATI doesn't advertise GLX_SGIX_fbconfig but it is available because their > libs are 1.3. They only ship the 1.3 functions so not the SGIX ones. > > > > So, ATI correctly advertises what they support, I don't see any reason > to have workarounds there. > > tom -- "Feel free" - 10 GB Mailbox, 100 FreeSMS/Monat ... Jetzt GMX TopMail testen: http://www.gmx.net/de/go/topmail

1 0

configure error
by Tijl Coosemans Sept. 16, 2006

Sept. 16, 2006

I get the following error running configure on FreeBSD: config.status: creating Makefile sed: 14: ./confstathEGAkR/subs-4.sed: unescaped newline inside substitute pattern There's one of those for every Makefile. The problem is with the following lines in configure.ac where I had to add '\\' to every line to work around the problem. Is this a FreeBSD only thing? AC_SUBST(DEPENDENCIES,"### Dependencies:\\ \\ .INIT: Makefile\\ .BEGIN: Makefile\\ Makefile: dummy\\ -\$(MAKEDEP) -C\$(SRCDIR) -S\$(TOPSRCDIR) -T\$(TOPOBJDIR) \$(EXTRAINCL) \$(DEPEND_SRCS)\\ \\ \$(ALL_OBJS): \$(IDL_SRCS:.idl=.h)\\ \$(LEX_SRCS:.l=.yy.o): \$(LEX_SRCS:.l=.yy.c)\\ \$(BISON_SRCS:.y=.tab.o): \$(BISON_SRCS:.y=.tab.c)")

2 1

Re: X11DRV: fix fbconfig regression
by Roderick Colenbrander Sept. 16, 2006

Sept. 16, 2006

> Roderick Colenbrander wrote: > > There's this check: > > if ((!WineGLInfo.glxDirect && !strcmp("1.2", > WineGLInfo.glxServerVersion)) || > > (WineGLInfo.glxDirect && !strcmp("1.2", > WineGLInfo.glxClientVersion))) > > > This is not the correct way of loading opengl functions or deciding > whether they are available or not. According to the > GLX_ARB_get_proc_address spec, we need to check _only_ > glXQueryExtensionsString() and glXQueryVersion() and it's not correct to > make assumptions based on server/client versions or extension strings. > > This patch removes all the client/server code and replaces it with the > correct checks. The patch fixes the code in has_opengl() (and also > replaces one wine_dlsym() with pglXGetProcAddressARB()) as well as the > helper function glxRequireExtension() and fixes the code that decices > whether pbuffers are available. Pbuffers are part of GLX 1.3, so if the > GLX version is 1.3 we _have_ pbuffers, no need to check for the > GLX_SGIX_pbuffers extension anymore (&& -> ||) > > If this patch works with ATI drivers, I don't know, but that's certainly > the correct way of querying functions. I'm aware of the GLX version / extension issues. The check is more or less a workaround for buggy drivers like ATI's :( I'm against workarounds but in this case without it, there won't be any OpenGL support on ATI's drivers as GLX_SGIX_fbconfig is a requirement for wine. (I have submitted a bugreport) BTW I had the same stuff in opengl32. The issue is that ATI's drivers don't advertise GLX_SGIX_fbconfig (which really all opengl drivers do) but on the other hand they advertise GLX client version 1.3 which promoted GLX_SGIX_fbconfig to core functionality. So ATI doesn't advertise GLX_SGIX_fbconfig but it is available because their libs are 1.3. They only ship the 1.3 functions so not the SGIX ones. When direct rendering is enabled we look at the GLX client version :( This only works for fbconfig stuff and not for other things like pbuffers (it does work on ATI cards but not on Mesa). I hope this clears it up, Roderick -- "Feel free" - 10 GB Mailbox, 100 FreeSMS/Monat ... Jetzt GMX TopMail testen: http://www.gmx.net/de/go/topmail

1 0

Re: Already Error while installing MS Office 2003
by jimtabor Sept. 16, 2006

Sept. 16, 2006

Hi! > Hello > > >Codeweavers version of Wine has changes that Alexandre deems > >unacceptable for the "clean" WineHQ tree. > > If this is that way, it would be real a horror. Why are code patches > which allows a huge major app (key application) to run on linux > "unacceptable" (From point of view of the MS users). Do we here make > trench warfares? Or are there quality conceivabilities which are > completely unrealistic? Shouldn't we not see the major goal of wine? > To run windows apps on linux? Not to have a few lines code a bit > better quality etc. I think we don't have the choice to reject code. > Be very very thankful for each line which is checked in!! I think some > responsible persons are to deep in the code. They are looking just > into the code but can't see the benefit of the code for the wine > project itself. Why don't just accept the code? There is enough time > later to make it more "beautiful" or correct it to a better quality. > But for the moment, some of the code directly allows important apps to > work. > > Roland > > > I checked the Codeweavers version of Wine, there are a lot of "Codeweaver "HACK HACK HACK" Alerts" everywhere in key dlls. I compared it with 0.9.16 since the Codeweavers wine VERSION stated it. Amazing! James > >----- Ursprüngliche Mail ---- > >Von: Jim White <jim(a)pagesmiths.com> > >An: Roland Kaeser <roli8200(a)yahoo.de> > >CC: wine-devel(a)winehq.com > >Gesendet: Freitag, den 15. September 2006, 19:41:45 Uhr > >Betreff: Re: Already Error while installing MS Office 2003 > > > >Roland Kaeser wrote: > > > >> Just tried to test install office 2003 on wine 0.9.21. But getting > >> already errors. I read in C't (german it magazine) that office 2003 > >> works on codeweavers. Can somebody declare me this? Does codeweavers > >> voluntary not provide the corrections to wine to sell their product? > >> Looks very strange to me. > >> ... > > > >Codeweavers version of Wine has changes that Alexandre deems > >unacceptable for the "clean" WineHQ tree. > > > >The Codeweavers Wine source is here: > > > >http://www.codeweavers.com/products/source/ > > > >Jim > > >------------------------------------------------------------------------ >

1 0

[OT] Wine reveals bug in Microsoft Installer
by Segin Sept. 16, 2006

Sept. 16, 2006

I was recently just feeding random crap to Wine, in an attempt to "stress-test" Wine, for a lack of better words. I fed it the Microsoft Platform SDK Installer, and it revealed a bug in the program, probably related to translation. Image URI: http://segin.no-ip.org/wtf.png -- The real problem with C++ for kernel modules is: the language just sucks. -- Linus Torvalds

1 0

Re: X11DRV: fix fbconfig regression
by Tomas carnecky Sept. 16, 2006

Sept. 16, 2006

Roderick Colenbrander wrote: > There's this check: > if ((!WineGLInfo.glxDirect && !strcmp("1.2", WineGLInfo.glxServerVersion)) || > (WineGLInfo.glxDirect && !strcmp("1.2", WineGLInfo.glxClientVersion))) This is not the correct way of loading opengl functions or deciding whether they are available or not. According to the GLX_ARB_get_proc_address spec, we need to check _only_ glXQueryExtensionsString() and glXQueryVersion() and it's not correct to make assumptions based on server/client versions or extension strings. This patch removes all the client/server code and replaces it with the correct checks. The patch fixes the code in has_opengl() (and also replaces one wine_dlsym() with pglXGetProcAddressARB()) as well as the helper function glxRequireExtension() and fixes the code that decices whether pbuffers are available. Pbuffers are part of GLX 1.3, so if the GLX version is 1.3 we _have_ pbuffers, no need to check for the GLX_SGIX_pbuffers extension anymore (&& -> ||) If this patch works with ATI drivers, I don't know, but that's certainly the correct way of querying functions. tom

1 1

Note and patch about strange UDP packets
by Florian Echtler Sept. 15, 2006

Sept. 15, 2006

Hi everybody, we are using a certain tracking software from Advanced Realtime Tracking. This is a windows software, but we decided yesterday that it would be more flexible to be able to use it under Linux, too, so I gave it a try with Wine. As this software communicates with customized (expensive) cameras via UDP, I noted a strange bug in the Wine debug channel "winsock": the software was sending UDP packets to IP _254_.255.255.255 (note the first octet). After some experimentation, I noticed that while Windows just sends this as a broadcast packet, Linux rejects this address, because the corresponding RFC states that the network 240.0.0.0/4 is reserved and should not be used. I therefore build a small patch which checks for these strange adresses and replaces them by broadcast adresses. This enabled our software to work. However, the question now is whether it makes sense to include this patch into Wine? On the one hand, it increases the similarity between Wine's behaviour and Windows's, OTOH it decreases Wine's RFC compliance.. Comments? Thanks, Yours, Florian

2 1

Re: [PATCH] PATCH: kernel 2.6 in README.de
by n0dalus Sept. 15, 2006

Sept. 15, 2006

On Wed Sep 13 10:57:33 2006 +0200, Marcus Meissner <marcus(a)beiboot.franken.de> wrote: > Mention kernel 2.6. > > --- > > documentation/README.de | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > 10857c3dbb8429ea599288ac93f4cecd8ffeff59 > diff --git a/documentation/README.de b/documentation/README.de > index c4a1324..680e077 100644 > --- a/documentation/README.de > +++ b/documentation/README.de > @@ -45,7 +45,7 @@ Linux-Info: > Obwohl Linux 2.2.x immer noch funktionieren sollte und Linux 2.0.x > vielleicht auch noch funktioniert (ältere 2.0.x-Versionen verzeichneten > Abstürze im Zusammenhang mit Threads), sollte man am besten einen > - aktuellen Kernel wie 2.4.x verwenden. > + aktuellen Kernel wie 2.4.x oder 2.6.x verwenden. > > FreeBSD-Info: > Stellen Sie sicher, dass Sie die Optionen USER_LDT, SYSVSHM, SYSVSEM und > -- > 1.2.4 > > > Hi, A few of the emails you've sent to the wine-patches mailing list lately have not had 'To:' headers. Many people have their mail clients configured to filter emails into folders based on 'To:' fields, so it would be great if you could sort that out (your last few ones, such as this one, ended up in my inbox instead of my wine folder). It probably means that whatever program you are using to send these emails is broken slightly. I hope you are able to work it out, n0dalus.

5 5

Troubles using current wine (wine-0.9.21)
by Louis. Lenders Sept. 15, 2006

Sept. 15, 2006

Hi, i'm having lots of troubles running games with current git. First of all i get these messages running every application i try: "err:wgl:has_opengl glx_version as 1.3 and GLX_SGIX_fbconfig extension is unsupported. Expect problems." Is it really that bad to not have glx_version 1.3? I have Ati radeon 9600 and i do not know if this is supported on my card anyway. Further, games that used to work now crash, for example Serious Sam I. The problem seems to be: 000f:Call gdi32.ChoosePixelFormat(000004f0,0033fc38) ret=004c70b9 000f:Call winex11.drv.ChoosePixelFormat(001aab78,0033fc38) ret=605c1358 trace:seh:raise_exception code=c0000005 flags=0 addr=(nil) --> crash Any hints on this? --------------------------------- Try the all-new Yahoo! Mail . "The New Version is radically easier to use" The Wall Street Journal

4 3

Re: advapi32, msi, rsaenh: MD4_CTX and MD5_CTX have the same elements, declare them in the same order.
by Hans Leidekker Sept. 15, 2006

Sept. 15, 2006

On Thursday 14 September 2006 11:13, Kai Blin wrote: > MD4_CTX and MD5_CTX have the same elements, declare them in the same order. Tell Microsoft about it ;-) Your patch breaks the md5 test on Windows because it depends on the struct layout: crypt_md5.c:90: Test failed: invalid initialization crypt_md5.c:93: Test failed: update doesn't work correctly crypt_md5.c:96: Test failed: update doesn't work correctly crypt_md5: 5 tests executed, 0 marked as todo, 3 failures. -Hans

3 5

rpcrt4 questions
by Damjan Jovanovic Sept. 15, 2006

Sept. 15, 2006

1. I was surprised to discover DCE RPCs (as opposed to DCOM) from wine to Windows do work. Is this documented anywhere? 2. Is there any reason why rpc_transport.c uses native sockets instead of winsock? 3. Would a patch to fix (2) be welcome? With a few more changes, it would also allow an RPC server to run on wine, as opposed to the current case where only clients work. Thank you Damjan __________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com

1 0

Re: wine 0.9.21 - World of warcraft appears to be broken on ATI hardware ? crashes with Blizzard critical error
by Nick Law Sept. 15, 2006

Sept. 15, 2006

Vitaliy Margolen wrote: > Nick Law wrote: > >> I'm having the same problems with 0.9.21 here, World of Warcraft is >> broken, no longer runs at all on ATI hardware, (confirmed on two >> separate systems running with ATO 9550. (maybe ok on Nvidia hardware >> although not tried it myself). >> >> > Nick please open a bug in bugzilla, and do proper regression testing. > > Vitaliy. > > ok, I'll see what I can do over the next few days. Nick

1 0

Re: TRY 2: Socket resets randomly
by Vitaliy Margolen Sept. 15, 2006

Sept. 15, 2006

Ryan Miller wrote: > ws2_32: Fixed bug with sockets randomly resetting due to GetLastError being > reset by setsockopt in Windows, but not in Wine. Also included is the > associated test. > > + SetLastError(1); Please use more accepted value of 0xdeadbeef Vitaliy

1 0

wine 0.9.21 - World of warcraft appears to be broken on ATI hardware ? crashes with Blizzard critical error
by Nick Law Sept. 14, 2006

Sept. 14, 2006

I'm having the same problems with 0.9.21 here, World of Warcraft is broken, no longer runs at all on ATI hardware, (confirmed on two separate systems running with ATO 9550. (maybe ok on Nvidia hardware although not tried it myself). Upon trying to start WoW, a Blizzard dialog box appears which contains the following ( see my post http://appdb.winehq.org/appview.php?iVersionId=5606) Here's a summary World of WarCraft (build 5595) Exe: C:\Program Files\World of Warcraft\WoW.exe Time: Sep 14, 2006 10:58:59.684 PM User: guest Computer: linux ------------------------------------------------------------------------------ This application has encountered a critical error: ERROR #132 (0x85100084) Fatal Exception Program: C:\Program Files\World of Warcraft\WoW.exe Exception: 0xC0000005 (ACCESS_VIOLATION) at 0073:00000000 The instruction at "0x00000000" referenced memory at "0x00000000". The memory could not be "read". WoWBuild: 5595 ------------------------------------------------------------------------------ ---------------------------------------- x86 Registers ---------------------------------------- EAX=7C093970 EBX=6103A374 ECX=00000003 EDX=610323CE ESI=00000000 EDI=7C340D00 EBP=0034FAF8 ESP=0034FA9C EIP=00000000 FLG=00210297 CS =0073 DS =007B ES =007B SS =007B FS =003B GS =0033 ---------------------------------------- Stack Trace (Manual) ---------------------------------------- Address Frame Logical addr Module 00000000 0034FAF8 0000:00000000 C:\Program Files\World of Warcraft\WoW.exe 61009DC0 0034FB58 0001:00038DC0 c:\windows\system\winex11.drv 6070C1FF 0034FB88 0001:0003B1FF c:\windows\system\gdi32.dll 0058C0DB 0034FC14 0001:0018B0DB C:\Program Files\World of Warcraft\WoW.exe 0058BF2C 0034FC24 0001:0018AF2C C:\Program Files\World of Warcraft\WoW.exe 0058CAA6 0034FC74 0001:0018BAA6 C:\Program Files\World of Warcraft\WoW.exe 00591B75 0034FC80 0001:00190B75 C:\Program Files\World of Warcraft\WoW.exe 0058C801 0034FCA0 0001:0018B801 C:\Program Files\World of Warcraft\WoW.exe 005896D7 0034FCB4 0001:001886D7 C:\Program Files\World of Warcraft\WoW.exe 00639E73 0034FD20 0001:00238E73 C:\Program Files\World of Warcraft\WoW.exe 004027EC 0034FE68 0001:000017EC C:\Program Files\World of Warcraft\WoW.exe 004021E0 0034FE78 0001:000011E0 C:\Program Files\World of Warcraft\WoW.exe 0040410E 0034FF08 0001:0000310E C:\Program Files\World of Warcraft\WoW.exe 603A6B81 0034FFE8 0001:00055B81 c:\windows\system\kernel32.dll ---------------------------------------- Stack Trace (Using DBGHELP.DLL) ---------------------------------------- 61009DC0 winex11.drv X11DRV_DescribePixelFormat+448 (0x001A9110,0x00000001,0x00000028,0x0034FBDC) (opengl.c,1004) 6070C1FF gdi32.dll DescribePixelFormat+175 (0x0000032C,0x00000001,0x00000028,0x0034FBDC) (painting.c,424) 0058C0DB WoW.exe +0 (0x0034FC74,0x00000000,0x0034FC74,0x0058CAA6) 0058BF2C WoW.exe +0 (0x0034FC3C,0x000002E4,0x011D0F30,0x011D0008) 0058CAA6 WoW.exe +0 (0x0034FCCC,0x0034FCA0,0x0058C801,0x0042CFD0) 00591B75 WoW.exe +0 (0x0042CFD0,0x0034FCCC,0x0034FD04,0x0042CFD0) 0058C801 WoW.exe +0 (0x0042CFD0,0x0034FCCC,0x00000000,0x0034FD20) 005896D7 WoW.exe +0 (0x0034FCCC,0x00401000,0x00000000,0x603D9968) 00639E73 WoW.exe +0 (0x00000001,0x6C726F57,0x666F2064,0x72615720) 004027EC WoW.exe +0 (0x00000001,0x00000001,0x0034FF08,0x0040410E) 004021E0 WoW.exe +0 (0x00409990,0x00400000,0x00000000,0x0011646C) 0040410E WoW.exe +0 (0x7FFDF000,0x00000000,0x00000000,0x00000000) 603A6B81 kernel32.dll start_process+209 (0x00000000,0x00000000,0x00000000,0x00000000) (process.c,822) 60020247 wine_switch_to_stack+23 (0x00000000,0x00000000,0x00000000,0x00000000) ---------------------------------------- Loaded Modules ---------------------------------------- 0x00350000 - 0x003E6000 C:\Program Files\World of Warcraft\fmod.dll 0x00400000 - 0x00D05000 C:\Program Files\World of Warcraft\WoW.exe 0x10000000 - 0x10069000 C:\Program Files\World of Warcraft\DivxDecoder.dll 0x60290000 - 0x602F7000 c:\windows\system\ntdll.dll 0x60350000 - 0x60435000 c:\windows\system\kernel32.dll 0x604B0000 - 0x604E3000 c:\windows\system\advapi32.dll 0x604F0000 - 0x60592000 c:\windows\system\comctl32.dll 0x605B0000 - 0x606B4000 c:\windows\system\user32.dll 0x606D0000 - 0x6073C000 c:\windows\system\gdi32.dll 0x60750000 - 0x60819000 c:\windows\system\shell32.dll 0x60830000 - 0x6086B000 c:\windows\system\shlwapi.dll 0x60880000 - 0x608F2000 c:\windows\system\ole32.dll 0x60900000 - 0x6093E000 c:\windows\system\rpcrt4.dll 0x60950000 - 0x6095B000 c:\windows\system\iphlpapi.dll 0x60960000 - 0x60974000 c:\windows\system\wsock32.dll 0x609A0000 - 0x609B7000 c:\windows\system\ws2_32.dll 0x609D0000 - 0x60A29000 c:\windows\system\opengl32.dll 0x60C00000 - 0x60C14000 c:\windows\system\imm32.dll 0x60D70000 - 0x60DDE000 c:\windows\system\winmm.dll 0x60DF0000 - 0x60E02000 c:\windows\system\msacm32.drv 0x60E10000 - 0x60E63000 c:\windows\system\msvcrt.dll 0x60E70000 - 0x60EA4000 c:\windows\system\wininet.dll 0x60EB0000 - 0x60EC3000 c:\windows\system\mpr.dll 0x60FD0000 - 0x61043000 c:\windows\system\winex11.drv 0x6A250000 - 0x6A27A000 c:\windows\system\wineoss.drv 0x6AEC0000 - 0x6AEE4000 c:\windows\system\uxtheme.dll 0x6AFB0000 - 0x6AFBD000 c:\windows\system\midimap.dll 0x6B1F0000 - 0x6B221000 c:\windows\system\dbghelp.dll 0x6B230000 - 0x6B236000 c:\windows\system\psapi.dll ---------------------------------------- Memory Dump ---------------------------------------- Code: 16 bytes starting at (EIP = 00000000) 00000000: Stack: 1024 bytes starting at (ESP = 0034FA9C) * = addr ** * 0034FA90: 60 86 09 7C DC FB 34 00 F8 FA 34 00 54 5F 00 61 `..|..4...4.T_.a 0034FAA0: 70 39 09 7C 00 00 18 00 0B 80 00 00 E4 FA 34 00 p9.|..........4. 0034FAB0: B4 05 1C 60 00 28 27 60 E4 FA 34 00 E0 FA 34 00 ...`.('`..4...4. 0034FAC0: 34 CC 03 61 CE 23 03 61 27 00 00 00 01 00 00 00 4..a.#.a'....... 0034FAD0: 00 00 00 00 40 FB 34 00 01 00 00 00 70 39 09 7C ....(a).4.....p9.| <mailto:....(a).4.....p9.%7C> 0034FAE0: 00 00 00 00 00 00 00 00 43 00 00 00 74 A3 03 61 ........C...t..a 0034FAF0: D4 28 04 61 DC FB 34 00 58 FB 34 00 C0 9D 00 61 .(.a..4.X.4....a 0034FB00: 44 FB 34 00 00 00 00 00 48 FB 34 00 4D 4F 00 00 D.4.....H.4.MO.. 0034FB10: 00 B8 72 60 54 FB 34 00 00 B8 72 60 44 FB 34 00 ..r`T.4...r`D.4. 0034FB20: 00 00 00 00 2C 03 00 00 58 FB 34 00 F0 0B 34 7C ....,...X.4...4| 0034FB30: 2C 03 00 00 FF FF 00 00 58 FB 34 00 C9 29 70 60 ,.......X.4..)p` 0034FB40: 00 00 00 00 B8 8F 1A 00 01 00 00 00 28 3C 72 60 ............(

1 0

WM_*UISTATE message implementation in wine
by Vijay Kiran Kamuju Sept. 14, 2006

Sept. 14, 2006

Hi, Has wine an implementation for WM_*UISTATE messages? Can anybody tell me where should it be implemented and how it should be implemented. Does it involve complex coding? Thanks, VJ

1 1

Re: LOSTWAGES: Update the bus information for WineConf
by Eric Pouech Sept. 14, 2006

Sept. 14, 2006

Hi Rob, any news about how to get the keys for the room at Reading U ? haven't seen nothing about it ? TIA 2006/9/14, Robert Shearman <rob(a)codeweavers.com>: > > ChangeLog: > Update the bus information for WineConf. > > -- > Rob Shearman > > > > Index: templates/en/wineconf/travel.template > =================================================================== > RCS file: /home/wine/lostwages/templates/en/wineconf/travel.template,v > retrieving revision 1.18 > diff -u -p -r1.18 travel.template > --- templates/en/wineconf/travel.template 2 Aug 2006 19:03:42 > -0000 1.18 > +++ templates/en/wineconf/travel.template 14 Sep 2006 13:43:45 -0000 > @@ -48,10 +48,11 @@ depending on your quality and travelling > <li><a href="http://www.nationalexpress.com/">National > Express</a></li> > <li><a href="http://www.reading-buses.co.uk/">Reading Buses</a>: > <ul> > - <li>You will want to take the number 12 bus to get to the > + <li>You will want to take the number 18 or 19 bus to get to > the > university from Reading station. Click here for the > - <a href=" > http://www.reading-buses.co.uk/tt_reading_july2005/012MF20.HTML > ">timetable</a>. > - You can ask the bus driver to let you know where to get > off.</li> > + <a href=" > http://www.reading-buses.co.uk/tt_reading_march2006/018MF31.HTML > ">timetable</a>. > + You should get off at the St Joseph's Convent bus stop, as > this > + is the nearest to Windsor Hall.</li> > </ul> > </ul> > <li>Taxi — around £5 - £10 from Reading Station to > the > > > > > > -- Eric Pouech

1 0

Re: *RESEND* wininet: Make CheckInternetConnection behave like Windows does
by Hans Leidekker Sept. 14, 2006

Sept. 14, 2006

On Thursday 14 September 2006 14:07, Daniel Skorka wrote: > The function CheckInternetConnection in dlls/wininet/internet.c is too > naive ATM; it just sends a ping to the specified host. Like Mike McCormack suggested in the bug report, UNIX connect() does not deal with a proxy. It would be nice if you could confirm that CheckInternetConnection on Windows ignores a configured proxy, especially since you seem to have access to such a configuration. -Hans

2 1

Re: usp10: Match output of WinXP for ScriptGetProperties.
by Jeff Latimer Sept. 14, 2006

Sept. 14, 2006

Hans Leidekker wrote: > -Hans > >Changelog > Match output of WinXP for ScriptGetProperties. > Add a test for ScriptGetProperties. > > >------------------------------------------------------------------------ > >+static const SCRIPT_PROPERTIES script_0 = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 }; >+static const SCRIPT_PROPERTIES script_1 = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 }; >+static const SCRIPT_PROPERTIES script_2 = { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 }; > > >------------------------------------------------------------------------ > > > > Hans, what is the status of this patch? Looks pretty comprehensive but is unapplied. Are you going pursue it? Jeff

2 1

Re: configure.ac: Do not pass "-fPIC" to mingw for "make crosstest"
by Detlef Riekenberg Sept. 14, 2006

Sept. 14, 2006

On Mi, 2006-09-13 at 15:23 +0200, Detlef Riekenberg wrote: > When we do a "make crosstest" with mingw32msvc-gcc, > the compiler give a warning about "-fPIC" > > > Changelog: > - configure.ac: Do not pass "-fPIC" to mingw for "make crosstest" > > > Changes for configure are not included. > I crosscompiled the tests, dlls and programs and it still works. > Please ignoge this patch. There are more gcc-only flags, that we use without checking -- By by ... Detlef

1 0

MSI : GetInstallDriver, Can not find InstallDriver in ROT table, Return Code = 0x800401e3
by wino＠piments.com Sept. 14, 2006

Sept. 14, 2006

Hi, I have a refractory (faulty ?) MSI to deal with . I ran an msi based installer for autosketch today and it all went very well. Thanks for the current progress on MSI. Greatly cheered by this news I returned to one that I have never got to work since trying with 0.9.8 . I always thought it was wine that was not up to speed but it seems there some less than sorted versions of installsheild which give problems on some machines. In fact it may well be ALL Installsheild from v6 to v9. Only the lastest is fixed. I get the following the log file from msi: The InstallScript engine version currently installed on this machine is adequate. Event 'E2716761' is created GetInstallDriver, Can not find InstallDriver in ROT table, Return Code = 0x800401e3 {DDDD0C4B-57F7-4A85-ACF0-DB3FC8F1DBB4} Extract supporting files Failed to extract _IsUser.dll, Ignore it. Failed to extract IGdi.dll, Ignore it. ISMsiServerStartup Failure, Failed to RegisterMsiServer, Error = 0x8000ffff MsiServerStartup failed. Abort installation. Shutting down the PRC server... RPC runtime reported exception 0x6ba Googling parts of that soon shows a known issue , even on windows: http://forums1.itrc.hp.com/service/forums/questionanswer.do?threadId=993353… http://support.installshield.com/kb/view.asp?articleid=Q107249 http://support.installshield.com/kb/view.asp?articleid=Q107182 and specifically : http://www.voicerecognition.com/forums/ubb/Forum1/HTML/002282-2.html symtomatically , when it gets to the stage of copying the files to $TEMP , it seems to create a few zero length files only before failing. So the bug is in InstallSheild but it does not affect all machines. This version _usually_ installs correctly , what remains to be found out is why Wine falls into the broken category and how to work around it. Since I see this is was one area marked in 0.9.21 changelog I thought this may be a good time to bring it to your attention. best regards.

1 0

Re: cabinet: Add breaks to switch statements
by Dan Kegel Sept. 13, 2006

Sept. 13, 2006

On 9/13/06, Andrew Talbot <Andrew.Talbot(a)talbotville.com> wrote: > On Wednesday 13 September 2006 04:22, you wrote: > > Does adding the breaks fix conformance tests? > > No, I haven't fixed anything with this patch: it was a neutral one, in that > respect. I just put it in because I perceived that those switch statements > were not relying on any fallthrough, so I thought it good practice to isolate > the cases. Let's see: your change was --- a/dlls/cabinet/fci.c 2006-09-12 11:55:48.000000000 +0100 +++ b/dlls/cabinet/fci.c 2006-09-12 23:29:02.000000000 +0100 @@ -454,11 +454,12 @@ switch (cb % 4) { case 3: ul |= (((ULONG)(*pb++)) << 16); + break; case 2: ul |= (((ULONG)(*pb++)) << 8); + break; case 1: ul |= *pb++; - default: break; } Offhand that looks like a more serious change than you thought. You'd better either really understand the code, or add a conformance test that shows this fixes a bug, IMHO. - Dan

3 2

Low-level coding
by Aaron Slunt Sept. 13, 2006

Sept. 13, 2006

Hi, I was interested in starting to play with wine and maybe even build a few functions for it...but then I realized that it's low-level coding, something I'm not familiar with (higher level is more of my expertise). Correct me if I'm wrong, I could be looking at the wrong files :S. Does anyone have any good places to go to to learn about low-level coding, and afterwards coding for wine specifically? Thanks. -- Tired of being hammered by spyware, or want to browse the internet in a whole new way? Download Firefox and enjoy a much better internet experience. http://getfirefox.com/

7 8

Re: cabinet: Add breaks to switch statements
by H. Verbeet Sept. 13, 2006

Sept. 13, 2006

On 13/09/06, Andrew Talbot <Andrew.Talbot(a)talbotville.com> wrote: > Changelog: > cabinet: Add breaks to switch statements. > > diff -urN a/dlls/cabinet/fci.c b/dlls/cabinet/fci.c > --- a/dlls/cabinet/fci.c 2006-09-12 11:55:48.000000000 +0100 > +++ b/dlls/cabinet/fci.c 2006-09-12 23:29:02.000000000 +0100 > @@ -454,11 +454,12 @@ > switch (cb % 4) { > case 3: > ul |= (((ULONG)(*pb++)) << 16); > + break; > case 2: > ul |= (((ULONG)(*pb++)) << 8); > + break; > case 1: > ul |= *pb++; > - default: > break; > } > csum ^= ul; > diff -urN a/dlls/cabinet/fdi.c b/dlls/cabinet/fdi.c > --- a/dlls/cabinet/fdi.c 2006-05-23 13:47:35.000000000 +0100 > +++ b/dlls/cabinet/fdi.c 2006-09-12 23:29:13.000000000 +0100 > @@ -302,9 +302,15 @@ > } > > switch (bytes & 3) { > - case 3: ul |= *data++ << 16; > - case 2: ul |= *data++ << 8; > - case 1: ul |= *data; > + case 3: > + ul |= *data++ << 16; > + break; > + case 2: > + ul |= *data++ << 8; > + break; > + case 1: > + ul |= *data; > + break; > } > csum ^= ul; > I think that's supposed to fall through.

1 0

RTL language support in usp.dll
by Vijay Kiran Kamuju Sept. 13, 2006

Sept. 13, 2006

Hi, Your patch for RTL support in usp10.dll. http://www.winehq.org/pipermail/wine-patches/2006-September/030571.html Only includes support for Arabic. If you could include for other RTL languages esp Hebrew (other RTL languages are Farsi and Urdu). I think that would be an easy task for you as you know the usp internals. ;) bye, VJ

2 1

Find out which handle corresponds to which window
by David Gümbel Sept. 13, 2006

Sept. 13, 2006

Hi, - I'm almost sure that question has been asked here before, but I can't seem to find the corresponding post: I am trying to debug an application (Net Support Manager) that has trouble displaying a window in fullscreen mode. NSM enables you (among other things) to remotely look onto somebody else's Desktop, a bit like VNC or NX. It's quite annoying not to be able to look onto that remote desktop in fullscreen mode. Anyway, my guess is that it tries to put itself on top of everything in a wierd way that does not work properly under wine. In order to verify or falsify my guess, it would be extremely helpful to find out which handle corresponds to the window showing the symtoms. I've tried my luck with Process Explorer, but that tool doesn't show me the information I need. Any help is greatly appreciated. Cheers, David

2 4

Re: resend: tools[1/2]: wineprefixcreate: create a freedesktop-enabled menu directory
by Alexandre Julliard Sept. 13, 2006

Sept. 13, 2006

Mikołaj Zalewski <mikolaj(a)zalewski.pl> writes: > When creating the wineprefix, create a directory for the menu and a > configuration file that enables this menu in the freedesktop.org menu > structure. As suggested on wine-devel the menu directory is a > subdirectory of ~/.wine, so if the user deletes it, all menu items > created there will be automatically removed. > > Is there something wrong with this patch? That sort of desktop-specific code does not belong in wineprefixcreate. I'm not really convinced that this approach to menus is correct, but even if it is, it should be handled by wineshelllink and friends. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: riched20: move some selection code from message handler to ME_SetSelection
by Alexandre Julliard Sept. 13, 2006

Sept. 13, 2006

"Matt Finnicum" <mattfinn(a)gmail.com> writes: > The special cases handled by EM_EXSETSEL should also be handled by > EM_SETSEL, so this patch moves that code into the common > ME_SetSelection function. Before this patch, sending EM_SETSEL with > any value less than -1 would crash richedit. > > Changelog: move some selection code from message handlers to ME_SetSelection This fails make test: ../../../tools/runtest -q -P wine -M riched20.dll -T ../../.. -p riched20_test.exe.so editor.c && touch editor.ok editor.c:1335: Test failed: EM_EXSETSEL: expected: 18 actual: 17 make: *** [editor.ok] Error 1 -- Alexandre Julliard julliard(a)winehq.org

1 0

[PATCH] Also show 24bit modes as supported, but still handle them like 32bit later
by Christoph Frick Sept. 13, 2006

Sept. 13, 2006

hi would someone please comment on that patch? this is a nearly unchanged version of my previous attempt - but this time against current git. --- dlls/winex11.drv/settings.c | 9 ++++----- 1 files changed, 4 insertions(+), 5 deletions(-) diff --git a/dlls/winex11.drv/settings.c b/dlls/winex11.drv/settings.c index 5de07eb..c9bced2 100644 --- a/dlls/winex11.drv/settings.c +++ b/dlls/winex11.drv/settings.c @@ -39,7 +39,8 @@ static LPDDHALMODEINFO dd_modes = NULL; static unsigned int dd_mode_count = 0; static unsigned int dd_max_modes = 0; static int dd_mode_default = 0; -static const unsigned int depths[] = {8, 16, 32}; +static const unsigned int depths[] = {8, 16, 32, 24}; +static const unsigned int nr_depths = sizeof(depths) / sizeof(depths[0]); /* pointers to functions that actually do the hard stuff */ static int (*pGetCurrentMode)(void); @@ -62,7 +63,7 @@ LPDDHALMODEINFO X11DRV_Settings_SetHandl TRACE("Resolution settings now handled by: %s\n", name); if (reserve_depths) /* leave room for other depths */ - dd_max_modes = (3+1)*(nmodes); + dd_max_modes = nr_depths*nmodes; else dd_max_modes = nmodes; @@ -111,7 +112,7 @@ void X11DRV_Settings_AddDepthModes(void) int existing_modes = dd_mode_count; DWORD dwBpp = screen_depth; if (dwBpp == 24) dwBpp = 32; - for (j=0; j<3; j++) + for (j=0; j<nr_depths; j++) { if (depths[j] != dwBpp) { @@ -164,8 +165,6 @@ void X11DRV_Settings_Init(void) */ BOOL X11DRV_EnumDisplaySettingsEx( LPCWSTR name, DWORD n, LPDEVMODEW devmode, DWORD flags) { - DWORD dwBpp = screen_depth; - if (dwBpp == 24) dwBpp = 32; devmode->dmDisplayFlags = 0; devmode->dmDisplayFrequency = 0; devmode->dmSize = sizeof(DEVMODEW); -- 1.4.2

1 0

question about storage32.c code and while loop
by EA Durbin Sept. 13, 2006

Sept. 13, 2006

I've mentioned this before but I'm no expert C programmer. Can someone explain why we do the following in storage32.c? if (cbRead > 0) { cbTotalRead += cbRead; resWrite = BlockChainStream_WriteAt(bbTempChain, offset, cbRead, buffer, &cbWritten); if (FAILED(resWrite)) break; cbTotalWritten += cbWritten; offset.u.LowPart += This->smallBlockSize; } } while (cbRead > 0); HeapFree(GetProcessHeap(),0,buffer); if (FAILED(resRead) || FAILED(resWrite)) { ERR("conversion failed: resRead = 0x%08lx, resWrite = 0x%08lx\n", resRead, resWrite); BlockChainStream_Destroy(bbTempChain); return NULL; } in the while loop above while (cbRead > 0); it appears it terminates without doing anything due to the semicolon, is this intentional or is this semicolon not supposed to be there and its supposed to execute the line below it? If it is intentional why is the empty while loop here?

1 1

wine complation fails with latest git
by EA Durbin Sept. 13, 2006

Sept. 13, 2006

I'm not able to compile wine with the latest source from git. gcc-3.4 -c -I. -I. -I../../include -I../../include -Wall -pipe -fno-strict-aliasing -gstabs+ -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -g -O2 -o parser.yy.o parser.yy.c parser.l: In function `parser_lex': parser.l:129: error: `parser_lval' undeclared (first use in this function) parser.l:129: error: (Each undeclared identifier is reported only once parser.l:129: error: for each function it appears in.) parser.l: In function `kw_token': parser.l:342: error: `parser_lval' undeclared (first use in this function) make[2]: *** [parser.yy.o] Error 1 make[2]: Leaving directory `/home/eric/wine/tools/widl' make[1]: *** [widl] Error 2 make[1]: Leaving directory `/home/eric/wine/tools' make: *** [tools] Error 2

2 2

Help with git
by Kapila De Silva Sept. 13, 2006

Sept. 13, 2006

Hi all, Hope this is the right place to post this, if not, my apologies. Im trying to track down an issue that occurred between 0.9.19 and 0.9.20, and am using git bisect to track the issue. In the process of trying to identify the cause of the issue, I would like to be able to get the code up till a certain patch, and then apply patches one by one as well. I have not been able to find out how to do this, despite several web searches and man pages. Can anyone give me any pointers to this please? Thanks in advance Kapila

3 2

State management in D3D
by Ivan Gyurdiev Sept. 12, 2006

Sept. 12, 2006

State management in D3D... is currently kind of a mess. Stefan D. posted a message about this some time ago, but I can't find it right now... - some states are applied immediately (via Set* calls) - some are applied at draw time (like shaders, textures, transforms, ...) - some are recorded into a stateblock structure, and applied when the app says apply(). - the GL code is all over the place, tightly coupled to the d3d code. =============================== There's a number of projects going on at the moment, from what I understand: - Roderick is working on making wined3d -> wgl - Stefan D. has expressed concern about multithreaded d3d (where the D3D device state is shared, but not necessarily the GL one - or can GL contexts be shared between multiple threads?) - I'm trying to get FBOs to work, where the FBO needs to be bound once both the render and stencil target have been assigned (that's 2 calls from the app in any order) - i.e I need those states to be applied in a delayed fashion, before draw - Henri Verbeet wants multiple render target support added to that (meaning the FBO needs to be bound once *all* render targets are assigned in addition to the depth/stencil one). ================================ All of the above have a common theme - better state management is needed, with more encapsulation, and better separation between D3D and GL. So, let's come up with a plan, and try to implement it. How about we redesign the stateblock object like this: - remove deltas. I.E. if a SetLightEnable() command is sent, fetch the light, enable it, then save the light back - store only state, and no deltas. This should make the recording stateblock (updateStateBlock) the same as the initial device stateblock, which stores states - provide a uniform internal interface for accessing states inside the stateblock - like: SetState(stateblock, ID_XYZ, (void*) state_data); GetState(stateblock, ID_XYZ, (void**) state_data); CaptureState(stateblock, ID_XYZ); ApplyState(stateblock, ID_XYZ); ID_XYZ could be an individual state, or a "trigger" keyword, which will refer to a whole group of states. Those would be private functions in addition to the standard interface. - provide a fn pointer table inside the stateblock [ which can be directed to OGL or WGL or AGL ], which, for each ID_XYZ, maps a get and set function using the same interface - move all device.c GL code into those functions - now all device.c Get* and Set* requests will do is: - error checking - recording into updateStateBlock - or writing to stateBlock (which may be applied later, at our discretion). - apply() would just loop through all the IDs and call the corresponding function pointer if the states are marked dirty (we'll keep the 'dirty' field) - capture() would do the same in the get* direction. - new object can be instantiated per device, or per device per thread to address multithreading. It could have an associated glContext, and can be locked as necessary. - it would use ideas from the d3d9 test framework for stateblock, except of course more competently written, and cleaner :)

3 12

Help with shader code
by Kapila De Silva Sept. 12, 2006

Sept. 12, 2006

Hi All, can someone help me understand some code from dlls/wined3d/vertexshader.c please? Im new to shader code, but to me the following looks a bit strange: shader_addline(&buffer, "ADD result.position.x, TMP_OUT.x, PROJECTIONX.w;\n"); shader_addline(&buffer, "ADD result.position.y, TMP_OUT.y, PROJECTIONY.w;\n"); /* Account for any inverted textures (render to texture case) by reversing the y coordinate * (this is handled in drawPrim() when it sets the MODELVIEW and PROJECTION matrices) */ shader_addline(&buffer, "MUL result.position.y, TMP_OUT.y, PROJECTIONY.y;\n"); Am I correct that the last line makes the second line redundant? Wont result.position.y be equal to TMP_OUT.y * PROJECTIONY.y no matter what? Thanks in advance. Kapila.

2 2

Re: comctl32: Remove dead code in tooltips.c (coverity)
by Alexandre Julliard Sept. 12, 2006

Sept. 12, 2006

Detlef Riekenberg <wine.dev(a)web.de> writes: > We return with FALSE at the top of the Function, > when lpToolInfo is NULL. I think that's where the real bug is, we shouldn't always return FALSE in that case. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: X11Drv: move WGL extension code
by Lionel Ulmer Sept. 12, 2006

Sept. 12, 2006

On Fri, Sep 01, 2006 at 01:57:24PM +0200, Roderick Colenbrander wrote: > The way WGL extensions are loaded is different. The old code correctly > advertised the names of available WGL extensions but the wglGetProcAddress > code didn't work correctly. Even if an extension wasn't advertised you could > get a function pointer to the functions. On Windows wglGetProcAddress would > return NULL. Note that (as far as I know), 'wglGetProcAddress' returns NULL on non-existant extensions for all extensions, whether they are GL or WGL ones. This means that our implementation of it is broken (at least on DRI / Mesa) as 'glXGetProcAddress' may return a non-NULL value when querying a non-supported extension. So if we really want to be windows-like, we would need to add a 'function name / extension name' database and check if the extension corresponding to the function is present before actually calling 'glXGetProcAddress'. But as I ever only once got a bug report linked to that, I never took the pain to fix it :-) Lionel (catching up his old mails) PS: I guess that this 'database' may be automatically build by the 'make_opengl' script though so it should not be that hard to do :-) -- Lionel Ulmer - http://www.bbrox.org/

1 0

Storage32 error
by EA Durbin Sept. 12, 2006

Sept. 12, 2006

I've been experiencing an error in several installers and it just appeared lately in installers that used to succeed, such as the battlefield 2 demo installer, this error is causing them to fail. Was this caused by a recent patch? err:storage:Storage32Impl_SmallBlocksToBigBlocks conversion failed: resRead = 0x8003001e, resWrite = 0x00000000 err:storage:Storage32Impl_SmallBlocksToBigBlocks conversion failed: resRead = 0x8003001e, resWrite = 0x00000000 err:storage:Storage32Impl_SmallBlocksToBigBlocks conversion failed: resRead = 0x8003001e, resWrite = 0x00000000

1 0

Re: include: Add new Vista SDK constants, fields and, structures documented at http://ian.joeuser.com/index.asp?AID=738
by Timo Kunze Sept. 11, 2006

Sept. 11, 2006

Alexandre Julliard <julliard(a)winehq.org> writes: > Francois Gouget <fgouget(a)free.fr> writes: >> > @@ -1707,6 +1732,7 @@ typedef struct tagTOOLINFOA { >> > LPSTR lpszText; >> > LPARAM lParam; >> > void *lpReserved; >> > + HBITMAP hbmp; >> > } TTTOOLINFOA, *LPTOOLINFOA, *PTOOLINFOA, *LPTTTOOLINFOA; > > I don't think our code is going to handle this properly. Before > extending the structures we need to fix places that depend on the size > of the old type. This website is absolutely outdated. Take a look at the current (build 5536) Windows SDK headers and you'll see, that e. g. the hbmp member of the TTTOOLINFO struct has been removed again. Regards Timo

1 0

Wineconf this weekend!
by Jeremy White Sept. 11, 2006

Sept. 11, 2006

Hi Folks, So this weekend is Wineconf, which is the mostly annual gathering of Wine developers and afficianodos. Everyone is welcome, although we do intend to run a very technical and developer focused agenda. It will be held in Reading, a bit west of London, in the UK. The formal program starts on Saturday. We have a tentative agenda posted on the Wiki here: http://wiki.winehq.org/WineConf2006 The one don't miss talk will be Alexandre's keynote, which will be at 9:00 am on Saturday. We're still working out the schedule for the other events. There is no charge, although you do have to take care of your own loding and travel. If you are planning to come, though, it would be handy if you could hit the RSVP page (if you haven't already): http://www.winehq.org/site/wineconf/rsvp That way we can make sure our catering counts aren't out of wack. If you are coming, particularly if you reserved a room on campus with me, you *must* sign up for this mailing list: http://www.winehq.org/mailman/listinfo/wineconf or you will miss out on important details, and you'll end up wandering around Reading desperately looking for your key. (And yes, that does mean I'll stop spamming this list, and start spamming that one :-/). Cheers, Jeremy

1 0

Re: dinput: [2/2] Create/destroy hook thread from DirectInput [try 3].
by Alexandre Julliard Sept. 11, 2006

Sept. 11, 2006

Vitaliy Margolen <wine-patch(a)kievinfo.com> writes: > +static void release_hook_thread(void) > +{ > + LONG ref; > + > + EnterCriticalSection(&dinput_hook_crit); > + ref = --hook_thread_refcount; > + TRACE("Releasing to %ld\n", ref); > + if (ref == 0) SendMessageW(hook_thread_hwnd, WM_DESTROY, 0, 0); WM_DESTROY is supposed to be sent by DestroyWindow, sending it manually is not very nice, you should use a WM_USER message instead. Also you need to clear hook_thread_hwnd first, otherwise there's a race since you don't wait for the thread to finish. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: Bug #4510: allowing -ansi for arpa/nameser.h in configure.ac
by Alexandre Julliard Sept. 11, 2006

Sept. 11, 2006

Joel Parker <jjk3(a)msstate.edu> writes: > This patch fixes bug #4510. It allows the use of -ansi when > running ./configure. I'm not sure I see the point, you'd need to fix all the places that use the file too, otherwise it won't compile, and it's not clear that defining BSD_SOURCE all over the place is a good idea. I'd suggest to fix the header instead. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: WINEDOCS: Update regression testing procedure to use Git
by Saulius Krasuckas Sept. 11, 2006

Sept. 11, 2006

* On Mon, 28 Aug 2006, Mike McCormack wrote: > Update regression testing procedure to use Git Any chances this patch gets into the webserver? :) http://marc.theaimsgroup.com/?l=wine-patches&m=115673906004873&w=2

2 1

Re: X11Drv: move WGL extension code
by Alexandre Julliard Sept. 11, 2006

Sept. 11, 2006

"Roderick Colenbrander" <thunderbird2k(a)gmx.net> writes: > The patch adds new WGL extension loading code based on code written > by an old opengl32 -> x11drv patch written by Tomas Carnacky. The > code properly advertises WGL extensions and only returns function > pointers when the extension is really available. It would be nicer to declare the extensions as constant variables instead of building everything at run time. Something like: static const struct wglextension swap_control = { "WGL_EXT_swap_control", { { "wglSwapIntervalEXT", X11DRV_wglSwapIntervalEXT }, { "wglGetSwapIntervalEXT", X11DRV_wglGetSwapIntervalEXT }, } }; static void load_extensions(void) { if (glxRequireExtension("GLX_SGI_swap_control")) register_extension( &swap_control ); ... etc. } Then the extension list can simply store a pointer instead of copying the whole thing once more, and you avoid a lot of nasty macros. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: include: Add new Vista SDK constants, fields and structures documented at http://ian.joeuser.com/index.asp?AID=738
by Alexandre Julliard Sept. 11, 2006

Sept. 11, 2006

Francois Gouget <fgouget(a)free.fr> writes: > @@ -1707,6 +1732,7 @@ typedef struct tagTOOLINFOA { > LPSTR lpszText; > LPARAM lParam; > void *lpReserved; > + HBITMAP hbmp; > } TTTOOLINFOA, *LPTOOLINFOA, *PTOOLINFOA, *LPTTTOOLINFOA; I don't think our code is going to handle this properly. Before extending the structures we need to fix places that depend on the size of the old type. -- Alexandre Julliard julliard(a)winehq.org

1 0

Making people aware about appdb and outdated Wine versions.
by killertux killertux Sept. 10, 2006

Sept. 10, 2006

Hello, I have problem... It's know as dublication of effort. Basicly there are sites that "compeate" with appdb it means HOWTOS, test result and everyting else are distributed in few thousands forums, wikis etc... Solution is that Wine should advertice Wine AppDB so people would actully use it rather than unoffical sites. So I would like to see some AppDB "adverticement" in winecfg. Winecfg about section is one good place to inform wonders of appdb but even better would be popup window when starting winecfg(at lest in first run) that informs about appdb. Normal users obtain copy of Wine trought their package manager and that means they do not know about appdb unless they they read it somewhere. This change must be done in order to kill useless stuff like this: http://forums.gentoo.org/viewtopic-t-496836.html Second: Old version of Wine are still quite used because of package managers just offers "stable" Wine versions over the "unstable" ones so I would like to see another popup or warning in winecfg that informs user that their Wine is getting old... so after specific number of days(from release of Wine version X) counter gives some warnings about old Wine version.

3 2

Re: Jonathan Ernst : winecfg: French translation update.
by Frank Richter Sept. 10, 2006

Sept. 10, 2006

On 10.09.2006 10:28, Alexandre Julliard wrote: > - LTEXT "This library is free software; you can redistribute it and/or modify it under the terms of the GNU Lesser General Public License as published by the Free Software Foundation; either version 2.1 of the License, or (at your option) any later version.", > + LTEXT "Cette biblioth�que est un logiciel libre�; vous pouvez la redistribuer et/ou la modifier conform�ment aux dispositions de la Licence Publique G�n�rale GNU, telle que publi�e par la Free Software Foundation�; version 2.1 de la licence, ou encore (� votre choix) toute version ult�rieure." > IDC_STATIC,119,44,124,72 It seems that all (L)GPL translations are marked "unofficial" by the FSF. That's why I though it wouldn't be bad to put in a translated "This library is ..." but keep the original English notice as well for the German translation. -f.r.

2 1

Re: Re: Making people aware about appdb and outdated Wine versions.
by killertux killertux Sept. 10, 2006

Sept. 10, 2006

OK, if not popup then fixme or someting like if there is problems they can go to Wine AppDB to get some compatibility help. >> --SNIP-- >> Second: Old version of Wine are still quite used because of package >> managers just offers "stable" Wine versions over the "unstable" ones so I would like >> to see another popup or warning in winecfg that informs user that their Wine is >> getting old... so after specific number of days(from release of Wine version X) >> counter gives some warnings about old Wine version. > >Having a popup (or anything that requires user interaction) could get >really irritating - there are situations where people would use an >older version of wine on purpose. Like if a large organization uses it >to run one specific app, and has a specific version that works >perfectly, they might not have any reason to update. > >How about a fixme when wine is run, informing them that the version >they are running was released X long ago, and that if they have any >issues they should try a newer version? > >It'd be nice, too, if people then reported bugs with that on the top >of the output - it could save a lot of work chasing down already- fixed >issues. > >--Matt >

3 2