[PATCH] user32: Reimplement TrackMouseEvent.

Alexandre Julliard julliard at winehq.org
Mon Nov 20 04:52:39 CST 2017


Rafał Harabień <rafalh1992 at o2.pl> writes:

> Improvements:
> - Don't use timer for TME_LEAVE and react immediately on mouse motion.
> It improves user experience for controls depending on this API like toolbar.
> - Fail when called with invalid flags
> - Hold tracking info for each win32 thread independently
> - Merge tracking requests if possible (leave+hover)
> - Return hover time 0 when hover tracking is disabled

I still don't see a test justifying the use of local thread data. In
fact, if you have to do things like this:

> +    /* use internal message to get access to user_thread_info for tracked window */
> +    return SendNotifyMessageW(ptme->hwndTrack, WM_WINE_TRACKMOUSEEVENT, ptme->dwFlags, ptme->dwHoverTime);

it suggests that maybe the thread data is not the right place.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list