Re: winemac: Add registry settings to make Option keys send Alt rather than accessing additional characters from the keyboard layout.

2013-10-09 Thread Ken Thomases
On Oct 9, 2013, at 4:49 PM, Phil Krylov wrote: > Thanks! You're welcome. > Do you consider adding an option to stop interpreting Command as Alt? I have not considered it. What would be gained? Do you want the Command key interpreted as the Windows key? Do you want something else to happen w

Re: winemac.drv: add fullscreen mode support for Mac OS X 10.7+

2013-10-08 Thread Ken Thomases
Hi, I finally got around to working on support for Cocoa full-screen mode in the Mac driver, based on the work of Kevin Eaves. I've attached a new patch. This patch can only be applied on top of the other Mac driver patches I just submitted to wine-patches. Some changes from Kevin's original

Re: [PATCH 3/3] winemac: Tell Wine when Cocoa has brought a window to the front.

2013-10-04 Thread Ken Thomases
On Oct 4, 2013, at 12:17 AM, Ken Thomases wrote: > --- > dlls/winemac.drv/cocoa_app.m| 8 ++-- > dlls/winemac.drv/cocoa_window.h | 1 + > dlls/winemac.drv/cocoa_window.m | 16 +--- > dlls/winemac.drv/event.c| 5 + > dlls/winemac.drv/macdrv.h

Looking for an icon for the Mac driver: generic program running under Wine

2013-10-02 Thread Ken Thomases
Hi, I have a need for a new icon in Wine and I'm hoping somebody with some graphics-design skills might be able to create it. The Mac driver attempts to extract an icon from the executable to use for the Dock icon of its process. This is also the icon that appears in the Command-Tab applicati

Re: winemac: Reapply display modes when switching back to app after "escaping" with Command-Tab.

2013-10-02 Thread Ken Thomases
On Oct 1, 2013, at 11:24 PM, Ken Thomases wrote: > May fix bug <http://bugs.winehq.org/show_bug.cgi?id=34475>. Should fix bug <http://bugs.winehq.org/show_bug.cgi?id=34209>, too. -Ken

Re: winex11: Don't trace a garbage value or read past end of caller's array in X11DRV_wglChoosePixelFormatARB().

