November 2008 - Wine-Devel - list.winehq.org

IMM / IME work
by Aric Stewart June 16, 2016

June 16, 2016

Hello all, I know this will only interest a small portion of you but thought i would give a quick update on the state of IMM32 since I have brought it to a major milestone. All the main patches are in which now separate IMM32 and IMEs. There is still more work to do but the major framework is in place. X11 XIM processing should be unchanged. However wine can now begin to load native windows IMEs as well. I have tested with windows ATOK20 (a popular Japanese IME) and successfully had text processing in a fully IME aware application. There are still clear issues to resolve in many aspects of this processing but we have forward progress. ImmInstallIME does not work yet, nor does switching keyboards. So to get the native IME to work you need to add this registry key. [System\\CurrentControlSet\\Control\\Keyboard Layouts\\<keyboard layout>] "Ime File"=<IME filename> so for example for ATOK20 in Japanese i used. [System\\CurrentControlSet\\Control\\Keyboard Layouts\\e0010411] "Ime File"="ATOK20W.IME" I would love to hear how well things work. I am sure using native IMEs will quickly show us many places where IMM32 needs to be improved. One issues I am going to investigate next is that sometimes non x11drv ime initialization, if occurring too early, causes x11drv to fail to create windows. I have not investigated with the latest changes to xim.c (which may already correct this problem) but if you see this problem this patch may help and i believe the IME_UpdateAssociation(NULL) is already unneeded. diff --git a/dlls/winex11.drv/xim.c b/dlls/winex11.drv/xim.c index d4df9f7..0c98136 100644 --- a/dlls/winex11.drv/xim.c +++ b/dlls/winex11.drv/xim.c @@ -475,7 +475,6 @@ static void X11DRV_OpenIM(Display *display, XPointer ptr, XP XUnregisterIMInstantiateCallback(display, NULL, NULL, NULL, X11DRV_OpenIM, wine_tsx11_unlock(); IME_XIMPresent(TRUE); - IME_UpdateAssociation(NULL); } thanks, -aric

5 7

What happened to the Fedora packages?
by Neal Gompa Aug. 2, 2014

Aug. 2, 2014

What happened to the Fedora packages? They have not been updated since 0.9.2!!!! Right now it is at 0.9.10!!! Nearly every other Linux distro supported has the up to date packages!!! And why does the Red Hat packages site not go to the SourceForge site as it does for SUSE packages and the others?? I have not really had the guts to ask until now, because I thought that maybe there was a slump, but now, its getting annoying!! And Fedora just released Fedora Core 5 yesterday!!! Please tell me new packages will be ready soon!!! Compiling WINE always crashes my computer, so I prefer to use the RPMs...

10 18

winspool: ERROR_INVALID_NAME in GetDefaultPrinter() ?
by Detlef Riekenberg Oct. 9, 2013

Oct. 9, 2013

Hi. >From which configuration does the "ERROR_INVALID_NAME" came from, when calling GetDefaultPrinter(NULL, &size) and no Printer is installed? This Test is Present in the current "dlls/winspool/tests/info.c". MSDN told us, that we receive an "ERROR_FILE_NOT_FOUND", if no Printer is installed: http://msdn.microsoft.com/library/en-us/gdi/prntspol_0hma.asp I get the "ERROR_FILE_NOT_FOUND" on win98se, winme, w2k and win2003 in this Situation. -- By By ... ... Detlef

3 3

re: Wine t-shirts?
by Dan Kegel Sept. 2, 2009

Sept. 2, 2009

Maarten wrote: > Why don't we have wine t-shirts any more? I dunno, where were they before? I just looked, couldn't find any on cafepress.com. I'd like to bring back the Codeweavers drunken penguin t-shirts, perhaps with a slightly more generic Wine design. Coincidentally, I found some site on the net had taken the logo from that shirt and erased the Crossover logo from the wine bottle; here's a cleaned-up copy: http://kegel.com/wine/wine-penguin-corkscrew.png

15 34

winequartz.drv Mac OS X UI discontinued?
by Adam Strzelecki July 11, 2009

July 11, 2009

Hi, I know that CodeWeavers supports quite nicely Mac builds of Wine, and CrossOver for Mac, yet all of them are X11. There was also once Darwine - an effort to provide Carbon Wine UI driver and Mac optimizations & L&F. Pretty many of those were merged back to Wine, but not winequartz.drv. Do you guys think about reviving winequartz.drv? This would greatly improve Wine on Mac as we could have native Mac fonts handling that's faster than FreeType, Dock icons for Wine applications, and few others. There's still old copy of the it hanging around, but it is abandoned, and no longer compiles with Wine. Or I think it would be cool to write new winecocoa.drv or sthing like that, as Apple officially states that Carbon is deprecated. While it is not well documented, it is pretty easy with Cocoa to make console non-bundled process to be UI foreground process and handle events and Dock: (This code is copyrighted by myself ;P) > int main(int argc, const char * argv[]) { > ProcessSerialNumber psn = {0, kCurrentProcess}; > OSStatus returnCode = TransformProcessType(&psn, > kProcessTransformToForegroundApplication); > SetFrontProcess(&psn); > > NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init]; > [NSApplication sharedApplication]; > > NSImage *icon = [NSImage imageNamed:@"NSApplicationIcon"]; > [NSApp setApplicationIconImage:icon]; > > [NSApp finishLaunching]; > > NSWindow *window = [[NSWindow alloc] > initWithContentRect:NSMakeRect(0, 0, 100, 100) > styleMask:NSTitledWindowMask| > NSClosableWindowMask|NSMiniaturizableWindowMask|NSResizableWindowMask > backing:NSBackingStoreBuffered defer:NO]; > [window center]; > [window setTitle:@"Test"]; > > NSButton *button = [[NSButton alloc] > initWithFrame:NSMakeRect(10, 10, 80, 80)]; > [button setTitle:@"What?"]; > [button setBezelStyle:1]; > [[window contentView] addSubview:button]; > [button release]; > > [window makeKeyAndOrderFront:NSApp]; > > while(1) { > NSEvent *event = [NSApp > nextEventMatchingMask:NSAnyEventMask > untilDate > :[NSDate distantFuture] > inMode:NSDefaultRunLoopMode > dequeue:YES > ]; > if(event) [NSApp sendEvent:event]; > } > [pool release]; > return 0; > } BTW. I heard that someone previously working for Wine project is now in Apple, it is true? Regards, -- Adam Strzelecki |: nanoant.com :|

20 55

Wine desktop integration: current directory of app launched with doubleclick?
by Dan Kegel Feb. 15, 2009

Feb. 15, 2009

