August 2006 - Wine-Devel - list.winehq.org

Re: wintrust: Only return ERROR_SUCCESS in WinVerifyTrust
by Andreas Mohr Aug. 7, 2006

Aug. 7, 2006

Hi, On Tue, Aug 01, 2006 at 01:21:57PM -0700, James Hawkins wrote: > My reasoning for reverting the change is that I'd rather have 5 more > apps installing, than one app working (and it's Process Explorer of > all things). Indeed, but I'm not sure it's a good idea to kill all comment annotations in the process. I'd rather even add the fact that this return code change killed more installers than it made apps work. Andreas

4 4

Time to commit remote ops for VirtualAllocEx, CreateRemoteThread
by Dan Kegel Aug. 7, 2006

Aug. 7, 2006

Tommy's APC version of his VirtualAllocEx / CreateRemoteThread patch seems to be safe (since APCs only run a points where threads in well-written programs are not holding locks), is complete enough to make many apps happy, and is probably the best that can be done without a service thread or a real kernel change. I've encouraged him to submit it for real. Let's get this puppy committed!

7 14

Re: riched20: new selection invalidation logic
by Dan Kegel Aug. 7, 2006

Aug. 7, 2006

On 8/7/06, Krzysztof Foltman <wdev(a)foltman.com> wrote: > +#define static That deserves a comment, at the very least! I don't think such a define should be allowed. Can you recode without this? I like the number of asserts. It's hard to get this stuff right, and the asserts will help make sure the code is correct. - Dan

1 0

Re: Tijl Coosemans : ntdll: Add FreeBSD default serial port device.
by Marcus Meissner Aug. 7, 2006

Aug. 7, 2006

On Mon, Aug 07, 2006 at 07:13:04AM -0500, Alexandre Julliard wrote: > Module: wine > Branch: refs/heads/master > Commit: e548d4083177125559f752d902907187cea60474 > URL: http://source.winehq.org/git/?p=wine.git;a=commit;h=e548d4083177125559f752d… > > Author: Tijl Coosemans <tijl(a)ulyssis.org> > Date: Sat Aug 5 23:12:22 2006 +0200 > > ntdll: Add FreeBSD default serial port device. > > --- > > dlls/ntdll/directory.c | 7 +++++++ > 1 files changed, 7 insertions(+), 0 deletions(-) > > diff --git a/dlls/ntdll/directory.c b/dlls/ntdll/directory.c > index 47d018a..84624a8 100644 > --- a/dlls/ntdll/directory.c > +++ b/dlls/ntdll/directory.c > @@ -175,6 +175,13 @@ #ifdef linux > strcpy( ret, "/dev/ttyS0" ); > ret[strlen(ret) - 1] = '0' + num - 1; > } > +#elif defined(__FreeBSD__) || defined(__FreeBSD_kernel__) > + ret = RtlAllocateHeap( GetProcessHeap(), 0, sizeof("/dev/cuad0") ); Surely you mean strlen()+1. sizeof("hallo") might just be sizeof(char*) Ciao, Marcus

2 1

DirectX can't change Screen depth
by Colin Pitrat Aug. 7, 2006

Aug. 7, 2006

Hello, I'm maintaining a game named Blip & Blop (http://appdb.winehq.org/appview.php?iAppId=3470) and I have some problems with color depth. If the X server is configured in 16 bits colors, it works well, but if it's in 24 or 32 bits, the app display in black and white, and I've got the following message : fixme:x11drv:X11DRV_desktop_SetCurrentMode Cannot change screen BPP from 32 to 16 Looking at the source, it seems that changing resolution is not implemented, and that doesn't surprise me as it doesn't seem to be a good idea to change X depth to please a wine app (especially if we're emulating a desktop). But shouldn't the 16 bits output be converted by wine to 24/32 bits to be displayed. Is it a problem with the way the game is coded (maybe it should check return code to fall back to another depth) ? Regards, -- Colin Pitrat (Bull Services Telco) Bull, Architect of an Open World (TM) Tél : +33 (0) 1 30 80 72 93 www.bull.com

2 1

regression in current GIT
by Aaron O Aug. 6, 2006

Aug. 6, 2006

hi, if this is the wrong mailing list, then please correct me. there is a regression in current git revision in Xilero. patch which causes the regression: 3d9e96f3bc34029c93ceb3fd3f9674d2c69e414d is first bad commit commit 3d9e96f3bc34029c93ceb3fd3f9674d2c69e414d Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Sat Aug 5 00:11:17 2006 +0200 wined3d: Volume textures fix. :040000 040000 b07704f9002468b7944e8660eac1434dd5f1ae3f 234b926bd8bb1981d2a07a1f5dd3831f33ce267b M dlls :040000 040000 7203ce1c26132c5232f6c436dd501ff17a0fa47d ab3a496643793857b97c736f706ec3e16d572af3 M include error in program: fixme:d3d:IWineD3DDeviceImpl_GetAvailableTextureMem (0x16c218) : stub, emulating 64MB for now, returning 64MB fixme:ddraw:IDirectDrawImpl_SetCooperativeLevel (0x16bb58)->(0x10024,00000c13) fixme:ddraw:IDirectDrawImpl_SetCooperativeLevel (0x16bb58) Unhandled flag DDSCL_MULTITHREADED, Uh Oh... fixme:x11drv:X11DRV_desktop_SetCurrentMode Cannot change screen BPP from 32 to 16 fixme:d3d:IWineD3DDeviceImpl_SetRenderState >>>>>>>>>>>>>>>>> 500 from glFinalCombinerInputNV() @ device.c / 3846 err:ddraw:IDirectDrawSurfaceImpl_Release (0x1bbb28) Failed to uninit 3D fixme:ddraw:IDirectDrawImpl_SetCooperativeLevel (0x16bb58)->((nil),00000008) fixme:d3d:IWineD3DStateBlockImpl_Release Releasing primary stateblock fixme:d3d:IWineD3DDeviceImpl_Release (0x16c218) Device released with resources still bound, acceptable but unexpected fixme:d3d:dumpResources Leftover resource 0x1bbdd8 with type 1,WINED3DRTYPE_SURFACE fixme:d3d:dumpResources Leftover resource 0x1bbc08 with type 1,WINED3DRTYPE_SURFACE thanks. i can provide any other needed information.

