January 2013 - Wine-commits

Ken Thomases : winemac: Implement a MOUSE_BUTTON event for mouse clicks.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: bd269786a505931c82f7fe93c6cc6d76102daac1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bd269786a505931c82f7fe93c… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Jan 27 16:19:37 2013 -0600 winemac: Implement a MOUSE_BUTTON event for mouse clicks. --- dlls/winemac.drv/Makefile.in | 1 + dlls/winemac.drv/cocoa_window.m | 35 +++++++++++ dlls/winemac.drv/event.c | 7 ++ dlls/winemac.drv/macdrv.h | 2 + dlls/winemac.drv/macdrv_cocoa.h | 8 +++ dlls/winemac.drv/mouse.c | 122 +++++++++++++++++++++++++++++++++++++++ 6 files changed, 175 insertions(+), 0 deletions(-) diff --git a/dlls/winemac.drv/Makefile.in b/dlls/winemac.drv/Makefile.in index 1de19ef..8484db8 100644 --- a/dlls/winemac.drv/Makefile.in +++ b/dlls/winemac.drv/Makefile.in @@ -7,6 +7,7 @@ C_SRCS = \ event.c \ gdi.c \ macdrv_main.c \ + mouse.c \ surface.c \ window.c diff --git a/dlls/winemac.drv/cocoa_window.m b/dlls/winemac.drv/cocoa_window.m index c05adae..402db30 100644 --- a/dlls/winemac.drv/cocoa_window.m +++ b/dlls/winemac.drv/cocoa_window.m @@ -158,6 +158,13 @@ static BOOL frame_intersects_screens(NSRect frame, NSArray* screens) } } + /* By default, NSView will swallow right-clicks in an attempt to support contextual + menus. We need to bypass that and allow the event to make it to the window. */ + - (void) rightMouseDown:(NSEvent*)theEvent + { + [[self window] rightMouseDown:theEvent]; + } + @end @@ -417,6 +424,22 @@ static BOOL frame_intersects_screens(NSRect frame, NSArray* screens) [self checkTransparency]; } + - (void) postMouseButtonEvent:(NSEvent *)theEvent pressed:(int)pressed + { + CGPoint pt = CGEventGetLocation([theEvent CGEvent]); + macdrv_event event; + + event.type = MOUSE_BUTTON; + event.window = (macdrv_window)[self retain]; + event.mouse_button.button = [theEvent buttonNumber]; + event.mouse_button.pressed = pressed; + event.mouse_button.x = pt.x; + event.mouse_button.y = pt.y; + event.mouse_button.time_ms = [NSApp ticksForEventTime:[theEvent timestamp]]; + + [queue postEvent:&event]; + } + /* * ---------- NSWindow method overrides ---------- @@ -446,6 +469,18 @@ static BOOL frame_intersects_screens(NSRect frame, NSArray* screens) /* + * ---------- NSResponder method overrides ---------- + */ + - (void) mouseDown:(NSEvent *)theEvent { [self postMouseButtonEvent:theEvent pressed:1]; } + - (void) rightMouseDown:(NSEvent *)theEvent { [self mouseDown:theEvent]; } + - (void) otherMouseDown:(NSEvent *)theEvent { [self mouseDown:theEvent]; } + + - (void) mouseUp:(NSEvent *)theEvent { [self postMouseButtonEvent:theEvent pressed:0]; } + - (void) rightMouseUp:(NSEvent *)theEvent { [self mouseUp:theEvent]; } + - (void) otherMouseUp:(NSEvent *)theEvent { [self mouseUp:theEvent]; } + + + /* * ---------- NSWindowDelegate methods ---------- */ - (void)windowDidMove:(NSNotification *)notification diff --git a/dlls/winemac.drv/event.c b/dlls/winemac.drv/event.c index da41884..4d35082 100644 --- a/dlls/winemac.drv/event.c +++ b/dlls/winemac.drv/event.c @@ -32,6 +32,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(event); static const char *dbgstr_event(int type) { static const char * const event_names[] = { + "MOUSE_BUTTON", "WINDOW_CLOSE_REQUESTED", "WINDOW_FRAME_CHANGED", }; @@ -50,6 +51,9 @@ static macdrv_event_mask get_event_mask(DWORD mask) if ((mask & QS_ALLINPUT) == QS_ALLINPUT) return -1; + if (mask & QS_MOUSEBUTTON) + event_mask |= event_mask_for_type(MOUSE_BUTTON); + if (mask & QS_POSTMESSAGE) { event_mask |= event_mask_for_type(WINDOW_CLOSE_REQUESTED); @@ -77,6 +81,9 @@ void macdrv_handle_event(macdrv_event *event) switch (event->type) { + case MOUSE_BUTTON: + macdrv_mouse_button(hwnd, event); + break; case WINDOW_CLOSE_REQUESTED: macdrv_window_close_requested(hwnd); break; diff --git a/dlls/winemac.drv/macdrv.h b/dlls/winemac.drv/macdrv.h index c40fb86..378b4ab 100644 --- a/dlls/winemac.drv/macdrv.h +++ b/dlls/winemac.drv/macdrv.h @@ -120,4 +120,6 @@ extern void set_surface_use_alpha(struct window_surface *window_surface, BOOL us extern void macdrv_window_close_requested(HWND hwnd) DECLSPEC_HIDDEN; extern void macdrv_window_frame_changed(HWND hwnd, CGRect frame) DECLSPEC_HIDDEN; +extern void macdrv_mouse_button(HWND hwnd, const macdrv_event *event) DECLSPEC_HIDDEN; + #endif /* __WINE_MACDRV_H */ diff --git a/dlls/winemac.drv/macdrv_cocoa.h b/dlls/winemac.drv/macdrv_cocoa.h index 70be367..4827d40 100644 --- a/dlls/winemac.drv/macdrv_cocoa.h +++ b/dlls/winemac.drv/macdrv_cocoa.h @@ -122,6 +122,7 @@ extern void macdrv_free_displays(struct macdrv_display* displays) DECLSPEC_HIDDE /* event */ enum { + MOUSE_BUTTON, WINDOW_CLOSE_REQUESTED, WINDOW_FRAME_CHANGED, NUM_EVENT_TYPES @@ -134,6 +135,13 @@ typedef struct macdrv_event { macdrv_window window; union { struct { + int button; + int pressed; + int x; + int y; + unsigned long time_ms; + } mouse_button; + struct { CGRect frame; } window_frame_changed; }; diff --git a/dlls/winemac.drv/mouse.c b/dlls/winemac.drv/mouse.c new file mode 100644 index 0000000..31d3db0 --- /dev/null +++ b/dlls/winemac.drv/mouse.c @@ -0,0 +1,122 @@ +/* + * MACDRV mouse driver + * + * Copyright 1998 Ulrich Weigand + * Copyright 2007 Henri Verbeet + * Copyright 2011, 2012, 2013 Ken Thomases for CodeWeavers Inc. + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include "config.h" + +#include "macdrv.h" +#include "winuser.h" +#include "wine/server.h" + +WINE_DEFAULT_DEBUG_CHANNEL(cursor); + + +/*********************************************************************** + * send_mouse_input + * + * Update the various window states on a mouse event. + */ +static void send_mouse_input(HWND hwnd, UINT flags, int x, int y, + DWORD mouse_data, unsigned long time) +{ + INPUT input; + HWND top_level_hwnd; + + top_level_hwnd = GetAncestor(hwnd, GA_ROOT); + + if ((flags & MOUSEEVENTF_MOVE) && (flags & MOUSEEVENTF_ABSOLUTE)) + { + RECT rect; + + /* update the wine server Z-order */ + SetRect(&rect, x, y, x + 1, y + 1); + MapWindowPoints(0, top_level_hwnd, (POINT *)&rect, 2); + + SERVER_START_REQ(update_window_zorder) + { + req->window = wine_server_user_handle(top_level_hwnd); + req->rect.left = rect.left; + req->rect.top = rect.top; + req->rect.right = rect.right; + req->rect.bottom = rect.bottom; + wine_server_call(req); + } + SERVER_END_REQ; + } + + input.type = INPUT_MOUSE; + input.mi.dx = x; + input.mi.dy = y; + input.mi.mouseData = mouse_data; + input.mi.dwFlags = flags; + input.mi.time = time; + input.mi.dwExtraInfo = 0; + + __wine_send_input(top_level_hwnd, &input); +} + + +/*********************************************************************** + * macdrv_mouse_button + * + * Handler for MOUSE_BUTTON events. + */ +void macdrv_mouse_button(HWND hwnd, const macdrv_event *event) +{ + UINT flags = 0; + WORD data = 0; + + TRACE("win %p button %d %s at (%d,%d) time %lu (%lu ticks ago)\n", hwnd, event->mouse_button.button, + (event->mouse_button.pressed ? "pressed" : "released"), + event->mouse_button.x, event->mouse_button.y, + event->mouse_button.time_ms, (GetTickCount() - event->mouse_button.time_ms)); + + if (event->mouse_button.pressed) + { + switch (event->mouse_button.button) + { + case 0: flags |= MOUSEEVENTF_LEFTDOWN; break; + case 1: flags |= MOUSEEVENTF_RIGHTDOWN; break; + case 2: flags |= MOUSEEVENTF_MIDDLEDOWN; break; + default: + flags |= MOUSEEVENTF_XDOWN; + data = 1 << (event->mouse_button.button - 3); + break; + } + } + else + { + switch (event->mouse_button.button) + { + case 0: flags |= MOUSEEVENTF_LEFTUP; break; + case 1: flags |= MOUSEEVENTF_RIGHTUP; break; + case 2: flags |= MOUSEEVENTF_MIDDLEUP; break; + default: + flags |= MOUSEEVENTF_XUP; + data = 1 << (event->mouse_button.button - 3); + break; + } + } + + send_mouse_input(hwnd, flags | MOUSEEVENTF_ABSOLUTE | MOUSEEVENTF_MOVE, + event->mouse_button.x, event->mouse_button.y, + data, event->mouse_button.time_ms); +}

1 0

Ken Thomases : winemac: Take control over when a window can become focused away from Cocoa.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: 6cde62ac18c9845c6f339b273a821135f7bf6794 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6cde62ac18c9845c6f339b273… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Jan 27 16:19:41 2013 -0600 winemac: Take control over when a window can become focused away from Cocoa. --- dlls/winemac.drv/cocoa_window.h | 2 + dlls/winemac.drv/cocoa_window.m | 56 ++++++++++++++++++++++++++++++++++++- dlls/winemac.drv/macdrv_cocoa.h | 1 + dlls/winemac.drv/window.c | 34 ++++++++++++++++++++++ dlls/winemac.drv/winemac.drv.spec | 1 + 5 files changed, 93 insertions(+), 1 deletions(-) diff --git a/dlls/winemac.drv/cocoa_window.h b/dlls/winemac.drv/cocoa_window.h index 1f81cc6..7708fa6 100644 --- a/dlls/winemac.drv/cocoa_window.h +++ b/dlls/winemac.drv/cocoa_window.h @@ -45,6 +45,8 @@ CGFloat colorKeyRed, colorKeyGreen, colorKeyBlue; BOOL usePerPixelAlpha; + + BOOL causing_becomeKeyWindow; } @end diff --git a/dlls/winemac.drv/cocoa_window.m b/dlls/winemac.drv/cocoa_window.m index 402db30..c1f90d2 100644 --- a/dlls/winemac.drv/cocoa_window.m +++ b/dlls/winemac.drv/cocoa_window.m @@ -440,14 +440,52 @@ static BOOL frame_intersects_screens(NSRect frame, NSArray* screens) [queue postEvent:&event]; } + - (void) makeFocused + { + NSArray* screens; + + [NSApp transformProcessToForeground]; + + /* If a borderless window is offscreen, orderFront: won't move + it onscreen like it would for a titled window. Do that ourselves. */ + screens = [NSScreen screens]; + if (!([self styleMask] & NSTitledWindowMask) && ![self isVisible] && + !frame_intersects_screens([self frame], screens)) + { + NSScreen* primaryScreen = [screens objectAtIndex:0]; + NSRect frame = [primaryScreen frame]; + [self setFrameTopLeftPoint:NSMakePoint(NSMinX(frame), NSMaxY(frame))]; + frame = [self constrainFrameRect:[self frame] toScreen:primaryScreen]; + [self setFrame:frame display:YES]; + } + + [self orderFront:nil]; + causing_becomeKeyWindow = TRUE; + [self makeKeyWindow]; + causing_becomeKeyWindow = FALSE; + if (latentParentWindow) + { + [latentParentWindow addChildWindow:self ordered:NSWindowAbove]; + self.latentParentWindow = nil; + } + if (![self isExcludedFromWindowsMenu]) + [NSApp addWindowsItem:self title:[self title] filename:NO]; + + /* Cocoa may adjust the frame when the window is ordered onto the screen. + Generate a frame-changed event just in case. The back end will ignore + it if nothing actually changed. */ + [self windowDidResize:nil]; + } + /* * ---------- NSWindow method overrides ---------- */ - (BOOL) canBecomeKeyWindow { + if (causing_becomeKeyWindow) return YES; if (self.disabled || self.noActivate) return NO; - return YES; + return [self isKeyWindow]; } - (BOOL) canBecomeMainWindow @@ -841,3 +879,19 @@ void macdrv_window_use_per_pixel_alpha(macdrv_window w, int use_per_pixel_alpha) [pool release]; } + +/*********************************************************************** + * macdrv_give_cocoa_window_focus + * + * Makes the Cocoa window "key" (gives it keyboard focus). This also + * orders it front and, if its frame was not within the desktop bounds, + * Cocoa will typically move it on-screen. + */ +void macdrv_give_cocoa_window_focus(macdrv_window w) +{ + WineWindow* window = (WineWindow*)w; + + OnMainThread(^{ + [window makeFocused]; + }); +} diff --git a/dlls/winemac.drv/macdrv_cocoa.h b/dlls/winemac.drv/macdrv_cocoa.h index 4827d40..4b7de10 100644 --- a/dlls/winemac.drv/macdrv_cocoa.h +++ b/dlls/winemac.drv/macdrv_cocoa.h @@ -205,5 +205,6 @@ extern void macdrv_set_window_color_key(macdrv_window w, CGFloat keyRed, CGFloat CGFloat keyBlue) DECLSPEC_HIDDEN; extern void macdrv_clear_window_color_key(macdrv_window w) DECLSPEC_HIDDEN; extern void macdrv_window_use_per_pixel_alpha(macdrv_window w, int use_per_pixel_alpha) DECLSPEC_HIDDEN; +extern void macdrv_give_cocoa_window_focus(macdrv_window w) DECLSPEC_HIDDEN; #endif /* __WINE_MACDRV_COCOA_H */ diff --git a/dlls/winemac.drv/window.c b/dlls/winemac.drv/window.c index a08e896..74894e9 100644 --- a/dlls/winemac.drv/window.c +++ b/dlls/winemac.drv/window.c @@ -43,6 +43,9 @@ static CRITICAL_SECTION win_data_section = { &critsect_debug, -1, 0, 0, 0, 0 }; static CFMutableDictionaryRef win_datas; +void CDECL macdrv_SetFocus(HWND hwnd); + + /*********************************************************************** * get_cocoa_window_features */ @@ -154,6 +157,12 @@ static void show_window(struct macdrv_win_data *data) TRACE("win %p/%p\n", data->hwnd, data->cocoa_window); data->on_screen = macdrv_order_cocoa_window(data->cocoa_window, NULL, NULL); + if (data->on_screen) + { + HWND hwndFocus = GetFocus(); + if (hwndFocus && (data->hwnd == hwndFocus || IsChild(data->hwnd, hwndFocus))) + macdrv_SetFocus(hwndFocus); + } } @@ -820,6 +829,31 @@ void CDECL macdrv_DestroyWindow(HWND hwnd) } +/***************************************************************** + * SetFocus (MACDRV.@) + * + * Set the Mac focus. + */ +void CDECL macdrv_SetFocus(HWND hwnd) +{ + struct macdrv_win_data *data; + + TRACE("%p\n", hwnd); + + if (!(hwnd = GetAncestor(hwnd, GA_ROOT))) return; + if (!(data = get_win_data(hwnd))) return; + + if (data->cocoa_window) + { + /* Set Mac focus */ + macdrv_give_cocoa_window_focus(data->cocoa_window); + data->on_screen = TRUE; + } + + release_win_data(data); +} + + /*********************************************************************** * SetLayeredWindowAttributes (MACDRV.@) * diff --git a/dlls/winemac.drv/winemac.drv.spec b/dlls/winemac.drv/winemac.drv.spec index 73e78ca..a0e1ab8 100644 --- a/dlls/winemac.drv/winemac.drv.spec +++ b/dlls/winemac.drv/winemac.drv.spec @@ -10,6 +10,7 @@ @ cdecl EnumDisplayMonitors(long ptr ptr long) macdrv_EnumDisplayMonitors @ cdecl GetMonitorInfo(long ptr) macdrv_GetMonitorInfo @ cdecl MsgWaitForMultipleObjectsEx(long ptr long long long) macdrv_MsgWaitForMultipleObjectsEx +@ cdecl SetFocus(long) macdrv_SetFocus @ cdecl SetLayeredWindowAttributes(long long long long) macdrv_SetLayeredWindowAttributes @ cdecl SetParent(long long long) macdrv_SetParent @ cdecl SetWindowRgn(long long long) macdrv_SetWindowRgn

1 0

Ken Thomases : winemac: Maintain a list of which windows have been "key" ( focused) recently.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: beccf0f8c0598191afc334f0a5d9d84686d67df2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=beccf0f8c0598191afc334f0a… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Jan 27 16:19:45 2013 -0600 winemac: Maintain a list of which windows have been "key" (focused) recently. --- dlls/winemac.drv/cocoa_app.h | 2 ++ dlls/winemac.drv/cocoa_app.m | 31 ++++++++++++++++++++++++++++++- 2 files changed, 32 insertions(+), 1 deletions(-) diff --git a/dlls/winemac.drv/cocoa_app.h b/dlls/winemac.drv/cocoa_app.h index 0955dea..40f9e9d 100644 --- a/dlls/winemac.drv/cocoa_app.h +++ b/dlls/winemac.drv/cocoa_app.h @@ -35,6 +35,8 @@ NSLock* eventQueuesLock; NSTimeInterval eventTimeAdjustment; + + NSMutableArray* keyWindows; } - (void) transformProcessToForeground; diff --git a/dlls/winemac.drv/cocoa_app.m b/dlls/winemac.drv/cocoa_app.m index 638e642..c69f09e 100644 --- a/dlls/winemac.drv/cocoa_app.m +++ b/dlls/winemac.drv/cocoa_app.m @@ -34,7 +34,9 @@ int macdrv_err_on; eventQueues = [[NSMutableArray alloc] init]; eventQueuesLock = [[NSLock alloc] init]; - if (!eventQueues || !eventQueuesLock) + keyWindows = [[NSMutableArray alloc] init]; + + if (!eventQueues || !eventQueuesLock || !keyWindows) { [self release]; return nil; @@ -45,6 +47,7 @@ int macdrv_err_on; - (void) dealloc { + [keyWindows release]; [eventQueues release]; [eventQueuesLock release]; [super dealloc]; @@ -118,6 +121,32 @@ int macdrv_err_on; return (eventTime + eventTimeAdjustment) * 1000; } + + /* + * ---------- NSApplicationDelegate methods ---------- + */ + - (void)applicationWillFinishLaunching:(NSNotification *)notification + { + NSNotificationCenter* nc = [NSNotificationCenter defaultCenter]; + + [nc addObserverForName:NSWindowDidBecomeKeyNotification + object:nil + queue:nil + usingBlock:^(NSNotification *note){ + NSWindow* window = [note object]; + [keyWindows removeObjectIdenticalTo:window]; + [keyWindows insertObject:window atIndex:0]; + }]; + + [nc addObserverForName:NSWindowWillCloseNotification + object:nil + queue:[NSOperationQueue mainQueue] + usingBlock:^(NSNotification *note){ + NSWindow* window = [note object]; + [keyWindows removeObjectIdenticalTo:window]; + }]; + } + @end /***********************************************************************

1 0

Ken Thomases : winemac: Implement a WINDOW_GOT_FOCUS event for when Cocoa tries to focus a window.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: 7863a230ec8412506d3256c20a1940d9bddb9776 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7863a230ec8412506d3256c20… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Jan 27 16:19:48 2013 -0600 winemac: Implement a WINDOW_GOT_FOCUS event for when Cocoa tries to focus a window. --- dlls/winemac.drv/cocoa_app.h | 5 +++ dlls/winemac.drv/cocoa_app.m | 70 +++++++++++++++++++++++++++++++++++++++ dlls/winemac.drv/cocoa_event.m | 7 ++++ dlls/winemac.drv/cocoa_window.h | 2 + dlls/winemac.drv/cocoa_window.m | 32 +++++++++++++++++- dlls/winemac.drv/event.c | 5 +++ dlls/winemac.drv/macdrv.h | 1 + dlls/winemac.drv/macdrv_cocoa.h | 7 ++++ dlls/winemac.drv/window.c | 34 +++++++++++++++++++ 9 files changed, 162 insertions(+), 1 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7863a230ec8412506d325…

1 0

Ken Thomases : winemac: Implement a WINDOW_LOST_FOCUS event.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: 3c5973139fa61d24d1e980de48c592da1dacafd3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3c5973139fa61d24d1e980de4… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Jan 27 16:19:53 2013 -0600 winemac: Implement a WINDOW_LOST_FOCUS event. --- dlls/winemac.drv/cocoa_window.m | 11 +++++++++++ dlls/winemac.drv/event.c | 5 +++++ dlls/winemac.drv/macdrv.h | 1 + dlls/winemac.drv/macdrv_cocoa.h | 1 + dlls/winemac.drv/window.c | 16 ++++++++++++++++ 5 files changed, 34 insertions(+), 0 deletions(-) diff --git a/dlls/winemac.drv/cocoa_window.m b/dlls/winemac.drv/cocoa_window.m index a6e6856..e5e78f9 100644 --- a/dlls/winemac.drv/cocoa_window.m +++ b/dlls/winemac.drv/cocoa_window.m @@ -556,6 +556,17 @@ static BOOL frame_intersects_screens(NSRect frame, NSArray* screens) [self windowDidResize:notification]; } + - (void)windowDidResignKey:(NSNotification *)notification + { + macdrv_event event; + + if (causing_becomeKeyWindow) return; + + event.type = WINDOW_LOST_FOCUS; + event.window = (macdrv_window)[self retain]; + [queue postEvent:&event]; + } + - (void)windowDidResize:(NSNotification *)notification { macdrv_event event; diff --git a/dlls/winemac.drv/event.c b/dlls/winemac.drv/event.c index 969c5e7..634fe52 100644 --- a/dlls/winemac.drv/event.c +++ b/dlls/winemac.drv/event.c @@ -36,6 +36,7 @@ static const char *dbgstr_event(int type) "WINDOW_CLOSE_REQUESTED", "WINDOW_FRAME_CHANGED", "WINDOW_GOT_FOCUS", + "WINDOW_LOST_FOCUS", }; if (0 <= type && type < NUM_EVENT_TYPES) return event_names[type]; @@ -60,6 +61,7 @@ static macdrv_event_mask get_event_mask(DWORD mask) event_mask |= event_mask_for_type(WINDOW_CLOSE_REQUESTED); event_mask |= event_mask_for_type(WINDOW_FRAME_CHANGED); event_mask |= event_mask_for_type(WINDOW_GOT_FOCUS); + event_mask |= event_mask_for_type(WINDOW_LOST_FOCUS); } return event_mask; @@ -95,6 +97,9 @@ void macdrv_handle_event(macdrv_event *event) case WINDOW_GOT_FOCUS: macdrv_window_got_focus(hwnd, event); break; + case WINDOW_LOST_FOCUS: + macdrv_window_lost_focus(hwnd, event); + break; default: TRACE(" ignoring\n"); break; diff --git a/dlls/winemac.drv/macdrv.h b/dlls/winemac.drv/macdrv.h index 0be4b59..e6c747f 100644 --- a/dlls/winemac.drv/macdrv.h +++ b/dlls/winemac.drv/macdrv.h @@ -120,6 +120,7 @@ extern void set_surface_use_alpha(struct window_surface *window_surface, BOOL us extern void macdrv_window_close_requested(HWND hwnd) DECLSPEC_HIDDEN; extern void macdrv_window_frame_changed(HWND hwnd, CGRect frame) DECLSPEC_HIDDEN; extern void macdrv_window_got_focus(HWND hwnd, const macdrv_event *event) DECLSPEC_HIDDEN; +extern void macdrv_window_lost_focus(HWND hwnd, const macdrv_event *event) DECLSPEC_HIDDEN; extern void macdrv_mouse_button(HWND hwnd, const macdrv_event *event) DECLSPEC_HIDDEN; diff --git a/dlls/winemac.drv/macdrv_cocoa.h b/dlls/winemac.drv/macdrv_cocoa.h index 35229ba..2084d78 100644 --- a/dlls/winemac.drv/macdrv_cocoa.h +++ b/dlls/winemac.drv/macdrv_cocoa.h @@ -128,6 +128,7 @@ enum { WINDOW_CLOSE_REQUESTED, WINDOW_FRAME_CHANGED, WINDOW_GOT_FOCUS, + WINDOW_LOST_FOCUS, NUM_EVENT_TYPES }; diff --git a/dlls/winemac.drv/window.c b/dlls/winemac.drv/window.c index c87897a..5356687 100644 --- a/dlls/winemac.drv/window.c +++ b/dlls/winemac.drv/window.c @@ -1453,3 +1453,19 @@ void macdrv_window_got_focus(HWND hwnd, const macdrv_event *event) TRACE("win %p/%p rejecting focus\n", hwnd, event->window); macdrv_window_rejected_focus(event); } + + +/*********************************************************************** + * macdrv_window_lost_focus + * + * Handler for WINDOW_LOST_FOCUS events. + */ +void macdrv_window_lost_focus(HWND hwnd, const macdrv_event *event) +{ + if (!hwnd) return; + + TRACE("win %p/%p fg %p\n", hwnd, event->window, GetForegroundWindow()); + + if (hwnd == GetForegroundWindow()) + SendMessageW(hwnd, WM_CANCELMODE, 0, 0); +}

1 0

Ken Thomases : winemac: Implement an APP_DEACTIVATED event.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: b564d972294fca82f3ebb3edf4600e69adb89f75 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b564d972294fca82f3ebb3edf… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Jan 27 16:19:56 2013 -0600 winemac: Implement an APP_DEACTIVATED event. --- dlls/winemac.drv/cocoa_app.m | 11 +++++++++++ dlls/winemac.drv/event.c | 5 +++++ dlls/winemac.drv/macdrv.h | 1 + dlls/winemac.drv/macdrv_cocoa.h | 1 + dlls/winemac.drv/window.c | 15 +++++++++++++++ 5 files changed, 33 insertions(+), 0 deletions(-) diff --git a/dlls/winemac.drv/cocoa_app.m b/dlls/winemac.drv/cocoa_app.m index 54a6d50..7b12185 100644 --- a/dlls/winemac.drv/cocoa_app.m +++ b/dlls/winemac.drv/cocoa_app.m @@ -179,7 +179,18 @@ int macdrv_err_on; */ - (void)applicationDidResignActive:(NSNotification *)notification { + macdrv_event event; + WineEventQueue* queue; + [self invalidateGotFocusEvents]; + + event.type = APP_DEACTIVATED; + event.window = NULL; + + [eventQueuesLock lock]; + for (queue in eventQueues) + [queue postEvent:&event]; + [eventQueuesLock unlock]; } - (void)applicationWillFinishLaunching:(NSNotification *)notification diff --git a/dlls/winemac.drv/event.c b/dlls/winemac.drv/event.c index 634fe52..654e691 100644 --- a/dlls/winemac.drv/event.c +++ b/dlls/winemac.drv/event.c @@ -32,6 +32,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(event); static const char *dbgstr_event(int type) { static const char * const event_names[] = { + "APP_DEACTIVATED", "MOUSE_BUTTON", "WINDOW_CLOSE_REQUESTED", "WINDOW_FRAME_CHANGED", @@ -58,6 +59,7 @@ static macdrv_event_mask get_event_mask(DWORD mask) if (mask & QS_POSTMESSAGE) { + event_mask |= event_mask_for_type(APP_DEACTIVATED); event_mask |= event_mask_for_type(WINDOW_CLOSE_REQUESTED); event_mask |= event_mask_for_type(WINDOW_FRAME_CHANGED); event_mask |= event_mask_for_type(WINDOW_GOT_FOCUS); @@ -85,6 +87,9 @@ void macdrv_handle_event(macdrv_event *event) switch (event->type) { + case APP_DEACTIVATED: + macdrv_app_deactivated(); + break; case MOUSE_BUTTON: macdrv_mouse_button(hwnd, event); break; diff --git a/dlls/winemac.drv/macdrv.h b/dlls/winemac.drv/macdrv.h index e6c747f..d1b73fe 100644 --- a/dlls/winemac.drv/macdrv.h +++ b/dlls/winemac.drv/macdrv.h @@ -121,6 +121,7 @@ extern void macdrv_window_close_requested(HWND hwnd) DECLSPEC_HIDDEN; extern void macdrv_window_frame_changed(HWND hwnd, CGRect frame) DECLSPEC_HIDDEN; extern void macdrv_window_got_focus(HWND hwnd, const macdrv_event *event) DECLSPEC_HIDDEN; extern void macdrv_window_lost_focus(HWND hwnd, const macdrv_event *event) DECLSPEC_HIDDEN; +extern void macdrv_app_deactivated(void) DECLSPEC_HIDDEN; extern void macdrv_mouse_button(HWND hwnd, const macdrv_event *event) DECLSPEC_HIDDEN; diff --git a/dlls/winemac.drv/macdrv_cocoa.h b/dlls/winemac.drv/macdrv_cocoa.h index 2084d78..dcdf91d 100644 --- a/dlls/winemac.drv/macdrv_cocoa.h +++ b/dlls/winemac.drv/macdrv_cocoa.h @@ -124,6 +124,7 @@ extern void macdrv_free_displays(struct macdrv_display* displays) DECLSPEC_HIDDE /* event */ enum { + APP_DEACTIVATED, MOUSE_BUTTON, WINDOW_CLOSE_REQUESTED, WINDOW_FRAME_CHANGED, diff --git a/dlls/winemac.drv/window.c b/dlls/winemac.drv/window.c index 5356687..810d3ae 100644 --- a/dlls/winemac.drv/window.c +++ b/dlls/winemac.drv/window.c @@ -1469,3 +1469,18 @@ void macdrv_window_lost_focus(HWND hwnd, const macdrv_event *event) if (hwnd == GetForegroundWindow()) SendMessageW(hwnd, WM_CANCELMODE, 0, 0); } + + +/*********************************************************************** + * macdrv_app_deactivated + * + * Handler for APP_DEACTIVATED events. + */ +void macdrv_app_deactivated(void) +{ + if (GetActiveWindow() == GetForegroundWindow()) + { + TRACE("setting fg to desktop\n"); + SetForegroundWindow(GetDesktopWindow()); + } +}

1 0

Nikolay Sivov : xmllite: Handle E_PENDING from stream for comment nodes.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: 9d664b2217c6b2b508f15fdd1cc152912fdbc832 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9d664b2217c6b2b508f15fdd1… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Jan 27 11:17:27 2013 +0400 xmllite: Handle E_PENDING from stream for comment nodes. --- dlls/xmllite/reader.c | 89 +++++++++++++++++++++++++++++++++++------- dlls/xmllite/tests/reader.c | 92 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 167 insertions(+), 14 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=9d664b2217c6b2b508f15…

1 0

Nikolay Sivov : xmllite: Fail to set input for external IXmlReaderInput.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: 8954671cdc64a63d5e10864fd9218d907a72b4ab URL: http://source.winehq.org/git/wine.git/?a=commit;h=8954671cdc64a63d5e10864fd… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Jan 27 11:54:58 2013 +0400 xmllite: Fail to set input for external IXmlReaderInput. --- dlls/xmllite/reader.c | 31 ++++++++++++++++++++++++------- 1 files changed, 24 insertions(+), 7 deletions(-) diff --git a/dlls/xmllite/reader.c b/dlls/xmllite/reader.c index 785d9bf..b3f7194 100644 --- a/dlls/xmllite/reader.c +++ b/dlls/xmllite/reader.c @@ -114,6 +114,8 @@ typedef struct unsigned int pending : 1; } xmlreaderinput; +static const struct IUnknownVtbl xmlreaderinputvtbl; + typedef struct { WCHAR *str; @@ -1764,7 +1766,7 @@ static HRESULT WINAPI xmlreader_QueryInterface(IXmlReader *iface, REFIID riid, v { xmlreader *This = impl_from_IXmlReader(iface); - TRACE("%p %s %p\n", This, debugstr_guid(riid), ppvObject); + TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppvObject); if (IsEqualGUID(riid, &IID_IUnknown) || IsEqualGUID(riid, &IID_IXmlReader)) @@ -1774,6 +1776,7 @@ static HRESULT WINAPI xmlreader_QueryInterface(IXmlReader *iface, REFIID riid, v else { FIXME("interface %s not implemented\n", debugstr_guid(riid)); + *ppvObject = NULL; return E_NOINTERFACE; } @@ -1814,6 +1817,7 @@ static ULONG WINAPI xmlreader_Release(IXmlReader *iface) static HRESULT WINAPI xmlreader_SetInput(IXmlReader* iface, IUnknown *input) { xmlreader *This = impl_from_IXmlReader(iface); + IXmlReaderInput *readerinput; HRESULT hr; TRACE("(%p)->(%p)\n", This, input); @@ -1838,11 +1842,23 @@ static HRESULT WINAPI xmlreader_SetInput(IXmlReader* iface, IUnknown *input) } /* now try IXmlReaderInput, ISequentialStream, IStream */ - hr = IUnknown_QueryInterface(input, &IID_IXmlReaderInput, (void**)&This->input); - if (hr != S_OK) + hr = IUnknown_QueryInterface(input, &IID_IXmlReaderInput, (void**)&readerinput); + if (hr == S_OK) { - IXmlReaderInput *readerinput; + if (readerinput->lpVtbl == &xmlreaderinputvtbl) + This->input = impl_from_IXmlReaderInput(readerinput); + else + { + ERR("got external IXmlReaderInput implementation: %p, vtbl=%p\n", + readerinput, readerinput->lpVtbl); + IUnknown_Release(readerinput); + return E_FAIL; + } + } + + if (hr != S_OK || !readerinput) + { /* create IXmlReaderInput basing on supplied interface */ hr = CreateXmlReaderInputWithEncodingName(input, NULL, NULL, FALSE, NULL, &readerinput); @@ -2160,7 +2176,7 @@ static HRESULT WINAPI xmlreaderinput_QueryInterface(IXmlReaderInput *iface, REFI { xmlreaderinput *This = impl_from_IXmlReaderInput(iface); - TRACE("%p %s %p\n", This, debugstr_guid(riid), ppvObject); + TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppvObject); if (IsEqualGUID(riid, &IID_IXmlReaderInput) || IsEqualGUID(riid, &IID_IUnknown)) @@ -2170,6 +2186,7 @@ static HRESULT WINAPI xmlreaderinput_QueryInterface(IXmlReaderInput *iface, REFI else { WARN("interface %s not implemented\n", debugstr_guid(riid)); + *ppvObject = NULL; return E_NOINTERFACE; } @@ -2207,7 +2224,7 @@ static ULONG WINAPI xmlreaderinput_Release(IXmlReaderInput *iface) return ref; } -static const struct IUnknownVtbl xmlreaderinput_vtbl = +static const struct IUnknownVtbl xmlreaderinputvtbl = { xmlreaderinput_QueryInterface, xmlreaderinput_AddRef, @@ -2281,7 +2298,7 @@ HRESULT WINAPI CreateXmlReaderInputWithEncodingName(IUnknown *stream, readerinput = heap_alloc(sizeof(*readerinput)); if(!readerinput) return E_OUTOFMEMORY; - readerinput->IXmlReaderInput_iface.lpVtbl = &xmlreaderinput_vtbl; + readerinput->IXmlReaderInput_iface.lpVtbl = &xmlreaderinputvtbl; readerinput->ref = 1; readerinput->imalloc = imalloc; readerinput->stream = NULL;

1 0

Nikolay Sivov : comctl32/listview: Implement dragging with right button with a message loop.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: 500da7def68934d2e2d74e611bb6cce77d59d60e URL: http://source.winehq.org/git/wine.git/?a=commit;h=500da7def68934d2e2d74e611… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Jan 28 07:56:01 2013 +0400 comctl32/listview: Implement dragging with right button with a message loop. --- dlls/comctl32/listview.c | 186 ++++++++++++++++++++++++++-------------------- 1 files changed, 105 insertions(+), 81 deletions(-) diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c index 49da83c..a87ab98 100644 --- a/dlls/comctl32/listview.c +++ b/dlls/comctl32/listview.c @@ -99,7 +99,6 @@ * -- LVN_GETINFOTIP * -- LVN_HOTTRACK * -- LVN_SETDISPINFO - * -- LVN_BEGINRDRAG * * Messages: * -- LVM_ENABLEGROUPVIEW @@ -325,7 +324,6 @@ typedef struct tagLISTVIEW_INFO /* mouse operation */ BOOL bLButtonDown; - BOOL bRButtonDown; BOOL bDragging; POINT ptClickPos; /* point where the user clicked */ INT nLButtonDownItem; /* tracks item to reset multiselection on WM_LBUTTONUP */ @@ -3984,17 +3982,23 @@ static VOID CALLBACK LISTVIEW_ScrollTimer(HWND hWnd, UINT uMsg, UINT_PTR idEvent */ static LRESULT LISTVIEW_MouseMove(LISTVIEW_INFO *infoPtr, WORD fwKeys, INT x, INT y) { + LVHITTESTINFO ht; + RECT rect; + POINT pt; + if (!(fwKeys & MK_LBUTTON)) infoPtr->bLButtonDown = FALSE; if (infoPtr->bLButtonDown) { - POINT tmp; - RECT rect; - LVHITTESTINFO lvHitTestInfo; - WORD wDragWidth = GetSystemMetrics(SM_CXDRAG); - WORD wDragHeight= GetSystemMetrics(SM_CYDRAG); + rect.left = rect.right = infoPtr->ptClickPos.x; + rect.top = rect.bottom = infoPtr->ptClickPos.y; + InflateRect(&rect, GetSystemMetrics(SM_CXDRAG), GetSystemMetrics(SM_CYDRAG)); + } + + if (infoPtr->bLButtonDown) + { if (infoPtr->bMarqueeSelect) { POINT coords_orig; @@ -4037,22 +4041,17 @@ static LRESULT LISTVIEW_MouseMove(LISTVIEW_INFO *infoPtr, WORD fwKeys, INT x, IN return 0; } - rect.left = infoPtr->ptClickPos.x - wDragWidth; - rect.right = infoPtr->ptClickPos.x + wDragWidth; - rect.top = infoPtr->ptClickPos.y - wDragHeight; - rect.bottom = infoPtr->ptClickPos.y + wDragHeight; - - tmp.x = x; - tmp.y = y; + pt.x = x; + pt.y = y; - lvHitTestInfo.pt = tmp; - LISTVIEW_HitTest(infoPtr, &lvHitTestInfo, TRUE, TRUE); + ht.pt = pt; + LISTVIEW_HitTest(infoPtr, &ht, TRUE, TRUE); /* reset item marker */ - if (infoPtr->nLButtonDownItem != lvHitTestInfo.iItem) + if (infoPtr->nLButtonDownItem != ht.iItem) infoPtr->nLButtonDownItem = -1; - if (!PtInRect(&rect, tmp)) + if (!PtInRect(&rect, pt)) { /* this path covers the following: 1. WM_LBUTTONDOWN over selected item (sets focus on it) @@ -4072,12 +4071,12 @@ static LRESULT LISTVIEW_MouseMove(LISTVIEW_INFO *infoPtr, WORD fwKeys, INT x, IN if (!infoPtr->bDragging) { - lvHitTestInfo.pt = infoPtr->ptClickPos; - LISTVIEW_HitTest(infoPtr, &lvHitTestInfo, TRUE, TRUE); + ht.pt = infoPtr->ptClickPos; + LISTVIEW_HitTest(infoPtr, &ht, TRUE, TRUE); /* If the click is outside the range of an item, begin a highlight. If not, begin an item drag. */ - if (lvHitTestInfo.iItem == -1) + if (ht.iItem == -1) { NMHDR hdr; @@ -4102,7 +4101,7 @@ static LRESULT LISTVIEW_MouseMove(LISTVIEW_INFO *infoPtr, WORD fwKeys, INT x, IN NMLISTVIEW nmlv; ZeroMemory(&nmlv, sizeof(nmlv)); - nmlv.iItem = lvHitTestInfo.iItem; + nmlv.iItem = ht.iItem; nmlv.ptAction = infoPtr->ptClickPos; notify_listview(infoPtr, LVN_BEGINDRAG, &nmlv); @@ -10015,6 +10014,52 @@ static LRESULT LISTVIEW_LButtonDblClk(LISTVIEW_INFO *infoPtr, WORD wKey, INT x, return 0; } +static LRESULT LISTVIEW_TrackMouse(const LISTVIEW_INFO *infoPtr, POINT pt) +{ + MSG msg; + RECT r; + + r.top = r.bottom = pt.y; + r.left = r.right = pt.x; + + InflateRect(&r, GetSystemMetrics(SM_CXDRAG), GetSystemMetrics(SM_CYDRAG)); + + SetCapture(infoPtr->hwndSelf); + + while (1) + { + if (PeekMessageW(&msg, 0, 0, 0, PM_REMOVE | PM_NOYIELD)) + { + if (msg.message == WM_MOUSEMOVE) + { + pt.x = (short)LOWORD(msg.lParam); + pt.y = (short)HIWORD(msg.lParam); + if (PtInRect(&r, pt)) + continue; + else + { + ReleaseCapture(); + return 1; + } + } + else if (msg.message >= WM_LBUTTONDOWN && + msg.message <= WM_RBUTTONDBLCLK) + { + break; + } + + DispatchMessageW(&msg); + } + + if (GetCapture() != infoPtr->hwndSelf) + return 0; + } + + ReleaseCapture(); + return 0; +} + + /*** * DESCRIPTION: * Processes mouse down messages (left mouse button). @@ -10619,94 +10664,76 @@ static LRESULT LISTVIEW_RButtonDblClk(const LISTVIEW_INFO *infoPtr, WORD wKey, I /*** * DESCRIPTION: - * Processes mouse down messages (right mouse button). + * Processes WM_RBUTTONDOWN message and corresponding drag operation. * * PARAMETER(S): * [I] infoPtr : valid pointer to the listview structure * [I] wKey : key flag - * [I] x,y : mouse coordinate + * [I] x, y : mouse coordinate * * RETURN: * Zero */ static LRESULT LISTVIEW_RButtonDown(LISTVIEW_INFO *infoPtr, WORD wKey, INT x, INT y) { - LVHITTESTINFO lvHitTestInfo; - INT nItem; + LVHITTESTINFO ht; + INT item; - TRACE("(key=%hu,X=%u,Y=%u)\n", wKey, x, y); + TRACE("(key=%hu, x=%d, y=%d)\n", wKey, x, y); /* send NM_RELEASEDCAPTURE notification */ if (!notify(infoPtr, NM_RELEASEDCAPTURE)) return 0; + /* determine the index of the selected item */ + ht.pt.x = x; + ht.pt.y = y; + item = LISTVIEW_HitTest(infoPtr, &ht, TRUE, TRUE); + /* make sure the listview control window has the focus */ if (!infoPtr->bFocus) SetFocus(infoPtr->hwndSelf); - /* set right button down flag */ - infoPtr->bRButtonDown = TRUE; - - /* determine the index of the selected item */ - lvHitTestInfo.pt.x = x; - lvHitTestInfo.pt.y = y; - nItem = LISTVIEW_HitTest(infoPtr, &lvHitTestInfo, TRUE, TRUE); - - if ((nItem >= 0) && (nItem < infoPtr->nItemCount)) + if ((item >= 0) && (item < infoPtr->nItemCount)) { - LISTVIEW_SetItemFocus(infoPtr, nItem); + LISTVIEW_SetItemFocus(infoPtr, item); if (!((wKey & MK_SHIFT) || (wKey & MK_CONTROL)) && - !LISTVIEW_GetItemState(infoPtr, nItem, LVIS_SELECTED)) - LISTVIEW_SetSelection(infoPtr, nItem); + !LISTVIEW_GetItemState(infoPtr, item, LVIS_SELECTED)) + LISTVIEW_SetSelection(infoPtr, item); } else - { LISTVIEW_DeselectAll(infoPtr); - } - - return 0; -} - -/*** - * DESCRIPTION: - * Processes mouse up messages (right mouse button). - * - * PARAMETER(S): - * [I] infoPtr : valid pointer to the listview structure - * [I] wKey : key flag - * [I] x,y : mouse coordinate - * - * RETURN: - * Zero - */ -static LRESULT LISTVIEW_RButtonUp(LISTVIEW_INFO *infoPtr, WORD wKey, INT x, INT y) -{ - LVHITTESTINFO lvHitTestInfo; - POINT pt; - TRACE("(key=%hu,X=%u,Y=%u)\n", wKey, x, y); + if (LISTVIEW_TrackMouse(infoPtr, ht.pt)) + { + if (ht.iItem != -1) + { + NMLISTVIEW nmlv; - if (!infoPtr->bRButtonDown) return 0; - - /* set button flag */ - infoPtr->bRButtonDown = FALSE; + memset(&nmlv, 0, sizeof(nmlv)); + nmlv.iItem = ht.iItem; + nmlv.ptAction = ht.pt; - /* Send NM_RCLICK notification */ - lvHitTestInfo.pt.x = x; - lvHitTestInfo.pt.y = y; - LISTVIEW_HitTest(infoPtr, &lvHitTestInfo, TRUE, FALSE); - if (!notify_click(infoPtr, NM_RCLICK, &lvHitTestInfo)) return 0; + notify_listview(infoPtr, LVN_BEGINRDRAG, &nmlv); + } + } + else + { + SetFocus(infoPtr->hwndSelf); - /* Change to screen coordinate for WM_CONTEXTMENU */ - pt = lvHitTestInfo.pt; - ClientToScreen(infoPtr->hwndSelf, &pt); + ht.pt.x = x; + ht.pt.y = y; + LISTVIEW_HitTest(infoPtr, &ht, TRUE, FALSE); - /* Send a WM_CONTEXTMENU message in response to the RBUTTONUP */ - SendMessageW(infoPtr->hwndSelf, WM_CONTEXTMENU, - (WPARAM)infoPtr->hwndSelf, MAKELPARAM(pt.x, pt.y)); + if (notify_click(infoPtr, NM_RCLICK, &ht)) + { + /* Send a WM_CONTEXTMENU message in response to the WM_RBUTTONUP */ + SendMessageW(infoPtr->hwndSelf, WM_CONTEXTMENU, + (WPARAM)infoPtr->hwndSelf, (LPARAM)GetMessagePos()); + } + } return 0; } - /*** * DESCRIPTION: * Sets the cursor. @@ -11600,9 +11627,6 @@ LISTVIEW_WindowProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam) case WM_RBUTTONDOWN: return LISTVIEW_RButtonDown(infoPtr, (WORD)wParam, (SHORT)LOWORD(lParam), (SHORT)HIWORD(lParam)); - case WM_RBUTTONUP: - return LISTVIEW_RButtonUp(infoPtr, (WORD)wParam, (SHORT)LOWORD(lParam), (SHORT)HIWORD(lParam)); - case WM_SETCURSOR: return LISTVIEW_SetCursor(infoPtr, wParam, lParam);

1 0

Dmitry Timoshkov : dwrite: Avoid triggering an exception on DLL_PROCESS_DETACH.
by Alexandre Julliard 28 Jan '13

28 Jan '13

Module: wine Branch: master Commit: 1ea524a861f94e09a609507d6b6d0079ff87dae2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1ea524a861f94e09a609507d6… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jan 28 14:37:36 2013 +0800 dwrite: Avoid triggering an exception on DLL_PROCESS_DETACH. --- dlls/dwrite/font.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/dwrite/font.c b/dlls/dwrite/font.c index 0ec581b..191397e 100644 --- a/dlls/dwrite/font.c +++ b/dlls/dwrite/font.c @@ -793,7 +793,8 @@ static void release_font_collection(IDWriteFontCollection *iface) void release_system_fontcollection(void) { - release_font_collection(system_collection); + if (system_collection) + release_font_collection(system_collection); } HRESULT get_system_fontcollection(IDWriteFontCollection **collection)

1 0