I *almost* have a great success story to report; the only thing keeping it from being a success story is the current directory chosen by Nautilus when double-clicking on .exe files. My wife hurt a finger trying to impersonate a Sampsonite Luggage gorilla, and had to go to a hand doctor. Along the way her hand got x-rayed, and the doctor handed her a cd-rom with the x-ray pictures on it. The disc has an autorun.inf on it that should start ViewSel.exe. I don't know if that's supposed to work with Wine and Nautilus, but probably doubleclicking on ViewSel.exe does the same thing. ViewSel.exe puts up two big buttons: low res (which launches a web browser on an html file), and high res (which launches a DICOM viewer). If you cd to the root of the cd-rom drive and run ViewSel, it works. If the current directory is anything else, it doesn't work. If you start the autorun app via Nautilus, those buttons don't work, so presumably it sets the current directory to something other than the root of the drive. To see, I created a wrapper shell script, ~/bin/mywine, containing #!/bin/sh pwd > /tmp/log and used "Start with" to launch ViewSel.exe with ~/bin/mywine. This showed that the current directory was $HOME. I had a look at the gnome code to see how it decided, but it was a bit hard to follow. (See gnome_vfs_mime_application_launch_with_env.) So I tried a little shell magic. I created a new wrapper shell script that assumes the argument is a path to a file, and sets the current directory to the directory containing that file: #!/bin/sh DIR=`dirname "$1"` DIR=`cd "$DIR"; pwd` cd "$DIR" wine "$@" That worked better; it let ViewSel.exe launch the DICOM viewer. So... I suppose the next step is to look at the debian/ubuntu packages for wine and see if that little wrapper script could be incorporated into the default way file browsers start wine? It sure would be nice if apps that expected the current directory to behave like this (it's not uncommon!) Just Worked. - Dan

4 9

Re: [5/9] Add support of WDM drivers for USB devices.
by Alexandre Julliard Jan. 20, 2009

Jan. 20, 2009

Alexander Morozov <amorozov(a)etersoft.ru> writes: > From ed6e12ca39e194829ad8d6be4fbd91d9f276d435 Mon Sep 17 00:00:00 2001 > From: Alexander Morozov <amorozov(a)etersoft.ru> > Date: Fri, 3 Oct 2008 17:57:00 +0400 > Subject: [PATCH] Add support of WDM drivers for USB devices. > > --- > configure.ac | 14 ++ > dlls/wineusbhub/Makefile.in | 15 ++ > dlls/wineusbhub/wineusbhub.c | 291 +++++++++++++++++++++++++++++++++++++++ > dlls/wineusbhub/wineusbhub.spec | 7 + > programs/winedevice/Makefile.in | 2 +- > programs/winedevice/device.c | 79 +++++++++++- > programs/wineusb/Makefile.in | 15 ++ > programs/wineusb/main.c | 224 ++++++++++++++++++++++++++++++ > server/device.c | 111 +++++++++++++++ > server/protocol.def | 16 ++ > tools/wine.inf.in | 10 ++ > 11 files changed, 782 insertions(+), 2 deletions(-) Your design needs a lot more thought. You can't add all these Wine-specific modules, or make winedevice special-case usb devices, or poll the server for the add_device request like you do. All this needs to be properly integrated in the existing infrastructure. -- Alexandre Julliard julliard(a)winehq.org

2 1

re: Wine integration with native (Gtk, Qt, Cocoa/Carbon) themes
by Dan Kegel Jan. 15, 2009

Jan. 15, 2009

FWIW, my opinion is that any deep integration is going to be hard, and that for the moment we should go with something much simpler: simply support mstheme files well, and provide themes that look like the default themes on Ubuntu / OpenSuse / Fedora.

6 15

Valgrind warning in wined3d/swapchain.c in IWineD3DSwapChainImpl_Destroy
by Dan Kegel Dec. 23, 2008

Dec. 23, 2008

Hi Stefan, you seem to have been in this code recently, could you have a look? This is a somewhat flaky error that happens about 80% of the time under heavy load on my quad core box. (This is valgrind's xml output format, it's pretty fluffy, sorry.) Thanks, Dan <error> <kind>UninitCondition</kind> <what>Conditional jump or move depends on uninitialised value(s)</what> <stack> <frame> <obj>dlls/wined3d/wined3d.dll.so</obj> <fn>IWineD3DSwapChainImpl_Destroy</fn> <dir>dlls/wined3d</dir> <file>swapchain.c</file> <line>75</line> </frame> <frame> <obj>dlls/d3d9/d3d9.dll.so</obj> <fn>IDirect3DSwapChain9Impl_Release</fn> <dir>dlls/d3d9</dir> <file>swapchain.c</file> <line>66</line> </frame> <frame> <obj>dlls/d3d9/d3d9.dll.so</obj> <fn>D3D9CB_DestroySwapChain</fn> <dir>dlls/d3d9</dir> <file>directx.c</file> <line>427</line> </frame> <frame> <obj>dlls/wined3d/wined3d.dll.so</obj> <fn>IWineD3DDeviceImpl_Uninit3D</fn> <dir>dlls/wined3d</dir> <file>device.c</file> <line>2426</line> </frame> <frame> <obj>dlls/d3d9/d3d9.dll.so</obj> <fn>IDirect3DDevice9Impl_Release</fn> <dir>dlls/d3d9</dir> <file>device.c</file> <line>98</line> </frame> <frame> <obj>dlls/d3d9/tests/d3d9_test.exe.so</obj> <fn>func_visual</fn> <dir>dlls/d3d9/tests</dir> <file>visual.c</file> <line>9960</line> </frame> <frame> <obj>dlls/d3d9/tests/d3d9_test.exe.so</obj> <fn>run_test</fn> <dir>dlls/d3d9/tests/../../../include/wine</dir> <file>test.h</file> <line>452</line> </frame> <frame> <obj>dlls/d3d9/tests/d3d9_test.exe.so</obj> <fn>main</fn> <dir>dlls/d3d9/tests/../../../include/wine</dir> <file>test.h</file> <line>502</line> </frame> </stack> </error>

2 1

RFC: Proposed new web site design
by Jeremy White Dec. 22, 2008

Dec. 22, 2008

At Wineconf, we made the decision to change the entry page to the Wine web site. The hope was to simplify and stream line it, and to put in place the infrastructure to start moving more content to the Wiki. Jeremy Newman and Jon Parshall have put a lot of time and energy into a proposed new design, following what we took away from the meetings at Wineconf. A mock up of that design is up now here: http://wine.codeweavers.com/winehq_new/ ...and the theming intended for all 'child' pages of the web site, starting with the Wiki: http://wine.codeweavers.com/winehq_new/wiki.html Beyond the normal controversy that any new design sparks there is one major potential controversial change. That is, I am asking for prime placement on the download page on behalf of CodeWeavers instead of the (stale and ugly, sorry) banner ads we used in the past. You can see our current concept around that positioning here: http://wine.codeweavers.com/winehq_new/download.html I hope that the general community sees that as a reasonable exchange for all of the work we put into Wine and hosting the Wine server, and a vast improvement over the banner ads. At any rate, comments and feedback are welcome. And in a vain attempt to prevent all the flames from scorching Jon and Jeremy, I'd like to officially state how much I appreciate their efforts. Cheers, Jeremy

19 32

Wine integration with native (Gtk, Qt, Cocoa/Carbon) themes
by Reece Dunn Dec. 20, 2008

