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

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

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 | 1 + dlls/winemac.drv

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

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

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 and

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. Cocoa makes use of the size limits spontaneously

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 k...@codeweavers.com writes: --- dlls/user32/driver.c | 12 dlls/user32/user_private.h |1 + dlls/user32/winpos.c |2 ++ 3 files changed, 15 insertions(+), 0 deletions(-) It's better

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 k...@codeweavers.com 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 all

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 k...@codeweavers.com 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 all

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 k...@codeweavers.com 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 some

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 out

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 - 3],

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 WCHAR *src; +unsigned long data_len; +unsigned long

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: 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 I

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. I have to wonder if there was a good reason Ken didn't use

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: 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 Mac

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 k...@codeweavers.com 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 CTFontDescriptor isn't toll-free

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 k...@codeweavers.com 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 acceptable to load the graphics driver

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 loaded even

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 to me. Actually, on further thought

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 protects against

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: 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 k...@codeweavers.com 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 the array

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 k...@codeweavers.com 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 still a chance that it's something

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 k...@codeweavers.com wrote: Instead of enabling the trace, try replacing it with something like: char dummy[256]; memset(dummy, 0x55, sizeof(dummy)); I bet the crash will change

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: [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 files

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: Need help with debugging a directx9 game crashing

2013-07-21 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 on

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:

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 bug opened in Crossover tracker: http

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

2013-06-18 Thread Ken Thomases
On Jun 18, 2013, at 1:26 AM, Dmitry Timoshkov wrote: Daniel Jeliński djelins...@gmail.com 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

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 etc)

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 k...@codeweavers.com 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 'make test' in dlls/user32/tests

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 | 5

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 | 6

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: [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 k...@codeweavers.com 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_CANCELMODE because

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

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, although clicking

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

2013-05-12 Thread Ken Thomases
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: The fullscreen button appears if a main window

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

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

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 a

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.

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: 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 tr elements containing /tests should be easy enough. Well... that's far from perfect, but could work.

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 have strokes extending into the line above

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

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 you

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 dead keys

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 compiles and when running dlls

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

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 k...@codeweavers.com 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 the man page

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-winemac-Track-drawn-surface-rect

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(void)

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: [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/windowing system that Wine can use, so it's not always

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, joerg-cyril.hoe...@t-systems.com joerg-cyril.hoe...@t-systems.com 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

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 k...@codeweavers.com wrote: +ShowWindow( hwnd, SW_SHOW ); SetWindowLongPtrW( hwnd, GWLP_WNDPROC, (LONG_PTR)desktop_wnd_proc ); SendMessageW( hwnd, WM_SETICON, ICON_BIG, (LPARAM)LoadIconW( 0

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.c.

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

2013-03-22 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 k...@codeweavers.com 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 Explorer draw the balloons itself à la Windows

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 consideration, I'm not sure

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 |2 + 3 files changed, 135 insertions(+), 2

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: [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 k...@codeweavers.com wrote: Doesn't support right-clicks, mouse moves, or notification balloons. Notification balloons can probably be done using either Notification Center or Growl Yeah, I

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 k...@codeweavers.com wrote: Doesn't support right-clicks, mouse moves, or notification balloons. Notification balloons can probably be done using

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

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:40 PM, Ken Thomases k...@codeweavers.com wrote: Doesn't support

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 =

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: [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: [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, joerg-cyril.hoe...@t-systems.com joerg-cyril.hoe...@t-systems.com wrote: Bug report: Toying around with built-in notepad, clock, winhlp32, I noticed

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 on

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-06 Thread Ken Thomases
On Mar 6, 2013, at 8:24 AM, joerg-cyril.hoe...@t-systems.com joerg-cyril.hoe...@t-systems.com 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

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-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 + 5

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)

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, possibly to confuse

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 them to bugzilla would likely be best to preserve

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

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, joerg-cyril.hoe...@t-systems.com joerg-cyril.hoe...@t-systems.com 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 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, 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 k...@codeweavers.com 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. NoDisplaySleep

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 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

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 k...@codeweavers.com wrote: On Jan 19, 2013, at 4:08 PM, C.W. Betts wrote: +CFNumberRef count = CFDictionaryGetValue(assertsionStats, kIOPMAssertionTypeNoDisplaySleep

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 sleep counts C

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

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 k...@codeweavers.com 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/ntdll.dll.so, 258): image not found I

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 k...@codeweavers.com writes: --- configure.ac |3 ++- libs/wine/Makefile.in |6 +++--- 2 files changed, 5 insertions(+), 4 deletions(-) It doesn't seem to work when installed: wine: failed

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 cdavi...@gmail.com 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

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

2012-12-17 Thread Ken Thomases
On Dec 17, 2012, at 8:06 AM, Alexandre Julliard wrote: Ken Thomases k...@codeweavers.com writes: --- dlls/winemac.drv/Makefile.in |1 + dlls/winemac.drv/gdi.c | 321 - dlls/winemac.drv/macdrv.h| 101 + 3 files changed, 415

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 k...@codeweavers.com 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 reason

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 k...@codeweavers.com writes: On Dec 17, 2012, at 11:53 AM, Alexandre Julliard wrote: Ken Thomases k...@codeweavers.com writes: Oh, right. The Mac driver requires 10.6 or later. So, I'll have to change the configure

  1   2   3   >