2013-09-20 Thread Ken Thomases
On Sep 20, 2013, at 9:44 AM, Roderick Colenbrander wrote: > That change looks suspicious. If switching to i+1 fixes a problem, the real > problem is that the if-statement in question gets entered in the first > place. Some bounds checking on the outer for-loop (the one which loops > through nCfgs

Re: [PATCH 1/5] user32: Add a SetMinMaxInfo driver entry point.

2013-09-18 Thread Ken Thomases
On Sep 16, 2013, at 1:21 PM, Ken Thomases wrote: > On Sep 16, 2013, at 12:55 PM, Alexandre Julliard wrote: > >> It's better to avoid adding entry points that don't correspond to >> Windows APIs. Instead you should request the info when you need it. > >

Re: [PATCH 1/5] user32: Add a SetMinMaxInfo driver entry point.

2013-09-16 Thread Ken Thomases
On Sep 16, 2013, at 12:55 PM, Alexandre Julliard wrote: > Ken Thomases writes: > >> --- >> dlls/user32/driver.c | 12 >> dlls/user32/user_private.h |1 + >> dlls/user32/winpos.c |2 ++ >> 3 files changed, 15 insertions(+), 0 de

Re: [PATCH 3/6] user32: Export WINPOS_GetMinMaxInfo() as __wine_get_min_max_info() so it can be called by drivers.

2013-09-12 Thread Ken Thomases
On Sep 12, 2013, at 10:48 AM, Dmitry Timoshkov wrote: > Ken Thomases wrote: > >>>> +@ cdecl __wine_get_min_max_info(long ptr ptr ptr ptr) WINPOS_GetMinMaxInfo >>> >>> What's wrong with calling SendMessage(WM_GETMINMAXINFO) from the driver? >>

Re: [PATCH 3/6] user32: Export WINPOS_GetMinMaxInfo() as __wine_get_min_max_info() so it can be called by drivers.

2013-09-12 Thread Ken Thomases
On Sep 12, 2013, at 12:23 AM, Dmitry Timoshkov wrote: > Ken Thomases wrote: > >> +@ cdecl __wine_get_min_max_info(long ptr ptr ptr ptr) WINPOS_GetMinMaxInfo > > What's wrong with calling SendMessage(WM_GETMINMAXINFO) from the driver? Because I would have to duplicate

Re: winemac: Don't return characters for Ctrl-(non-letter symbol) keypresses from ToUnicodeEx() as X11 driver already does (try 2)

2013-09-10 Thread Ken Thomases
On Sep 10, 2013, at 5:45 AM, Phil Krylov wrote: > On Tue, Sep 10, 2013 at 6:05 AM, Ken Thomases wrote: >> Is there a specific problem that you're trying to fix? > > Yes. I didn't file a bug report yet, but an application I am using > (IBM Translation Manager) has

Re: winemac: Don't return characters for Ctrl-(non-letter symbol) keypresses from ToUnicodeEx() as X11 driver already does (try 2)

2013-09-09 Thread Ken Thomases
Hi, On Sep 9, 2013, at 7:38 PM, Phil Krylov wrote: > Sorry, my first try had wrong patch attached > > -- Ph. > <0001-winemac-Don-t-return-characters-for-Ctrl-non-letter-try2.patch.txt> Is there a specific problem that you're trying to fix? The Mac driver's ToUnicodeEx() implementation started

Re: RFC: Mark dylib/mach-o on Mac OS X

2013-08-27 Thread Ken Thomases
Hi, On Aug 27, 2013, at 4:08 PM, André Hentschel wrote: > @@ -94,6 +95,9 @@ static void module_fill_module(const WCHAR* in, WCHAR* out, > size_t size) > if (len > 3 && !strcmpiW(&out[len - 3], S_DotSoW) && > (l = match_ext(out, len - 3))) > strcpyW(&out[len - l -

Re: [PATCH] winemac.drv: Support the public UTF-16 type for Unicode text. (try 2)

2013-08-22 Thread Ken Thomases
On Aug 22, 2013, at 12:07 AM, Charles Davis wrote: > Try 2: Eliminiate dead store. Sort export functions by name. Looks OK to me. -Ken

Re: [PATCH] winemac.drv: Support the public UTF-16 type for Unicode text.

2013-08-22 Thread Ken Thomases
On Aug 21, 2013, at 11:49 PM, Charles Davis wrote: > On Aug 21, 2013, at 10:34 PM, Ken Thomases wrote: > >> On Aug 21, 2013, at 9:42 PM, Charles Davis wrote: >>> +static HANDLE import_utf16_to_unicodetext(CFDataRef data) >>> +{ >>> +const WCHA

Re: [PATCH] winemac.drv: Support the public UTF-16 type for Unicode text.

2013-08-21 Thread Ken Thomases
On Aug 21, 2013, at 9:42 PM, Charles Davis wrote: > @@ -90,6 +93,9 @@ static CFDataRef export_hdrop_to_filenames(HANDLE data); > static CFDataRef export_oemtext_to_utf8(HANDLE data); > static CFDataRef export_text_to_utf8(HANDLE data); > static CFDataRef export_unicodetext_to_utf8(HANDLE data); >

Re: [PATCH] winemac.drv: Use the public UTF-16 type for Unicode text.

2013-08-21 Thread Ken Thomases
On Aug 21, 2013, at 10:12 AM, Ken Thomases wrote: > On Aug 20, 2013, at 10:49 PM, Charles Davis wrote: > >> In the Windows world, "Unicode" almost universally means "UTF-16". So, >> use the well-known UTF-16 type instead of making up our own. >> &g

Re: [PATCH] winemac.drv: Use the public UTF-16 type for Unicode text.

2013-08-21 Thread Ken Thomases
On Aug 20, 2013, at 10:49 PM, Charles Davis wrote: > In the Windows world, "Unicode" almost universally means "UTF-16". So, > use the well-known UTF-16 type instead of making up our own. > > I have to wonder if there was a good reason Ken didn't use this > initially. Please hold this patch while

Re: configure.ac: allow disabling winemac.drv

2013-08-08 Thread Ken Thomases
On Aug 8, 2013, at 5:56 PM, Austin English wrote: > +AC_ARG_WITH(winemac, AS_HELP_STRING([--without-winemac],[do not build > native Mac (Cocoa) driver]), > +[if test "x$withval" = "xno"; then > ac_cv_header_ApplicationServices_ApplicationServices_h=no; fi]) A switch to disable the

Re: gdi32: Avoid a crash by loading AppKit before using Core Text if might use the Mac driver.

2013-08-02 Thread Ken Thomases
On Aug 2, 2013, at 9:57 AM, Ken Thomases wrote: > OK. So, is it acceptable to load AppKit regardless of which graphics driver > is configured? And, if so, would you prefer that I just link gdi32 against AppKit rather than loading it dynamically? That would mean that it would be loade

Re: gdi32: Avoid a crash by loading AppKit before using Core Text if might use the Mac driver.

2013-08-02 Thread Ken Thomases
On Aug 2, 2013, at 9:19 AM, Alexandre Julliard wrote: > Ken Thomases writes: > >> On Aug 2, 2013, at 4:22 AM, Alexandre Julliard wrote: >>> That's ugly. freetype.c has no business knowing about the details of the >>> graphics driver. >> >> Is it

Re: gdi32: Avoid a crash by loading AppKit before using Core Text if might use the Mac driver.

2013-08-02 Thread Ken Thomases
On Aug 2, 2013, at 4:22 AM, Alexandre Julliard wrote: > Ken Thomases writes: > >> +/* If the Mac driver might be used, then load AppKit now before using >> the Core Text API. >> + Otherwise, AppKit crashes on Mac OS X 10.7+ because CTFontDescri

Re: Need help with debugging a directx9 game crashing

2013-07-25 Thread Ken Thomases
On Jul 25, 2013, at 2:11 PM, Qian Hong wrote: > On Fri, Jul 26, 2013 at 2:52 AM, Ken Thomases wrote: >> I think your Valgrind results are telling us that there's a bug in the game >> where it's using an uninitialized stack variable. >> >> There's

Re: Need help with debugging a directx9 game crashing

2013-07-25 Thread Ken Thomases
Hi Qian, On Jul 25, 2013, at 11:53 AM, Qian Hong wrote: > On Thu, Jul 25, 2013 at 2:33 AM, Ken Thomases wrote: >> Hmm. You may need to mark the dummy array as volatile to prevent it from >> being optimized away. Worth trying. You might also increase the size of >> th

Re: [PATCH] winemac.drv: Advertise some legacy WGL extensions in the GL_EXTENSIONS string. (try 3)

2013-07-25 Thread Ken Thomases
On Jul 25, 2013, at 10:49 AM, Charles Davis wrote: > Try 3: Don't advertise WGL_EXT_swap_control if vsync is disabled. Looks good. Thanks, Ken

Re: [PATCH] winemac.drv: Advertise some legacy WGL extensions in the GL_EXTENSIONS string. (try 2)

2013-07-25 Thread Ken Thomases
On Jul 25, 2013, at 9:57 AM, Ken Thomases wrote: > If vsync is disallowed, the corresponding functions are left as NULL in the > OpenGL function table, so clients may crash if WGL_EXT_swap_control is > advertised. Well, this part is wrong. They won't crash because opengl32 pr

Re: [PATCH] winemac.drv: Advertise some legacy WGL extensions in the GL_EXTENSIONS string. (try 2)

2013-07-25 Thread Ken Thomases
On Jul 24, 2013, at 9:41 PM, Ken Thomases wrote: > On Jul 24, 2013, at 9:34 PM, Charles Davis wrote: > >> Try 2: Don't duplicate a constant string; we might need to add more legacy >> extensions. (Hopefully not. ;) > > Thanks for making that change. Looks good

Re: [PATCH] winemac.drv: Advertise some legacy WGL extensions in the GL_EXTENSIONS string. (try 2)

2013-07-24 Thread Ken Thomases
On Jul 24, 2013, at 9:34 PM, Charles Davis wrote: > Try 2: Don't duplicate a constant string; we might need to add more legacy > extensions. (Hopefully not. ;) Thanks for making that change. Looks good to me. Cheers, Ken

Re: [PATCH] winemac.drv: Also return the number of pixel formats if the caller didn't supply a PIXELFORMATDESCRIPTOR.

2013-07-24 Thread Ken Thomases
On Jul 23, 2013, at 10:26 PM, Charles Davis wrote: > This is akin to what glxdrv already does, and apparently what Windows > does. It allows the GLView sample from the OpenGL SuperBible to > correctly enumerate pixel formats with the Mac driver. > --- > dlls/winemac.drv/opengl.c |2 +- > 1 file

Re: [PATCH] winemac.drv: Advertise some legacy WGL extensions in the GL_EXTENSIONS string.

2013-07-24 Thread Ken Thomases
Hi, On Jul 23, 2013, at 10:26 PM, Charles Davis wrote: > @@ -1226,8 +1227,10 @@ static BOOL init_gl_info(void) > gl_info.glVersion = HeapAlloc(GetProcessHeap(), 0, strlen(str) + 1); > strcpy(gl_info.glVersion, str); > str = (const char*)opengl_funcs.gl.p_glGetString(GL_EXTENSIONS); >

Re: Need help with debugging a directx9 game crashing

2013-07-24 Thread Ken Thomases
On Jul 24, 2013, at 12:49 PM, Qian Hong wrote: > Hi Ken, > > On Sun, Jul 21, 2013 at 2:39 PM, Ken Thomases wrote: >> Instead of enabling the trace, try replacing it with something like: >> >>char dummy[256]; >>memset(dummy, 0x55, sizeof(dummy)); >&g

Re: Need help with debugging a directx9 game crashing

2013-07-20 Thread Ken Thomases
On Jul 19, 2013, at 8:34 AM, Qian Hong wrote: > I was debugging on a popular game [1] [2] (9 GB), it crashes on start. > > +relay,+seh,+tid log show that there are some calls to > wined3d.wined3d_mutex_lock / wined3d.wined3d_buffer_map / > wined3d.wined3d_mutex_unlock before crashing, so I turn o

Re: Fix to support input of characters with diacritic symbols using AltGr

2013-07-01 Thread Ken Thomases
On Jul 1, 2013, at 4:26 PM, Ken Thomases wrote: > On Jul 1, 2013, at 3:29 PM, Юрий Воротилов wrote: > >> Fix to support input of characters with diacritic symbols using AltGr (e.g. >> Latvian - AltGr+a=ā). >> Tested on Ubuntu 12.04 and Centos 6. >> I also have a b

Re: Fix to support input of characters with diacritic symbols using AltGr

2013-07-01 Thread Ken Thomases
On Jul 1, 2013, at 3:29 PM, Юрий Воротилов wrote: > Fix to support input of characters with diacritic symbols using AltGr (e.g. > Latvian - AltGr+a=ā). > Tested on Ubuntu 12.04 and Centos 6. > I also have a bug opened in Crossover tracker: > http://www.codeweavers.com/support/tickets/browse/?tick

Re: comctl32: Use default GUI font as a fallback instead of a (non-existent) Arial.

2013-06-17 Thread Ken Thomases
On Jun 18, 2013, at 1:26 AM, Dmitry Timoshkov wrote: > Daniel Jeliński wrote: > >> I'm quite sure you didn't mean to ignore return value of GetStockObject and >> leave logfont uninitialized here. > > GetStockObject(DEFAULT_GUI_FONT) can't fail. I think the question is: what does line 1917 do?

Re: Using of mac driver completely breaks tablets support?

2013-06-12 Thread Ken Thomases
Hi, On Jun 12, 2013, at 10:47 AM, Andrey Upadyshev wrote: > I've installed a Wine 1.5.31 instead of 1.4.smthng and found that tablets > support is broken. Some research gives me that wintab32 DLL fails to init > because winemac driver doesn't implement tablet related functions > (LoadTabletInfo e

Re: [PATCH 1/8] winemac: Don't unminimize a window on first activation during start-up.

2013-06-04 Thread Ken Thomases
On Jun 4, 2013, at 5:15 AM, Dmitry Timoshkov wrote: > Ken Thomases wrote: > >> --- >> dlls/winemac.drv/cocoa_app.h |2 ++ >> dlls/winemac.drv/cocoa_app.m |3 ++- >> 2 files changed, 4 insertions(+), 1 deletions(-) > > I wonder, do you actually run &

Re: (try 2)macdrv: rework the way we handle cursor position and composition text

2013-05-29 Thread Ken Thomases
On May 29, 2013, at 1:11 PM, Aric Stewart wrote: > --- > dlls/winemac.drv/cocoa_window.m | 9 + > dlls/winemac.drv/event.c| 5 --- > dlls/winemac.drv/ime.c | 85 + > dlls/winemac.drv/macdrv.h | 1 - > dlls/winemac.drv/macdrv_cocoa.h

Re: macdrv: rework the way we handle cursor position and composition text

2013-05-29 Thread Ken Thomases
On May 29, 2013, at 10:37 AM, Aric Stewart wrote: > --- > dlls/winemac.drv/cocoa_window.m | 9 + > dlls/winemac.drv/event.c| 3 -- > dlls/winemac.drv/ime.c | 85 + > dlls/winemac.drv/macdrv.h | 1 - > dlls/winemac.drv/macdrv_cocoa.h

Re: [PATCH 2/2] user32/tests: Test that sending WM_CANCELMODE cancels tracking in window's menu bar.

2013-05-16 Thread Ken Thomases
On May 17, 2013, at 12:21 AM, Dmitry Timoshkov wrote: > Ken Thomases wrote: > >> dlls/user32/tests/menu.c | 21 - > > Please add a message test instead, it would much more clearer show what is > going on. I added to the existing tests of WM_CANCELMOD

Re: winemac.drv: add fullscreen mode support for Mac OS X 10.7+

2013-05-16 Thread Ken Thomases
On May 15, 2013, at 8:49 PM, Kevin Eaves wrote: > … The title is also empty when exiting borderless to titled, which can be > fixed here. Hmm, good catch. I'll have to check, but I suspect the window no longer remembers its title at that point. And the Mac driver doesn't currently store it

Re: winemac: Disable the zoom button for windows that are disabled.

2013-05-14 Thread Ken Thomases
I have submitted an alternative patch to fix this issue. It ended up being more involved. First, on considering exactly how disabling a window should interact with its resizability features, I decided that disabling should not remove NSResizableWindowMask from the style mask. Consider a windo

Re: winemac: Disable the zoom button for windows that are disabled.

2013-05-13 Thread Ken Thomases
On May 12, 2013, at 3:22 PM, Ken Thomases wrote: > As mentioned in the other thread, I'm not seeing the problem that this is > meant to solve on 10.6. I will test 10.7+ tomorrow. OK, I've tested on 10.8 and I see that the zoom button highlights when the mouse moves over it, a

Re: winemac: Disable the zoom button for windows that are disabled.

2013-05-12 Thread Ken Thomases
Hi Kevin, As mentioned in the other thread, I'm not seeing the problem that this is meant to solve on 10.6. I will test 10.7+ tomorrow. Also, I'm pretty sure that setting the style mask after disabling the buttons can reenable them. At least in some versions of the frameworks, we've seen that

Re: winemac.drv: add fullscreen mode support for Mac OS X 10.7+

2013-05-12 Thread Ken Thomases
nasty, but works great. The only real problem is with user32. > Everything else is easy to clean up and get working with Wine properly > without any issues. Just wanted to show that. Sure, that's what I figured. > On May 9, 2013, at 9:56 PM, Ken Thomases wrote: > >>&g

Re: winemac.drv: add fullscreen mode support for Mac OS X 10.7+

2013-05-09 Thread Ken Thomases
Hi, On May 9, 2013, at 5:48 PM, Kevin Eaves wrote: > Attached patch is a proof-of-concept for the fullscreen APIs provided in OS X > 10.7+, which allows applications to enter fullscreen in a separate desktop > space. Thanks for this. It's quite interesting. > The fullscreen button appears if

Re: [1/3] winex11: Select for XRandR screen change events and implement an event handler.

2013-05-09 Thread Ken Thomases
On May 9, 2013, at 1:36 PM, Alexandre Julliard wrote: > Handling external resizes > should be done in the desktop process, and most likely involves the > wineserver too. It certainly won't be an easy task. Hmm. Then that probably means I messed it up in the Mac driver, since what I did didn't s

Re: [PATCH] kernel32: Fixed string comparision for non-canonical locale identifiers such as zh_CN VS zh-Hans on OS X.

2013-04-29 Thread Ken Thomases
On Apr 29, 2013, at 3:41 PM, Qian Hong wrote: > Thanks Ken for many helps! You're welcome and, regarding the patch: looks good to me. -Ken

Re: opengl32: add a test for WGL_EXT_swap_control [try 2]

2013-04-29 Thread Ken Thomases
On Apr 29, 2013, at 10:42 AM, Roderick Colenbrander wrote: > Updated version of the patch which marks the failing test as todo_wine. The > final goal is to make the swap interval in the winex11 driver not global > anymore (and store it with the drawable) and add some new functionality. > <0001-ope

Re: Clang static analyzer results / wine-1.5.28-66-g6899279

2013-04-24 Thread Ken Thomases
On Apr 24, 2013, at 5:10 AM, Jacek Caban wrote: > On 04/24/13 11:35, Frédéric Delanoy wrote: > >> Although, one can sort/filter by file, or if it's really that >> annoying, a quick script removing elements containing "/tests" >> should be easy enough. > > Well... that's far from perfect, but co

Re: [PATCH 2/2] gdi32: Clip font glyphs to fit within text metrics.

2013-04-23 Thread Ken Thomases
On Apr 23, 2013, at 2:45 AM, Sam Edwards wrote: > On 04/22/2013 07:08 PM, Ken Thomases wrote: >> I can't speak to how Windows would render it, but one such font is Zapfino. >> It's an "exuberant" calligraphic font with lots of flourishes, some of which >

Re: [PATCH 2/2] gdi32: Clip font glyphs to fit within text metrics.

2013-04-22 Thread Ken Thomases
On Apr 22, 2013, at 6:40 PM, Sam Edwards wrote: > On 04/19/2013 10:32 AM, Max TenEyck Woodbury wrote: >> As I understand it, some fonts deliberately have glyphs larger than >> their metrics bounding boxes. Clipping them is almost certainly not a >> good idea. > > Forgive my disbelief, but can yo

Re: kernel32/path: Potential off-by-one error, help needed!

2013-04-22 Thread Ken Thomases
On Apr 22, 2013, at 1:17 PM, Kirill Smirnov wrote: >While debugging bug #33307 (http://bugs.winehq.org/show_bug.cgi?id=33307) > I found a suspicious piece of code, looking like off-by-one error. > >Unfortunately, I'm not familiar with this part of wine and I don't know > how to write co

Re: winex11.drv: Transmit WM_DEADCHAR messages to applications

2013-04-16 Thread Ken Thomases
On Apr 16, 2013, at 3:43 AM, Simon Lipp wrote: >> That's how it's supposed to work. > > But it doesn’t work that way under Windows. Shouldn’t Wine try to stick > to Windows behavior as close as possible ? > >> Wine should never generate WM_DEADCHAR message, it relies in the >> driver to handle

Re: secur32: Take schannel backend capabilities into account when configuring enabled protocols.

2013-04-11 Thread Ken Thomases
On Apr 11, 2013, at 8:49 AM, Jacek Caban wrote: > On 04/10/13 16:16, Ken Thomases wrote: >> However, Apple's guidance on using weak linking says that you must >> explicitly compare against NULL. They don't quite say that testing the >> symbol as a standalone boo

Re: secur32: Take schannel backend capabilities into account when configuring enabled protocols.

2013-04-10 Thread Ken Thomases
Hi Jacek, On Apr 10, 2013, at 4:24 AM, Jacek Caban wrote: > On 3/28/13 8:31 PM, Ken Thomases wrote: >> Mac OS X 10.8 introduced support for TLS 1.1 and 1.2. > > Can someone with Mac OS X 10.8 test the attached patch for me, please. All I > need is to verify that it compile

Re: winehtml5.drv: Added new HTML5 driver.

2013-04-01 Thread Ken Thomases
On Apr 1, 2013, at 6:41 AM, Jacek Caban wrote: > There are still some missing features. Most notably, it lacks support > for system tray baloons. I know; that's, like, the hardest thing to do! Great work, though! Actually, since HTML is available everywhere, there's little reason for me to con

Re: secur32: Take schannel backend capabilities into account when configuring enabled protocols.

2013-03-28 Thread Ken Thomases
On Mar 28, 2013, at 6:05 AM, Jacek Caban wrote: > --- a/dlls/secur32/schannel_macosx.c > +++ b/dlls/secur32/schannel_macosx.c > @@ -630,6 +630,11 @@ static OSStatus schan_push_adapter(SSLConnectionRef > transport, const void *buff, > return ret; > } > > +DWORD schan_imp_enabled_protocols(

Re: winemac: Track drawn surface rect to reduce black flicker for new or resized windows.

2013-03-28 Thread Ken Thomases
On Mar 27, 2013, at 3:31 PM, Ken Thomases wrote: > dlls/winemac.drv/macdrv.h | 12 +++- > dlls/winemac.drv/surface.c | 16 ++-- > dlls/winemac.drv/window.c | 17 ++--- > 3 files changed, 27 insertions(+), 18 deletions(-) > > <0001-winema

Re: winemac: Null-terminate an snprintf-ed key name buffer in case of short buffer.

2013-03-28 Thread Ken Thomases
On Mar 28, 2013, at 2:26 AM, Per Johansson wrote: > On Wed, Mar 27, 2013 at 7:22 PM, Ken Thomases wrote: >> --- > > This has probably been discussed before, but shouldn't snprintfW be > safe against this kind of thing, the way POSIX snprintf is? I actually checked th

Re: [1/3] kernel32: Set GetLargestConsoleWindowSize based on screen resolution

2013-03-27 Thread Ken Thomases
On Mar 27, 2013, at 6:08 PM, Hugh McMaster wrote: > Ken Thomases wrote: > >>> This approach is untenable. Kernel32 can't be made to depend on X11. It >>> has to still work when X11 isn't available. Also, X11 is just one possible >>> graphics/wind

Re: [1/3] kernel32: Set GetLargestConsoleWindowSize based on screen resolution

2013-03-27 Thread Ken Thomases
On Mar 27, 2013, at 6:41 AM, Hugh McMaster wrote: > This patch modifies the dlls/kernel32 Makefile.in to (1) compile a new source > file (screensize.c) (patch 3/3) and (2) add -lX11 to the ExtraDllFlags line. This approach is untenable. Kernel32 can't be made to depend on X11. It has to still

Re: explorer: Create desktop window hidden and only show it if it wins the race.

2013-03-26 Thread Ken Thomases
On Mar 26, 2013, at 3:20 AM, Dmitry Timoshkov wrote: > Ken Thomases wrote: > >> +ShowWindow( hwnd, SW_SHOW ); >> SetWindowLongPtrW( hwnd, GWLP_WNDPROC, (LONG_PTR)desktop_wnd_proc ); >> SendMessageW( hwnd, WM_SETICON, ICON_BIG, (LPARAM)LoadIconW(

Mac driver screen flicker on launch (Was: Re: [1/3] Make mac driver the default on OS X)

2013-03-26 Thread Ken Thomases
On Mar 8, 2013, at 6:20 AM, wrote: > I had a very annoying experience: when I started winemac the first > time, the whole screen turned black for a fraction of a second. That > was very unwelcome and reminded me of the infamous Intel-XOrg screen > flickering issue that plagued many Linux distr

Re: Linker error when improving GetLargestConsoleWindowSize

2013-03-25 Thread Ken Thomases
On Mar 25, 2013, at 8:51 PM, Hugh McMaster wrote: > I've written a function to determine the largest possible screen buffer that > wineconsole is able to display. This function replaces the hard-coded > constants listed in both instances of GetLargestConsoleWindowSize in > dlls/kernel32/console

Re: winemac: Calculate surface rect correctly in macdrv_UpdateLayeredWindow().

2013-03-22 Thread Ken Thomases
On Mar 22, 2013, at 1:21 AM, Ken Thomases wrote: > --- > dlls/winemac.drv/window.c | 33 - > 1 files changed, 16 insertions(+), 17 deletions(-) > > <0001-winemac-Calculate-surface-rect-correctly-in-macdrv_U.patch> Hmm. On further consid

Re: [PATCH 4/5] winemac: Implement rudimentary support for system tray icons as Mac status items.

2013-03-21 Thread Ken Thomases
On Mar 21, 2013, at 3:41 PM, C.W. Betts wrote: > On Mar 18, 2013, at 3:22 PM, Ken Thomases wrote: > >> On Mar 18, 2013, at 4:04 PM, Charles Davis wrote: >> >>> At this point, though, I'm wondering if it wouldn't just be easier to have >>> Exp

Re: winemac: Implement GetDeviceGammaRamp() and SetDeviceGammaRamp().

2013-03-20 Thread Ken Thomases
On Mar 20, 2013, at 1:33 PM, André Hentschel wrote: > Am 18.03.2013 04:41, schrieb Ken Thomases: >> --- >> dlls/winemac.drv/display.c | 131 >> >> dlls/winemac.drv/gdi.c |4 +- >> dlls/winemac.drv/macdrv.h

Re: Mac driver status: ready for contributions

2013-03-19 Thread Ken Thomases
On Mar 19, 2013, at 1:31 AM, Charles Davis wrote: > I have one more of my own to add: > > * Fix crash whenever an app makes a GLU call. Ah, yes, I forgot that one. Thanks for reminding me. > I suspect this is because Mesa's GLU calls Mesa's libGL--which crashes > horribly because there's no

Re: [10/10] winemenubuilder: Put appbundle icon inside the bundle.

2013-03-18 Thread Ken Thomases
On Mar 17, 2013, at 12:18 PM, Per Johansson wrote: > +CFStringRef iconstr = CFStringCreateWithCString(NULL, icon, > CFStringGetSystemEncoding()); Another place to use CFStringCreateWithFileSystemRepresentation(). -Ken

Re: [7/10] winemenubuilder: Create a basic Info.plist.

2013-03-18 Thread Ken Thomases
On Mar 17, 2013, at 12:18 PM, Per Johansson wrote: > +namestr = CFStringCreateWithCString(NULL, link_name, > CFStringGetSystemEncoding()); You should use CFStringCreateWithFileSystemRepresentation() to create CFStrings from POSIX path strings. > +pathstr = CFStringCreateWithCString(NU

Re: [PATCH 4/5] winemac: Implement rudimentary support for system tray icons as Mac status items.

2013-03-18 Thread Ken Thomases
On Mar 18, 2013, at 4:04 PM, Charles Davis wrote: > On Mar 18, 2013, at 2:23 PM, Ken Thomases wrote: > >> On Mar 18, 2013, at 2:15 PM, Ken Thomases wrote: >> >>> On Mar 18, 2013, at 3:03 PM, C.W. Betts wrote: >>> >>>> On Mar 17, 2013, at 9

Mac driver status: ready for contributions

2013-03-18 Thread Ken Thomases
Hi, With my latest set of patches, the Mac driver in Wine is up to speed with what's in CrossOver 12 (plus some extra goodies). Of course, that's not to say that the Mac driver is done, but if you were holding off getting involved because you figured big chunks of already-done work were pendin

Re: [PATCH 4/5] winemac: Implement rudimentary support for system tray icons as Mac status items.

2013-03-18 Thread Ken Thomases
On Mar 18, 2013, at 3:15 PM, Ken Thomases wrote: > On Mar 18, 2013, at 3:03 PM, C.W. Betts wrote: > >> On Mar 17, 2013, at 9:40 PM, Ken Thomases wrote: >> >>> Doesn't support right-clicks, mouse moves, or notification balloons. >> Notification ball

Re: [PATCH 4/5] winemac: Implement rudimentary support for system tray icons as Mac status items.

2013-03-18 Thread Ken Thomases
On Mar 18, 2013, at 3:03 PM, C.W. Betts wrote: > On Mar 17, 2013, at 9:40 PM, Ken Thomases wrote: > >> Doesn't support right-clicks, mouse moves, or notification balloons. > Notification balloons can probably be done using either Notification Center > or Growl Yeah,

Re: winemac: fix compilation on Lion and later

2013-03-07 Thread Ken Thomases
On Mar 7, 2013, at 4:15 PM, C.W. Betts wrote: > This patch defines __gltypes_h_ so that the Mac header doesn't get included. Thanks! -Ken

Re: [1/3] Make mac driver the default on OS X

2013-03-07 Thread Ken Thomases
On Mar 7, 2013, at 5:12 PM, Charles Davis wrote: > In the case that the window doesn't have its own grow box, though, it can be > hard to tell that you actually still can resize the window by dragging the > corner. (In notepad's case, it's weird, because I can click and drag the Down > button o

Re: [1/3] Make mac driver the default on OS X

2013-03-07 Thread Ken Thomases
On Mar 7, 2013, at 12:03 PM, Charles Davis wrote: > On Mar 6, 2013, at 7:56 AM, Ken Thomases wrote: > >> On Mar 6, 2013, at 8:24 AM, >> wrote: >> >>> Bug report: >>> Toying around with built-in notepad, clock, winhlp32, I noticed that >>>

Re: [PATCH 1/7] winemac: Implement OpenGL support.

2013-03-07 Thread Ken Thomases
On Mar 7, 2013, at 9:39 AM, C.W. Betts wrote: > It looks like if GL_ARB_shader_objects is defined, then the OS X GLhandleARB > isn't typedef'd. I think a better approach is to define __gltypes_h_ just to no-op that whole header. Does that fix compilation for you? If so, please submit it as a

Re: [PATCH 1/7] winemac: Implement OpenGL support.

2013-03-06 Thread Ken Thomases
On Mar 6, 2013, at 1:58 PM, C.W. Betts wrote: > It seems like there's conflicting types for GLhandleARB, one defined by Wine, > the other by OS X, on Mountain Lion. I assume that the #define __gl_h_ is a > try to work around the issue, but OS X's gltypes.h still gets included. Ugh. Thanks for

Re: [1/3] Make mac driver the default on OS X

2013-03-06 Thread Ken Thomases
On Mar 6, 2013, at 8:24 AM, wrote: > Bug report: > Toying around with built-in notepad, clock, winhlp32, I noticed that > notepad and winhlp32 are not resizable, whereas the system > preferences "control" is. By comparison, with the x11 driver, > notepad's window is resizable even though it ha

Re: [3/3] Make mac driver the default on OS X

2013-03-05 Thread Ken Thomases
On Mar 5, 2013, at 6:32 PM, Josh DuBois wrote: > diff --git a/dlls/winemac.drv/macdrv_main.c b/dlls/winemac.drv/macdrv_main.c > index cd1cc0c..1f16694 100644 > --- a/dlls/winemac.drv/macdrv_main.c > +++ b/dlls/winemac.drv/macdrv_main.c > @@ -85,7 +85,7 @@ static BOOL process_attach(void) > ma

Re: [1/3] Make mac driver the default on OS X

2013-03-05 Thread Ken Thomases
On Mar 5, 2013, at 6:32 PM, Josh DuBois wrote: > --- > dlls/gdi32/driver.c | 102 +++-- > dlls/gdi32/gdi32.spec |1 + > dlls/gdi32/gdi_private.h |2 + > dlls/user32/driver.c | 27 ++-- > include/wine/gdi_driver.h | 11 + >

Re: Making wine default to the Mac Driver on OS X

2013-03-04 Thread Ken Thomases
My feeling (like Josh's and Per's) is that the Mac driver should be what you get by default, in most cases, without taking any special steps. If the concern is what you get when you're ssh'd into a remote system, then it's probably possible for the Mac driver to detect when it doesn't have GUI

Re: [PATCH] ntdll: Support a protection scheme which reset the GS selector - v3

2013-02-17 Thread Ken Thomases
On Feb 17, 2013, at 4:18 AM, Alessandro Pignotti wrote: > Third attempt > > > > Hi everyone, > > I've found two different games: > > -) Of orcs and men > -) The testament of sherlock holmes > > which are using a protection scheme which reset the GS segment selector, > possibl

Re: Keynote made me remember Mac Icons

2013-02-03 Thread Ken Thomases
the >> default terminal icon. I also had a winecfg icon somewhere but I can't >> remember where that one is right now. >> >> Let me know and tell me where to send them. > > Ken Thomases (cc'ed) is the person on winemac.drv, though wine-devel > or attaching th

Re: macdrv: implement getting and setting the screen saver state, version 6

2013-01-22 Thread Ken Thomases
On Jan 22, 2013, at 5:33 PM, C.W. Betts wrote: > Yet another fix. Looks good to me. Thanks for doing this and for putting up with me. Cheers, Ken

Re: macdrv: implement getting and setting the screen saver state. if(Style)

2013-01-22 Thread Ken Thomases
On Jan 22, 2013, at 6:38 AM, wrote: > Ken Thomases wrote: >>> +if(success != kIOReturnSuccess) >> Another style nitpick: please put a space between "if" and the condition. >> That applies to the "if(count2)" above, too. > >

Re: macdrv: implement getting and setting the screen saver state, version 5

2013-01-22 Thread Ken Thomases
On Jan 22, 2013, at 12:22 PM, C.W. Betts wrote: > This version should fix the problems you mentioned Sorry about this – I should have thought about this last time – but I think you shouldn't fall through in the SPI_SETSCREENSAVEACTIVE case. It's asking for trouble in the future, if anybody add

Re: macdrv: implement getting and setting the screen saver state, version 4

2013-01-21 Thread Ken Thomases
Hi, On Jan 21, 2013, at 6:21 PM, C.W. Betts wrote: > This version falls through in SPI_SETSCREENSAVEACTIVE. Sadly, you used C++ style for the fall-through comment. :-/ Also, you have some trailing whitespace and the patch no longer applies cleanly after today's commits of mine. It should be e

Re: macdrv: implement getting and setting the screen saver state, version 3

2013-01-21 Thread Ken Thomases
On Jan 21, 2013, at 11:50 AM, C.W. Betts wrote: > This version gets rid of redundant comments. According to Alexandre's comment on your X11 driver patch, you should be returning FALSE from the "SET" case so that user32 will update the registry. Otherwise, looking good. -Ken

Re: macdrv: implement getting and setting the screen saver state, version 2

2013-01-21 Thread Ken Thomases
On Jan 21, 2013, at 9:31 AM, C.W. Betts wrote: > On Jan 20, 2013, at 11:06 PM, Ken Thomases wrote: > >> I'm not sure it's correct that these are two different names for roughly the >> same thing. I think the two assertion types do slightly different things.

Re: macdrv: implement getting and setting the screen saver state, version 2

2013-01-20 Thread Ken Thomases
Hi, On Jan 20, 2013, at 2:00 PM, C.W. Betts wrote: > This version implements changes and advice from Ken Thomases, including using > kIOPMAssertionTypePreventUserIdleDisplaySleep on Lion and later. Also, some > comments were added. > +//Get pre-Lion no display

Re: macdrv: implement getting and setting the screen saver state.

2013-01-20 Thread Ken Thomases
On Jan 20, 2013, at 11:47 AM, C.W. Betts wrote: > On Jan 20, 2013, at 12:38 AM, Ken Thomases wrote: > >> On Jan 19, 2013, at 4:08 PM, C.W. Betts wrote: >> >>> +CFNumberRef count = >>> CFDictionaryGetValue(assertsionSta

Re: macdrv: implement getting and setting the screen saver state.

2013-01-19 Thread Ken Thomases
Hi, On Jan 19, 2013, at 4:08 PM, C.W. Betts wrote: > This implements getting and setting the screen saver state on the Mac Wine > driver. Thanks for your contribution to the Mac driver. There are some issues with the patch: > +CFDictionaryRef assertsionStats = NULL; That va

Re: libwine: Use rpath-based install name and library references for libwine on Mac.

2013-01-12 Thread Ken Thomases
On Jan 12, 2013, at 4:12 AM, Alexandre Julliard wrote: > Ken Thomases writes: > >> On Jan 11, 2013, at 11:13 AM, Alexandre Julliard wrote: >>> It doesn't seem to work when installed: >>> >>> wine: failed to initialize: dlopen(/usr/local/lib/wine/nt

Re: libwine: Use rpath-based install name and library references for libwine on Mac.

2013-01-11 Thread Ken Thomases
On Jan 11, 2013, at 11:13 AM, Alexandre Julliard wrote: > Ken Thomases writes: > >> --- >> configure.ac |3 ++- >> libs/wine/Makefile.in |6 +++--- >> 2 files changed, 5 insertions(+), 4 deletions(-) > > It doesn't seem to work when i

Re: [PATCH 3/6] loader: On Mac, embed Info.plist in (__TEXT, __info_plist) section.

2013-01-07 Thread Ken Thomases
On Jan 7, 2013, at 1:23 AM, Per Johansson wrote: > On Mon, Jan 7, 2013 at 4:48 AM, Charles Davis wrote: >> >> With this, you won't be able to launch Wine from the Finder or with open(1) >> prior to 10.6. > > I don't see how you'd do use open or the Finder to launch wine, could > you explain mo

Re: [PATCH 3/8] winemac: Implement GetDeviceCaps().

2012-12-17 Thread Ken Thomases
On Dec 17, 2012, at 1:35 PM, Alexandre Julliard wrote: > Ken Thomases writes: > >> On Dec 17, 2012, at 11:53 AM, Alexandre Julliard wrote: >> >>> Ken Thomases writes: >>> >>>> Oh, right. The Mac driver requires 10.6 or later. So, I'

Re: [PATCH 3/8] winemac: Implement GetDeviceCaps().

2012-12-17 Thread Ken Thomases
On Dec 17, 2012, at 11:53 AM, Alexandre Julliard wrote: > Ken Thomases writes: > >> Oh, right. The Mac driver requires 10.6 or later. So, I'll have to change >> the configure script to only enable building of it on such systems. > > Is there really a compelling

  1   2   3   >