Dec. 20, 2008

Hi, There is a discussion going on over at https://bugs.launchpad.net/ubuntu/+source/wine/+bug/111061 about improving Wine's look and feel to better match the system that it is running on. The situation is as follows: == Colour Schemes Wine supports these by reading the settings from the registry. Winecfg can load a .theme file containing a colour profile and adapt accordingly (saving those settings to the registry). At the moment, if you need to script this (e.g. when installing Wine) you need to manipulate the registry. It would be helpful if winecfg (or some other helper utility) supported setting the theme file on the command line. In addition to this, if the user changes the theme used on their system it would not be reflected by Wine. == Windows Theme Support Windows theming support is in place to some extent, w.r.t. the XP theming APIs. There are some user32 controls and the window/dialog handlers that don't support theming yet and there are some performance issues that need resolving. This would be set via the command line used to set the colour scheme since they can be set via the same .theme file. This is limited in that you need an XP theme for each native theme available. == Native Theme Support The main thing here is that Wine would monitor the native system for theme changes. The colours of the native theme would be mapped to the Windows system colours, saved to the registry and then a WM_SYSCOLORCHANGED message would be sent to all open top-level windows so they could update to the new look. Any metric data would be mapped in a similar way. As for the theme, an implementation of uxtheme would map the API calls to the native calls. Here, it may be possible to just send a request to redraw everything on each active window. If not, a WM_THEMECHANGED message would need to be sent to all active windows. The challenge with native theme support is two-fold: (1) it should work on any system - some have Gtk, some Qt and some Cocoa/Carbon, while others (like the *BSDs and OpenSolaris) are likely not to have those engines available; (2) it should not break any Windows application. Note that as Vista has a different msstyles theming engine (it is a DLL), we could have the msstyles DLL expose the uxtheme API and have uxtheme call msstyles to do the rendering. That way, we could have a gtk.msstyles, qt3.msstyles, qt4.msstyles and an carbon.msstyles that would bind to the corresponding theming engine. If the msstyles DLL does not expose those methods, the uxtheme engine could then fallback to the current XP theme processing. For Mac theme support that would possibly require Objective-C code to be done properly, that theme could be an external package (possibly in darwine) that would be installed in addition to Wine. The same thing for the Qt engines, allowing them to directly interface with the C++ Qt APIs. Thoughts? Ideas? - Reece

7 19

No response to updated patch for gdi32/StretchDIBits
by Mathias Kosch Dec. 13, 2008

Dec. 13, 2008

Hallo! About two weeks ago I tried to hand in a (corrected version of my) patch for "StretchDIBits" [1]. However it isn't accepted until now and I didn't yet notice any response. Would someone please tell me what's wrong or if it was just missed out accidentally? [1] http://www.winehq.org/pipermail/wine-patches/2008-September/062221.html -- Mathias Kosch <info(a)mkosch.de> | http://www.mkosch.de/ RSA Public Key: http://www.mkosch.de/pubkey Roboter zum selber bauen: http://www.heise.de/ct/projekte/ct-bot/

2 2

winetest.exe now available on test.winehq.org
by Paul Vriens Dec. 1, 2008

Dec. 1, 2008

Hi, Test.winehq.org is the new home of the cross compiled winetest.exe. Alexandre put in some magic to create the new winetest.exe on winehq. The new link: http://test.winehq.org/builds/winetest-latest.exe The link can also be found at the bottom of the test.winehq.org index page. I already changed the wiki to reflect these changes. This also means that Paul Millars winetest is no longer available. I'd like to thank Paul for all the hard work he did with keeping up that site and providing us for years with a cross compiled version of winetest. I also like to thank the people that provided the necessary MinGW patches in the past to get that winetest compiled again. -- Cheers, Paul.

2 1

Bugzilla's down again
by Austin English Dec. 1, 2008

Dec. 1, 2008

Different error than usual though: <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN"> <html><head> <title>500 Internal Server Error</title> </head><body> <h1>Internal Server Error</h1> <p>The server encountered an internal error or misconfiguration and was unable to complete your request.</p> <p>Please contact the server administrator, bugs-admin(a)winehq.org and inform them of the time the error occurred, and anything you might have done that may have caused the error.</p> <p>More information about this error may be available in the server error log.</p> </body></html> Is there anything we can do to prevent it from going down so much? -- -Austin

3 3

Re: wined3d: Sign-compare warnings fix
by Austin English Dec. 1, 2008

Dec. 1, 2008

On Sat, Nov 29, 2008 at 11:42 AM, Andrew Talbot <andrew.talbot(a)talbotville.com> wrote: > Changelog: > wined3d: Sign-compare warnings fix. > > @@ -504,7 +504,7 @@ static void shader_generate_glsl_declarations(IWineD3DBaseShader *iface, const s > IWineD3DBaseShaderImpl* This = (IWineD3DBaseShaderImpl*) iface; > IWineD3DDeviceImpl *device = (IWineD3DDeviceImpl *) This->baseShader.device; > int i; > - unsigned int extra_constants_needed = 0; > + unsigned int i, extra_constants_needed = 0; > const local_constant *lconst; > > /* There are some minor differences between pixel and vertex shaders */ > > > You forgot to remove 'int i' here. -- -Austin

2 1

Error Message and Office 2000
by SWEETSKY＠ROCKETMAIL.COM Dec. 1, 2008

Dec. 1, 2008

Hi, I'm in Southern California. I did a search for the event when my Office 2000 Professional gave me an error message. It won't setup. The event was 1000 and the fault address is the same as a read out of your computer that you posted online. http://www.winehq.org/pipermail/wine-devel/2003-July/018313.html Did you ever figure out how to get it to work? Thanks! Ciao! Connie

2 1

Sound regression (in Warcraft III)
by Adam Petaccia Nov. 30, 2008

Nov. 30, 2008

Maarten, could you look at http://bugs.winehq.org/show_bug.cgi?id=16297 as you're the author of 717df5b2972b3cb998ca5a43279ae2283b117eaa? I tried adding you as CC but bugzilla complained.

2 1

Re: Fw: selinux in fedora 10 regarding text relocation in wine and mono.
by Hin-Tak Leung Nov. 30, 2008

Nov. 30, 2008

Hi all, I had managed to fix my selinux/wine/mono problem in fedora 10... it is actually two issues, in fact: 1) fedora 10 ships some much stricter selinux policies (but make some exception for wine). 2) I have kernel support for miscellaneous binary formats enabled and have win32 PE registration in the kernel, and run win32 mono.exe (or other PE executables) direct from time to time. so "wine [path]/mono.exe [.NET.exe]" works within the restriction of f10 selinux policies, but "[path]/mono.exe [.NET.exe]" does not. I could either relax the "execmod" policy in selinux, if I want to run any PE .exe's wholesale; or I just need to remember always putting "wine" in front of PE .exe's I want to run... Sorry for the noise.

1 0

Did anyone have any trouble compiling wine today? Or was it just me ....
by Susan Cragin Nov. 30, 2008