3 4

Re: wine.inf: Windows 9x and Windows NT store the time zone data in different places.
by Vitaliy Margolen Aug. 6, 2006

Aug. 6, 2006

Friday, August 4, 2006, 9:47:38 AM, Francois Gouget wrote: > So duplicate it so programs find it no matter where they look. > Note: > Another option would be to have a [Timezones9x] section that would set > this data in the alternate location. But we would still need to > duplicate all the data in the inf file anyway. This would also make > checking the inf file consistency harder too. And finally, it would make > it impossible to change the Windows version of a Wine environment after > the fact. So I chose to just duplicate it all. You missed one more option - use registry symlinks. The only problem is - win2k and winXP doesn't have those registry keys in win9x place. Vitaliy Margolen

2 1

Re: wine-ShowCursor-D3D-over-GDI.patch
by Stefan Dösinger Aug. 6, 2006

Aug. 6, 2006

Am Samstag 29 Juli 2006 11:17 schrieb Andrew Riedi: > Well, here is my first patch, feel free to tell me what all is wrong with > it. I noticed a problem in WoW d3d, so I tracked it down and checked out > msdn. It seems that implementing ShowCursor() over GDI is acceptable, at > least according to msdn, and it seems to work great. Anyhow, this patch > fixes the two mouse cursor problem in WoW, and makes wined3d more > compatible with windows, at least msdn. Here goes nothing. MSDN says that the cursor image set with SetCursorProperties can be shown as a user cursor if it has a size of 32x32 pixels. Otherwise the cursor is emulated like we do it at already in swapchain.c, IDirect3DSwapChain9::Present. So if the cursor image is 32x32 pixels, upload it to a gdi cursor, and do not upload it to a gl texture(doine in SetCusorProperties with the preload call), and do not draw it in present. Activate the user cursor in ShowCursor. If the cursor is not 32x32 pixels then disable the gdi cursor, and draw the cursor as it is done now(upload to a gl texture, call Blt in present. I don't know how to set a gdi cursor, but you can find the image in the d3d surface at surface->resource.allocatedMemory. Be aware that allocatedMemory may be NULL if the image is just stored in a gl texture, so the cleaner way is to call IWineD3DSurface::LockRect to retrieve the image.

3 5

Problems (and fix) when compiling CVS on 64-bit Kubuntu (erroneous freetype-config message)
by Evil Jay Aug. 6, 2006

Aug. 6, 2006

When compiling Wine on 64-bit Kubuntu 6.06LTS (using the instructions found at http://wiki.winehq.org/WineOn64bit#head-56206e8bc74083807ffe06ccb471d3f964c…) I received a spurious error regarding freetype-config: configure: WARNING: Your system appears to have the FreeType 2 runtime libraries configure: WARNING: installed, but 'freetype-config' is not in your PATH. Install configure: WARNING: the freetype-devel package (or its equivalent on your distribution) configure: WARNING: to enable Wine to use TrueType fonts. configure: WARNING: FreeType is missing. configure: WARNING: Fonts will not be built. Dialog text may be invisible or unaligned. Everything seems to work fine after I compile it though. The fonts display perfectly. I also had to modify the .configure script. The unicode-related section directs it to use the libraries in /usr/lib (whereas I needed it to be using /usr/lib32). If you don't modify this section, make will fail when it tries to link the 64-bit versions. Other than these two minor annoyances, it compiles quite well on 64-bit. ------------------------------------------ evil(a)pluto:~/install/wine$ LDFLAGS="-L/lib32 -L/usr/lib32 -Wl,-rpath,/lib32 -Wl,-rpath,/usr/lib32" ./configure | grep "type\|font" ... checking for fontforge... fontforge ... checking fontconfig/fontconfig.h usability... yes checking fontconfig/fontconfig.h presence... yes checking for fontconfig/fontconfig.h... yes ... checking for freetype-config... freetype-config checking for FT_Init_FreeType in -lfreetype... no ... checking for -lfreetype soname... libfreetype.so.6 checking for -lfontconfig soname... libfontconfig.so ... config.status: creating fonts/Makefile configure: WARNING: Your system appears to have the FreeType 2 runtime libraries configure: WARNING: installed, but 'freetype-config' is not in your PATH. Install configure: WARNING: the freetype-devel package (or its equivalent on your distribution) configure: WARNING: to enable Wine to use TrueType fonts. configure: WARNING: FreeType is missing. configure: WARNING: Fonts will not be built. Dialog text may be invisible or unaligned. evil(a)pluto:~/install/wine$ locate freetype-config /usr/share/man/man1/freetype-config.1.gz /usr/bin/freetype-config

1 0

x11drv: Add Xcursor support, prefer it when available.
by Stefan Siebert Aug. 5, 2006

Aug. 5, 2006

Thanks for this great piece of work ! - Stefan

1 0