July 2003 - Wine-Devel - list.winehq.org

Wine Status DLL's Final :)

by Tom

Okay here is the final version that I have.... Any comments before I send it to Wine-Patches ? Changlelog from last version 1) Add more WWN links 2) Add Robert North as worker to Wintab32 3) change Wintab32 status from 80% to 25% 4) Add iphlpapi.dll, Juan as worker & 50% Tom

July 2, 2003

WCMD not working anymore

by Sylvain Petreolle

Trying to run builtin commands with WCMD like dir or exit returns 'File not found'. Doing some debug, it seems there is a problem with kernel.CompareStringA. In wcmdmain.c, we have this call : for (i=0; i<=WCMD_EXIT; i++) { if (CompareString (LOCALE_USER_DEFAULT, NORM_IGNORECASE | SORT_STRINGSORT, whichcmd, count, inbuilt[i], -1) == 2) break; With the command "DIR" entered we have : ... 0009:Call kernel32.CompareStringA(00000400,00001001,40792654 "DIR",00000003,4057bcd3 "DIR",ffffffff) ret=405769b9 0009:Ret kernel32.CompareStringA() retval=00000001 ret=405769b9 .... where the return code should be 2 (CSTR_EQUAL) ===== Sylvain Petreolle (spetreolle_at_users_dot_sourceforge_dot_net) ICQ #170597259 alias upsf='false ; while [ $? -ne 0 ] ; do cvs update -APd ; done 2>&1 |tee cvslog' "What if tomorrow the War could be over ?" Morpheus, in "Reloaded". ___________________________________________________________ Do You Yahoo!? -- Une adresse @yahoo.fr gratuite et en français ! Yahoo! Mail : http://fr.mail.yahoo.com

July 2, 2003

Re: [RESENT]Don't add empty lines when writing out profile files

by Uwe Bonnes

>>>>> "Alexandre" == Alexandre Julliard <julliard(a)winehq.org> writes: Alexandre> Uwe Bonnes <bon(a)elektron.ikp.physik.tu-darmstadt.de> writes: >> Changelog: wine/files/profile.c: PROFILE_Save Don't add lines for >> empty sections Alexandre> This will cause us to delete keys that are before the first Alexandre> section header, I don't think that's correct. Why do you need Alexandre> that? I was annoyed by the added empty lines and I thought I understood the problem. Obvious I did not fully. Thanks for the feedback. -- Uwe Bonnes bon(a)elektron.ikp.physik.tu-darmstadt.de Institut fuer Kernphysik Schlossgartenstrasse 9 64289 Darmstadt --------- Tel. 06151 162516 -------- Fax. 06151 164321 ----------

July 2, 2003

Compiler warnings

by Marcelo Duarte

gcc -c -I. -I. -I../../include -I../../include -D_REENTRANT -fPIC -D__WINESRC__ -D_OLEAUT32_ -DCOM_NO_WINDOWS_H -Wall -mpreferred-stack-boundary=2 -gstabs+ -Wpointer-arith -g -O2 -o variant.o variant.c variant.c: In function `VarBstrFromDate': variant.c:3576: warning: `%x' yields only last 2 digits of year in some locales variant.c:3580: warning: `%x' yields only last 2 digits of year in some locales make[2]: Entering directory `/home/mduarte/wine/dlls/crypt32' gcc -c -I. -I. -I../../include -I../../include -D_REENTRANT -fPIC -D__WINESRC__ -D_CRYPT32_ -Wall -mpreferred-stack-boundary=2 -gstabs+ -Wpointer-arith -g -O2 -o main.o main.c In file included from ../../include/mssip.h:49, from main.c:22: ../../include/pshpack8.h:47:6: warning: #warning "8 as alignment is not supported" In file included from ../../include/mssip.h:80, from main.c:22: ../../include/pshpack8.h:47:6: warning: #warning "8 as alignment is not supported" In file included from ../../include/mssip.h:88, from main.c:22: ../../include/pshpack8.h:47:6: warning: #warning "8 as alignment is not supported" In file included from ../../include/mssip.h:97, from main.c:22: ../../include/pshpack8.h:47:6: warning: #warning "8 as alignment is not supported" In file included from ../../include/mssip.h:109, from main.c:22: ../../include/pshpack8.h:47:6: warning: #warning "8 as alignment is not supported" In file included from ../../include/mssip.h:123, from main.c:22: ../../include/pshpack8.h:47:6: warning: #warning "8 as alignment is not supported" In file included from ../../include/mssip.h:140, from main.c:22: ../../include/pshpack8.h:47:6: warning: #warning "8 as alignment is not supported" -- Marcelo Duarte

July 2, 2003

Re: still many warnings from Mesa

by Sylvain Petreolle

Hi Lionel, Im using Xfree86 standard headers (no NVidia/ATI/... headers) and Linux RH 9. I'm getting the same warning as Gerard's however. ========= Well, the Linux ABI tells this : " 4.4. glx.h must include the prototype of the glXGetProcAddressARB extension described above. 4.5. All OpenGL 1.2 and ARB_multitexture, GLU 1.3, and GLX 1.3 entry points and enumerants must be present in the corresponding header files gl.h, glu.h, and glx.h, even if only OpenGL 1.1 is implemented at runtime by the associated runtime libraries. " So if your system was compliant, you should have all the glMulti* prototypes defined in your gl.h file and glXGetProcAddressARB too.... So you should not have any warning. Nothing is mentionned in the ABI about any need for 'GL_GLEXT_PROTOTYPES' protection of these calls, so well, I still think that it's somehow a problem in your headers :-) ===== Sylvain Petreolle (spetreolle_at_users_dot_sourceforge_dot_net) ICQ #170597259 alias upsf='false ; while [ $? -ne 0 ] ; do cvs update -APd ; done 2>&1 |tee cvslog' "What if tomorrow the War could be over ?" Morpheus, in "Reloaded". ___________________________________________________________ Do You Yahoo!? -- Une adresse @yahoo.fr gratuite et en français ! Yahoo! Mail : http://fr.mail.yahoo.com

July 2, 2003

Re: Add support for the ~MHz registry key

by Alexandre Julliard

Mike Hearn <mike(a)theoretic.com> writes: > If anybody knows a handy call to find out the CPU speed (that doesn't > use QueryPerformanceFrequency), it would be nice to use it here. > Otherwise this fix will do. Actually QueryPerformanceFrequency should be merged with the rest of the CPU detection, and then you should be able to use it to set the right value in the registry. -- Alexandre Julliard julliard(a)winehq.com

July 2, 2003

Re: iphlpapi Solaris fixes

by Alexandre Julliard

Robert Lunnon <bob(a)yarrabee.net.au> writes: > @@ -787,7 +789,11 @@ DWORD getInterfaceMtuByName(const char * > if ((ioctl(fd, SIOCGIFMTU, &ifr))) > ret = ERROR_INVALID_DATA; > else { > +#if !defined sun > *mtu = ifr.ifr_mtu; > +#else > + *mtu=ifr.ifr_metric; > + #endif You should add a configure check for the structure field. -- Alexandre Julliard julliard(a)winehq.com

July 2, 2003

Re: [RESENT]Don't add empty lines when writing out profile files

by Alexandre Julliard

Uwe Bonnes <bon(a)elektron.ikp.physik.tu-darmstadt.de> writes: > Changelog: > wine/files/profile.c: PROFILE_Save > Don't add lines for empty sections This will cause us to delete keys that are before the first section header, I don't think that's correct. Why do you need that? -- Alexandre Julliard julliard(a)winehq.com

July 2, 2003

Re: Don't set stacking order against unvisible windows

by Alexandre Julliard

Mike Hearn <m.hearn(a)signal.qinetiq.com> writes: > However, I think it unbreaks InstallShield, which is a fantastically > common application. It might be a case of a "lesser of two evils" > decision. No it's a "correct vs. wrong" decision. The existing code is correct and there's no reason to break it, even if doing that happens to mask another bug. -- Alexandre Julliard julliard(a)winehq.com

July 2, 2003

RE: [RESENT] Always print version on startup

by PETREOLLE Sylvain

-----Message d'origine----- De : Francois Gouget [mailto:fgouget(a)free.fr] Envoyé : mercredi 2 juillet 2003 01:18 À : PETREOLLE Sylvain Cc : wine-devel(a)winehq.org Objet : RE: [RESENT] Always print version on startup Now we need to be sure of that... And since WineSetupTk is wine-specific, it should be able to handle this. Actually, MESSAGE probably prints to stderr already. But still, some programs will assume that an error occurred if something is printed on stderr. For instance this will happen to any application written in tcl unless the developper went to great lengths to avoid it. So such a change would likely break WineSetupTk. If stderr is used, they shouldnt have the message in any way. > I will add one more remark (!) : I dont know many programs that expect "Wine > version" in their output. Precisely. And they should not have to deal with it. This is absolutely equivalent to modifying grep so it systematically prints its version information on stdout or stderr. This would break tons of scripts out there.

July 1, 2003