Nov. 30, 2008

and my jaunty jackalope with 2.6.27-7 generic kernel MAKE gcc -c -I. -I. -I../../include -I../../include -D__WINESRC__ -D_NTSYSTEM_ -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wwrite-strings -Wtype-limits -Wpointer-arith -g -O2 -o serial.o serial.c In file included from serial.c:77: /usr/include/linux/serial.h:164: error: expected specifier-qualifier-list before ‘__u32’ make[2]: *** [serial.o] Error 1 make[2]: Leaving directory `/home/susan/wine/dlls/ntdll' make[1]: *** [ntdll] Error 2 make[1]: Leaving directory `/home/susan/wine/dlls' make: *** [dlls] Error 2 INSTALL make[2]: Leaving directory `/home/susan/wine/dlls/newdev' make[2]: Entering directory `/home/susan/wine/dlls/ntdll' gcc -c -I. -I. -I../../include -I../../include -D__WINESRC__ -D_NTSYSTEM_ -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wwrite-strings -Wtype-limits -Wpointer-arith -g -O2 -o serial.o serial.c In file included from serial.c:77: /usr/include/linux/serial.h:164: error: expected specifier-qualifier-list before ‘__u32’ make[2]: *** [serial.o] Error 1 make[2]: Leaving directory `/home/susan/wine/dlls/ntdll' make[1]: *** [ntdll/__install__] Error 2 make[1]: Leaving directory `/home/susan/wine/dlls' make: *** [dlls/__install__] Error 2

3 2

Re: includes: Fix macros for ITextServices
by Dylan Smith Nov. 30, 2008

Nov. 30, 2008

The problems with having the macros in the public headers are that they won't work on x86 machines, and are not included in the public headers. Therefore someone developing for another platform will have their code break when using the native public headers or when porting the code to an x86 where the thiscall calling convention is used. On Sun, Nov 30, 2008 at 5:42 AM, Maarten Lankhorst <m.b.lankhorst(a)gmail.com>wrote: > Commit e4ef9c5c removed the ITextServices macros from > wine/include/textserv.h. With this they are added again, and COBJMACROS > before including that header, so that they won't conflict. > --- > > From c04fcab5168309395602f4c4fd07a319a9a532ab Mon Sep 17 00:00:00 2001 > From: Maarten Lankhorst <maarten(a)codeweavers.com> > Date: Sun, 30 Nov 2008 11:37:54 +0100 > Subject: [PATCH] includes: Fix macros for ITextServices > > --- > dlls/riched20/tests/txtsrv.c | 8 +++++++- > include/textserv.h | 19 +++++++++++++++++++ > 2 files changed, 26 insertions(+), 1 deletions(-) > > diff --git a/dlls/riched20/tests/txtsrv.c b/dlls/riched20/tests/txtsrv.c > index 3d47490..c28f5a7 100644 > --- a/dlls/riched20/tests/txtsrv.c > +++ b/dlls/riched20/tests/txtsrv.c > @@ -29,6 +29,9 @@ > #include <objbase.h> > #include <richedit.h> > #include <initguid.h> > + > +#undef COBJMACROS > +/* Don't define ITextServices, we override it here */ > #include <textserv.h> > #include <wine/test.h> > > @@ -44,6 +47,9 @@ static HMODULE hmoduleRichEdit; > #define TXTSERV_VTABLE(This) (This)->lpVtbl > #endif /* __i386__ */ > > +#define ITextServices_QueryInterface(p,a,b) > (p)->lpVtbl->QueryInterface(p,a,b) > +#define ITextServices_AddRef(p) (p)->lpVtbl->AddRef(p) > +#define ITextServices_Release(p) (p)->lpVtbl->Release(p) > #define ITextServices_TxSendMessage(This,a,b,c,d) > TXTSERV_VTABLE(This)->TxSendMessage(This,a,b,c,d) > #define ITextServices_TxDraw(This,a,b,c,d,e,f,g,h,i,j,k,l) > TXTSERV_VTABLE(This)->TxDraw(This,a,b,c,d,e,f,g,h,i,j,k,l) > #define ITextServices_TxGetHScroll(This,a,b,c,d,e) > TXTSERV_VTABLE(This)->TxGetHScroll(This,a,b,c,d,e) > @@ -84,7 +90,7 @@ static HRESULT WINAPI > ITextHostImpl_QueryInterface(ITextHost *iface, > > if (IsEqualIID(riid, &IID_IUnknown) || IsEqualIID(riid, > &IID_ITextHost)) { > *ppvObject = This; > - ITextHost_AddRef((ITextHost *)*ppvObject); > + IUnknown_AddRef((IUnknown *)*ppvObject); > return S_OK; > } > > diff --git a/include/textserv.h b/include/textserv.h > index 977b742..820fc0a 100644 > --- a/include/textserv.h > +++ b/include/textserv.h > @@ -147,6 +147,25 @@ DECLARE_INTERFACE_(ITextServices,IUnknown) > #define ITextServices_QueryInterface(p,a,b) > (p)->lpVtbl->QueryInterface(p,a,b) > #define ITextServices_AddRef(p) (p)->lpVtbl->AddRef(p) > #define ITextServices_Release(p) (p)->lpVtbl->Release(p) > +/*** ITextServices methods ***/ > +#define ITextServices_TxSendMessage(p,a,b,c,d) > (p)->lpVtbl->TxSendMessage(p,a,b > +#define ITextServices_TxDraw(p,a,b,c,d,e,f,g,h,i,j,k,l) > (p)->lpVtbl->TxDraw(p,a > +#define ITextServices_TxGetHScroll(p,a,b,c,d,e) > (p)->lpVtbl->TxGetHScroll(p,a,b > +#define ITextServices_TxGetVScroll(p,a,b,c,d,e) > (p)->lpVtbl->TxGetVScroll(p,a,b > +#define ITextServices_OnTxSetCursor(p,a,b,c,d,e,f,g,h,i) > (p)->lpVtbl->OnTxSetCu > +#define ITextServices_TxQueryHitPoint(p,a,b,c,d,e,f,g,h,i,j) > (p)->lpVtbl->TxQue > +#define ITextServices_OnTxInplaceActivate(p,a) > (p)->lpVtbl->OnTxInplaceActivate > +#define ITextServices_OnTxInplaceDeactivate(p) > (p)->lpVtbl->OnTxInplaceDeactiva > +#define ITextServices_OnTxUIActivate(p) (p)->lpVtbl->OnTxUIActivate(p) > +#define ITextServices_OnTxUIDeactivate(p) (p)->lpVtbl->OnTxUIDeactivate(p) > +#define ITextServices_TxGetText(p,a) (p)->lpVtbl->TxGetText(p,a) > +#define ITextServices_TxSetText(p,a) (p)->lpVtbl->TxSetText(p,a) > +#define ITextServices_TxGetCurrentTargetX(p,a) > (p)->lpVtbl->TxGetCurrentTargetX > +#define ITextServices_TxGetBaseLinePos(p,a) > (p)->lpVtbl->TxGetBaseLinePos(p,a) > +#define ITextServices_TxGetNaturalSize(p,a,b,c,d,e,f,g,h) > (p)->lpVtbl->TxGetNat > +#define ITextServices_TxGetDropTarget(p,a) > (p)->lpVtbl->TxGetDropTarget(p,a) > +#define ITextServices_OnTxPropertyBitsChange(p,a,b) > (p)->lpVtbl->OnTxPropertyBi > +#define ITextServices_TxGetCachedSize(p,a,b) > (p)->lpVtbl->TxGetCachedSize(p,a,b > #endif > > #undef INTERFACE > -- > 1.5.6.5 > > > > >

1 0

Re: winedos: Fixed a problem with timer values that are below 50ms.
by Michael Stefaniuc Nov. 30, 2008

Nov. 30, 2008

Hello Peter, Peter Dons Tychsen wrote: > Fixed a problem with timer values that are smaller than the minimum time > that can be assigned to the SetTimer() win32 call (approx 50ms). The > patch splits up the timer into several interrupts if too long time has > passed.This fixes good old OutRun, which uses timers of approx 10ms. As > a bonus, this aalso fixes situations where we have fallen behind for > other reasons. your patch introduces a lot of dangling whitespace at the end of lines. bye michael

4 5

today's git does not compile with yesterday's new gcc -- regression test results
by Susan Cragin Nov. 30, 2008

Nov. 30, 2008

I did the regression test. But how legitimate are the results if the problem is the new gcc that I downloaded recently, and not wine? gcc (Ubuntu 4.3.2-2ubuntu5) 4.3.3 20081129 (prerelease) susan(a)ubuntu:~/wine$ git bisect bad Bisecting: 0 revisions left to test after this [0b7478a79fb0594daba5cee3c0288884c352ad39] widl: Make sure format strings for structure and union types are only output once. Susan

3 2

Help need list of wine git versions
by Susan Cragin Nov. 30, 2008

Nov. 30, 2008

Need the number of the current git because am doing regression on it, because it doesn't compile on my machine using my gcc. gcc (Ubuntu 4.3.2-2ubuntu5) 4.3.3 20081129 (prerelease) (And BTW can the regression test page link to this page?) Thank you very much. Susan

3 2

Fw: selinux in fedora 10 regarding text relocation in wine and mono.
by Hin-Tak Leung Nov. 29, 2008

Nov. 29, 2008

I sent this a few days ago from my "other" e-mail alias and it hasn't come back, so it probably got lost in the spam-filtering - anybody has similiar issue? > So I have upgraded to fedora 10... now when I run wine with > mono for some .Net application I get selinux blocking it, > saying: > > ----------------- > > The mscorsvw.exe application attempted to load > /home/Hin-Tak/.wine/drive_c/windows/Microsoft.NET/Framework/v2.0.50727/mscorsvw.exe > which requires text relocation. This is a potential security > problem. Most libraries do not need this permission. > Libraries are sometimes coded incorrectly and request this > permission. > > ------------------ > > And a few other similar errors. Should I file this with > redhat with their shipped policy, wine or mono? I guess it > should be with redhat, but > somehow I have the impression that I might be asked to file > this with wine-devel. > > Any thoughts on this?

1 0

Hmm, guess we have to update our web site a bit
by Dan Kegel Nov. 29, 2008

Nov. 29, 2008

Somebody's walkthrough of installing crossover got lots of diggs: http://digg.com/linux_unix/Running_Windows_Programs_On_Ubuntu_8_10_With_Cro… The comments point out that http://www.winehq.org/site/myths could use a little updating...

2 1

wineconsole: Wait on events in curses backend using select instead of poll
by Kirill K. Smirnov Nov. 29, 2008

Nov. 29, 2008

>Should I add a >check for sys/select.h in the configure script and conditionally included >it, surrounded by HAVE_SYS_SELECT_H? Or would sys/time.h, sys/types.h and >unistd.h be enough to consider it reasonably compatible? Ideally - should. But... 1) There isn't any 'select.h' substring in whole wine tree. 2) There is a file libs/port/usleep.c which uses select() but it does not include any meaningful headers except "config.h" and "wine/port.h". Are these two headers enough wineconsole - I have no idea. Alexandre knows better ;-) Try and see ;-) >After fixing this, I guess I should just submit a new patch against the >current upstream git (not against the old patched version), with the same >commit message, replacing the old one? Yes, and mark a new mail subject with tag '[try 2]' to easily distinguish between different version of the same fix. -- Kirill

2 1

wineconsole: Wait on events in curses backend using select instead of poll
by Kirill K. Smirnov Nov. 29, 2008

Nov. 29, 2008

Hi, Martin, >+ ret = select(pipefd + 1, &readfds, &writefds, &errorfds, NULL); Passing NULL instead of &writefds in this case is better, IMO. It is not used anyway. ...and... I strongly believe that errorfds parameter os select() is useful for sockets only, not for stdio (I am not sure whether it is true for Darwin). Please, check this. -- Kirill

2 1

Implementing wtmpnam in msvcrt
by Tim Holy Nov. 29, 2008

Nov. 29, 2008

Hello, I'm trying to implement the "stub" function wtmpnam in msvcrt. Below is a patch at an attempt; there are a number of questions I have about the right way to implement it, and since I don't know the Wine codebase I thought perhaps readers of this list might (with minimal effort) be able to point me in the right direction. Below is the text of the patch (it's short). My questions are embedded after //-style comments (4 in total). Many thanks in advance for your help! --Tim diff -U 3 -r winecopy/wine-1.1.8/dlls/msvcrt/file.c wine-1.1.8/dlls/msvcrt/file.c --- winecopy/wine-1.1.8/dlls/msvcrt/file.c 2008-11-07 10:09:33.000000000 -0600 +++ wine-1.1.8/dlls/msvcrt/file.c 2008-11-29 07:32:43.000000000 -0600 @@ -88,6 +88,7 @@ /* INTERNAL: Static buffer for temp file name */ static char MSVCRT_tmpname[MAX_PATH]; +static MSVCRT_wchar_t MSVCRT_wtmpname[MAX_PATH]; static const unsigned int EXE = 'e' << 16 | 'x' << 8 | 'e'; static const unsigned int BAT = 'b' << 16 | 'a' << 8 | 't'; @@ -2990,6 +2991,32 @@ } /********************************************************************* + * wtmpnam (MSVCRT.@) + */ +MSVCRT_wchar_t * CDECL MSVCRT_wtmpnam(MSVCRT_wchar_t *s) +{ + static int unique; + char tmpstr[16]; + MSVCRT_wchar_t wtmpstr[16]; + MSVCRT_wchar_t *p; + int count; + if (s == 0) + s = MSVCRT_wtmpname; + msvcrt_int_to_base32(GetCurrentProcessId(), tmpstr); + p = s + MSVCRT_swprintf(s, "\\s%s.", tmpstr); // "implicit declaration" error, function is defined in wcs (but no wcs.h); but how to convert format string to MSVCRT_wchar_t? + for (count = 0; count < MSVCRT_TMP_MAX; count++) + { + msvcrt_int_to_base32(unique++, tmpstr); + mbstowcs(wtmpstr,tmpstr,strlen(tmpstr)); // need a MSVCRT_wchar version + wcscpy(p, wtmpstr); // need a MSVCRT_wchar version + if (GetFileAttributesW(s) == INVALID_FILE_ATTRIBUTES && + GetLastError() == ERROR_FILE_NOT_FOUND) + break; + } + return s; +} + +/********************************************************************* * tmpfile (MSVCRT.@) */ MSVCRT_FILE* CDECL MSVCRT_tmpfile(void) diff -U 3 -r winecopy/wine-1.1.8/dlls/msvcrt/msvcrt.spec wine-1.1.8/dlls/msvcrt/msvcrt.spec --- winecopy/wine-1.1.8/dlls/msvcrt/msvcrt.spec 2008-11-07 10:09:33.000000000 -0600 +++ wine-1.1.8/dlls/msvcrt/msvcrt.spec 2008-11-29 07:12:10.000000000 -0600 @@ -579,7 +579,7 @@ @ cdecl _wstrtime(ptr) @ cdecl _wsystem(wstr) @ cdecl _wtempnam(wstr wstr) -@ stub _wtmpnam #(ptr) +@ cdecl wtmpnam(ptr) MSVCRT_wtmpnam // needs an underscore? @ cdecl _wtoi(wstr) ntdll._wtoi @ cdecl _wtoi64(wstr) ntdll._wtoi64 @ cdecl _wtol(wstr) ntdll._wtol

3 6

new winetricks-20081129: new verbs mshflxgd, controlpad/fm20, native_oleaut32
by Dan Kegel Nov. 29, 2008

Nov. 29, 2008

Also clean up temp directory between verbs, as some verbs were writing readonly files with unique names like 'setup.exe' and stumbling over each other. As always, online at http://kegel.com/wine/winetricks Please submit bug reports to http://code.google.com/p/winezeug/issues/list

1 0

Re: winex11.drv: change bpp fixme to a warn
by James Hawkins Nov. 28, 2008

Nov. 28, 2008

On Fri, Nov 28, 2008 at 4:25 PM, Austin English <austinenglish(a)gmail.com> wrote: > This isn't something we can really fix on our end, so a FIXME isn't appropriate. > diff --git a/dlls/advapi32/security.c b/dlls/advapi32/security.c index 8229377..16aa8c3 100644 --- a/dlls/advapi32/security.c +++ b/dlls/advapi32/security.c @@ -1155,6 +1155,7 @@ DWORD WINAPI BuildSecurityDescriptorW( BOOL WINAPI InitializeSecurityDescriptor( PSECURITY_DESCRIPTOR pDescr, DWORD revision ) { + pDescr = 0; return set_ntstatus( RtlCreateSecurityDescriptor(pDescr, revision )); } Extraneous change. -- James Hawkins

2 1

Re: shell32/autocomplete: fix handling of WM_KILLFOCUS messages
by Austin English Nov. 28, 2008

Nov. 28, 2008

On Fri, Nov 28, 2008 at 10:53 AM, Jan de Mooij <jandemooij(a)gmail.com> wrote: > In eMule after moving focus from the searchbox to another control, it > still shows the caret and search is unusable without switching tabs. > > Autocomplete overrides the textbox message handler. But after handling > WM_KILLFOCUS messages we should send it to the old handler so it can > move the focus from the textbox, hide the caret, etc. > > Fixes bug 4090: http://bugs.winehq.org/show_bug.cgi?id=4090 > > Regards > > > > Can you add a testcase for this? -- -Austin

1 0

Re: dinput: Remove WINAPI on static functions where not needed.
by Vitaliy Margolen Nov. 28, 2008

Nov. 28, 2008

Francois Gouget wrote: > --- > > If you know other places where the functions touched by this patch > are used, please let me know. For more details, see: > > http://www.winehq.org/pipermail/wine-patches/2008-November/064995.html > > > dlls/dinput/joystick_linux.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/dlls/dinput/joystick_linux.c b/dlls/dinput/joystick_linux.c > index 4146f1d..47c734d 100644 > --- a/dlls/dinput/joystick_linux.c > +++ b/dlls/dinput/joystick_linux.c > @@ -966,7 +966,7 @@ static HRESULT WINAPI JoystickAImpl_GetProperty( > /****************************************************************************** > * GetObjectInfo : get object info > */ > -static HRESULT WINAPI JoystickWImpl_GetObjectInfo(LPDIRECTINPUTDEVICE8W iface, > +static HRESULT JoystickWImpl_GetObjectInfo(LPDIRECTINPUTDEVICE8W iface, > LPDIDEVICEOBJECTINSTANCEW pdidoi, DWORD dwObj, DWORD dwHow) > { > static const WCHAR axisW[] = {'A','x','i','s',' ','%','d',0}; Please don't apply this patch. This function should be used in vtable but isn't by mistake. Sending a patch to fix this. Vitaliy.

1 0

Re: [1/2] [try 4] mlang: Add implementation offnIMLangFontLink_GetCharCodePages
by Dmitry Timoshkov Nov. 28, 2008

Nov. 28, 2008

"Vitaly Perov" <vitperov(a)etersoft.ru> wrote: > Changes since last send: > - Used local mlang_data structure instead of calling internal function > wine_cp_enum_table. > - Used macro instead of "iface->lpVtbl->...." Looks good to me. -- Dmitry.

2 1

New winetricks 20081127: new verbs mono20, ie6, msmask, ole2
by Dan Kegel Nov. 27, 2008

Nov. 27, 2008

Also updates to latest directx9, and fixes a Mac problem. As always, online at http://kegel.com/wine/winetricks Please submit bug reports to http://code.google.com/p/winezeug/issues/list Thanks to everybody who sent in suggestions and patches! ie6 - Steven Edwards msmask - Austin mac bugfix - Ryan Schmidt

1 0

Re: programs: Fix broken compilation on win64
by Rob Shearman Nov. 27, 2008

Nov. 27, 2008

2008/11/27 Maarten Lankhorst <maarten(a)codeweavers.com>: > --- > Set the Spare[0] at runtime after a critical section is entered. :-) > > I'm not testing Spare[0] at all calls to EnterCriticalSection, but if this > is a problem I will send a revised patch IMHO, this is too fragile. My solution would be two macros: one for declaring a variable storing the name for a critical section, and a second for putting that name into the critical section structure. The latter would be something like the following for 64-bit: #define WINE_DEBUG_USE_CRIT_SECT_NAME(name) { (DWORD)(DWORD_PTR)name, (DWORD)((DWORD_PTR)name >> 32) } It might even be better to let the macro fill in / declare the entire CRITICAL_SECTION_DEBUG stucture. -- Rob Shearman

2 1

Re: d3dxof: DPRINTF -> TRACE.
by Christian Costa Nov. 27, 2008

Nov. 27, 2008

DPRINTF use here is intentional. Christian Andrew Riedi a écrit : > Changelog: > d3dxof: DPRINTF -> TRACE. > > > ------------------------------------------------------------------------ > >

1 0

Re: [3/5] WineD3D: Fix D3DFMT_R32F, R16F, G16R16F, ... in the pixel shader
by Roderick Colenbrander Nov. 27, 2008

Nov. 27, 2008

Hi, I don't know much about the current shader code but is this the right way to go thinking about GL_EXT_texture_swizzle + GL_ARB_texture_rg? For instance Nvidia drivers will soon get the swizzle extension and for that you don't have to fixups anymore. Is moving this way the right approach, so that it won't become a mess later on? Roderick -- Psssst! Schon vom neuen GMX MultiMessenger gehört? Der kann`s mit allen: http://www.gmx.net/de/go/multimessenger

3 11

Re: [1/2] [try 3] mlang: Add implementation offnIMLangFontLink_GetCharCodePages
by Dmitry Timoshkov Nov. 27, 2008

Nov. 27, 2008

"Vitaly Perov" <vitperov(a)etersoft.ru> wrote: > Changes since last send: > - Used WINAPI function instead of accessing Wine internal code page > structures. You still access internal Wine code page structures, I'm not sure why you claim otherwise. > "Dmitry Timoshkov" <dmitry(a)codeweavers.com> wrote: >>It *is* possible to use EnumSystemCodePages > > Yes, it is possible. But : > 1) this function call wine_cp_enum_table, then converts the result (codepage) > to the string. So we need convert the result back to the number. > 2) It pass code pages to the callback-function. The callback function uses > only one argument - string with codepage number. So it's very difficult to > pass this value back to the GetCharCodePages function. > > So, I think the best decision is to use internal function wine_cp_enum_table What's happend to the idea to use mlang_data? And again: forget about Wine internal code page structures, that's the implementation detail that's internal to kernel32/ntdll, that structures should not be used outside of kernel32/ntdll. > + iface->lpVtbl->CodePageToCodePages(iface, > + cpTable->info.codepage, &codePages); Have a look how other parts in Wine call vtable members, the tests have enough code samples. -- Dmitry.

1 0

Re: user32: Improve the LoadImage() test.
by Dmitry Timoshkov Nov. 27, 2008

Nov. 27, 2008

"Andrew Riedi" <andrewriedi(a)gmail.com> wrote: > - icon_entry->xHotspot = 1; > - icon_entry->yHotspot = 1; > + icon_entry->xHotspot = 1; /* Color planes for .ico. */ > + icon_entry->yHotspot = ICON_BPP; /* BPP for .ico. */ ... > ok(icon_info.xHotspot == 1, "xHotspot is %u.\n", icon_info.xHotspot); > - ok(icon_info.yHotspot == 1, "yHotspot is %u.\n", icon_info.yHotspot); > + ok(icon_info.yHotspot == 32, "yHotspot is %u.\n", icon_info.yHotspot); It should be ICON_BPP instead of 32 for consistency. > + /* Test loading an icon as an icon. */ > + SetLastError(0xdeadbeef); > + handle = LoadImageA(NULL, "icon.ico", IMAGE_ICON, 0, 0, LR_LOADFROMFILE); > + ok(handle != NULL, "LoadImage() failed.\n"); > + error = GetLastError(); > + ok(error == 0 || > + broken(error == 0xdeadbeef) || /* Win9x */ > + broken(error == ERROR_BAD_PATHNAME), /* Win98, WinMe */ > + "Last error: %u\n", error); There is no point in testing last error value if the API didn't fail. > + /* Test the icon information. */ > + SetLastError(0xdeadbeef); > + ret = GetIconInfo(handle, &icon_info); > + ok(ret, "GetIconInfo() failed.\n"); > + error = GetLastError(); > + ok(error == 0xdeadbeef, "Last error: %u\n", error); Same here. > + if (ret) > + { > + ok(icon_info.fIcon == TRUE, "fIcon == FALSE.\n"); > + ok(icon_info.xHotspot == 16, "xHotspot is %u.\n", icon_info.xHotspot); > + ok(icon_info.yHotspot == 16, "yHotspot is %u.\n", icon_info.yHotspot); > + ok(icon_info.hbmColor != NULL, "No hbmColor!\n"); > + ok(icon_info.hbmMask != NULL, "No hbmMask!\n"); > + } 'if (ret)' is useless if you don't expect a failure. > + > + /* Clean up. */ > + SetLastError(0xdeadbeef); > + ret = DestroyIcon(handle); > + ok(ret, "DestroyIcon() failed.\n"); > + error = GetLastError(); > + ok(error == 0xdeadbeef, "Last error: %u\n", error); Same here. -- Dmitry.

3 2

Re: ole32: Add Co*InitializeSpy functions
by Rob Shearman Nov. 27, 2008

Nov. 27, 2008

2008/11/26 Maarten Lankhorst <maarten(a)codeweavers.com>: > + > +HRESULT WINAPI CoRevokeInitializeSpy(ULARGE_INTEGER cookie) > +{ > + struct oletls *info = COM_CurrentInfo(); > + TRACE("(%s)\n", wine_dbgstr_longlong(cookie.QuadPart)); > + > + if (!info || !info->spy || cookie.QuadPart != (DWORD_PTR)info->spy) > + return E_INVALIDARG; > + > + IUnknown_Release(info->spy); > + return S_OK; > +} No function documentation and you forget to clear info->spy before returning. -- Rob Shearman

2 1

Re: [2/5] WineD3D: Use a separate enum for pixel format fixups
by Henri Verbeet Nov. 27, 2008

Nov. 27, 2008

2008/11/27 Stefan Dösinger <stefan(a)codeweavers.com>: > +enum shader_color_conv > +{ > + fmtconv_none = 1, > + set_blue_1 = 2, > + signed_2_no_ba = 3, > + signed_2 = 4, > + signed_4 = 5, > + signed_l6v5u5 = 6, > + swap_rg_blue_1 = 7, > + a_to_g_ba_1 = 8, > + yuv_yuy2 = 9, > + yuv_uyvy = 10, > + yuv_yv12 = 11, > + force_dword = 0x7ffffff > +}; > Could you please use uppercase enum elements like the rest of the world? :-/

1 0

Re: ntdll: DPRINTF -> TRACE. (heap.c)
by Dmitry Timoshkov Nov. 27, 2008

Nov. 27, 2008

"Andrew Riedi" <andrewriedi(a)gmail.com> wrote: > Changelog: > ntdll: DPRINTF -> TRACE. (heap.c) DPRINTF is used for a reason here. -- Dmitry.

2 2

Re: imm32: Add some really basic tests for ImmSetCompositionString().
by Dmitry Timoshkov Nov. 27, 2008

Nov. 27, 2008

"Andrew Riedi" <andrewriedi(a)gmail.com> wrote: > +static int test_ImmSetCompositionString(void) It's a usual practice to have the test cases return 'void', especially if you don't use the return value. > +{ > + HIMC imc; > + BOOL ret; > + DWORD error; > + > + imc = ImmGetContext(hwnd); First thing to test is the return value of ImmGetContext() and report last error in the case of a failure. -- Dmitry.

2 1

Re: [1/5] D3D9: Document a Microsoft header bug
by Adam Petaccia Nov. 26, 2008

Nov. 26, 2008

On Thu, 2008-11-27 at 01:30 +0100, Stefan Dösinger wrote: > > + > + /* Note: Microsoft's d3d9.h does not declare > IDirect3D9Ex::RegisterSoftwareDevice . This would mean that > + * the offsets of the other methods in the Vtable change too. > This, however, seems wrong. In Microsoft's > + * d3d9.dll, the offsets for the other functions are still > compatible with IDirect3D9. > + * (This is probably because even in MS's header IDirect3D9Ex > inherits from IDirect3D9, which makes the > + * C++ inferface compatible, and nobody uses the C interface in > Windows world > + */ Maybe this should be documented via Wine's API doc "BUGS: " declaration as well: http://www.winehq.org/site/docs/winedev-guide/api-docs

1 0

Re: mshtml: Do a case-insensitive compare of type (resend 1)
by Alistair Leslie-Hughes Nov. 26, 2008

Nov. 26, 2008

Any thing wrong with this patch? Best Regards Alistair Leslie-Hughes "Alistair Leslie-Hughes" <leslie_alistair(a)hotmail.com> wrote in message news:492681BB.2090608(a)hotmail.com... > Hi, > > Changelog: > mshtml: Do a case-insensitive compare of type > -------------------------------------------------------------------------------- > >From ab3346563682349645f07147c5adbcfc3ce90ebf Mon Sep 17 00:00:00 2001 > From: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> > Date: Thu, 13 Nov 2008 21:38:13 +1100 > Subject: [PATCH] Do a case-insensitive compare of type > To: wine-patches <wine-patches(a)winehq.org> > > --- > dlls/mshtml/script.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/dlls/mshtml/script.c b/dlls/mshtml/script.c > index 4d10853..90bfef9 100644 > --- a/dlls/mshtml/script.c > +++ b/dlls/mshtml/script.c > @@ -640,7 +640,7 @@ static BOOL get_guid_from_type(LPCWSTR type, GUID > *guid) > {'t','e','x','t','/','j','a','v','a','s','c','r','i','p','t',0}; > > /* FIXME: Handle more types */ > - if(!strcmpW(type, text_javascriptW)) { > + if(!strcmpiW(type, text_javascriptW)) { > *guid = CLSID_JScript; > }else { > FIXME("Unknown type %s\n", debugstr_w(type)); > -- > 1.5.4.3 > > -------------------------------------------------------------------------------- > >

1 0

re: RFC: Proposed new web site design
by Dan Kegel Nov. 26, 2008

Nov. 26, 2008

Looks great so far. Comments: 1) nested scrollbars are evil. Better to put the news on its own tab, lose the little sidebar the news was in, and have a very small area in the main box for just the one most important headline. 2) The main choice text needs to be closer to user tasks. So instead of Information Overview of Wine features. we should have About What is Wine, and why should I use it? and we're missing the user task Will my app work with Wine? which should link to the appdb.

8 8

Re: atl: Remove WINAPI on static functions where not needed.
by Ken Thomases Nov. 26, 2008

Nov. 26, 2008

Thanks for taking care of this, Francois. On Nov 23, 2008, at 5:15 PM, Francois Gouget wrote: > Astute readers will notice that this leaves one loophole: nasty macro > use like in dlls/winmm/message16.c: > > #define A(_x,_y) MMDRV_InstallMap(_x, \ > MMDRV_##_y##_Map16To32W, MMDRV_##_y##_UnMap16To32W, \ > MMDRV_##_y##_Map32WTo16, MMDRV_##_y##_UnMap32WTo16, \ > MMDRV_##_y##_Callback) > A(MMDRV_AUX, Aux); > > A grep would not find that MMDRV_Aux_Callback is referenced by the > above > code. Could you perhaps grep the output from the preprocessor, instead? -Ken

2 1

Re: mshtml: suppress module unloading while gecko is loaded (take 2)
by Paul Vriens Nov. 26, 2008

Nov. 26, 2008

Dan Kegel wrote: > Thanks to Mike Hearn for pointing out that mshtml reference > counting was wrong, and Jacek for pointing out how to fix > it for now. > > Once gecko is loaded, don't let mshtml unload. > This fixes a crash in starting Sketchup (bug 16164), > and probably fixes a number of other apps that > went pear-shaped after unloading mshtml... > > > ------------------------------------------------------------------------ > > C++ style comment? Was this a Patchwatcher test ;) -- Cheers, Paul.

1 0

Icons as cursors patch
by Krzysztof Drewniak Nov. 25, 2008

Nov. 25, 2008

This is my first patch. It allows icons to be loaded as cursors because it defaults hotspots to x = 0 and y = 0 if the hotspot isn't found. Pasting patch into body of message. This is against the latest git . -------------------------------------------------------------------------------------------------------------------------- --- cursoricon.c.orig 2008-11-23 15:07:32.000000000 -0600 +++ cursoricon.c 2008-11-23 15:13:22.000000000 -0600 @@ -932,7 +932,10 @@ static HICON CURSORICON_LoadFromFile( LP entry = CURSORICON_FindBestIconFile( dir, width, height, colors ); if ( !entry ) - goto end; + /* this is my change. If there is no hotspot, default to 0, 0. */ + hotspot.x = 0; + hotspot.y = 0; + goto iconskip; /* check that we don't run off the end of the file */ if ( entry->dwDIBOffset > filesize ) @@ -942,6 +945,7 @@ static HICON CURSORICON_LoadFromFile( LP hotspot.x = entry->xHotspot; hotspot.y = entry->yHotspot; +iconskip: hIcon = CURSORICON_CreateIconFromBMI( (BITMAPINFO *)&bits[entry->dwDIBOffset], hotspot, !fCursor, 0x00030000, width, height, loadflags ); -------------------------------------------------------------------------------------------------------------------------- Please look over the code. Krzys D.

4 7

Re: mshtml: suppress module unloading until we figure out why some apps crash with it enabled
by Jacek Caban Nov. 25, 2008

Nov. 25, 2008

Dan Kegel wrote: > This prevents http://bugs.winehq.org/show_bug.cgi?id=16164 > > ( http://www.microsoft.com/msj/0197/activex/activex0197.aspx looks > like a nice, scary article on the subject. ) > - return module_ref ? S_FALSE : S_OK; + /* Some apps crash if we let mshtml be unloaded. + * Until we figure out why, don't let mshtml be unloaded. + */ + return S_FALSE; /* module_ref ? S_FALSE : S_OK; */ I think we should prevent unloading mshtml.dll if Gecko was loaded. To do it I'd suggest to increase module count in load_gecko function (it will never be decreased so DLL won't be unloaded). Jacek

1 0