Re: [1/7] server: Implement RegisterHotKey/UnregisterHotKey.

[Alexandre Julliard]

"Vincent Povirk"  writes:
> @@ -2632,6 +2632,29 @@ enum coords_relative
>  #define SET_USER_OBJECT_FLAGS 1
>
> +/* Register a hotkey */
> +@REQ(register_hotkey)
> - user_handle_t  window;        /* handle to the window */
> - int            id;            /* hotkey identifier */
> - int            flags;         /* modifier keys */
> - int            vkey;          /* virtual key code */
>
> +@REPLY
> - int            replaced;      /* did we replace an existing hotkey? */
> - int            flags;         /* flags of replaced hotkey */
> - int            vkey;          /* virtual key code of replaced hotkey */
>
> +@END

The docs say that hotkeys are no longer replaced in recent Windows
versions. It would be a good idea to have a test for that.

[1/7] server: Implement RegisterHotKey/UnregisterHotKey.

[Vincent Povirk]

I now have a test, and it turns out the docs are incorrect. After
registering a new hotkey with the same id/hwnd as an existing hotkey,
the old key combination no longer generates a WM_HOTKEY message, and
the new one does, on all Windows versions that winetestbot tested.

I'll send that after I find out what happens to the current series.

On Fri, Jun 17, 2011 at 5:32 AM, Alexandre Julliard  wrote:
> "Vincent Povirk"  writes:
> > @@ -2632,6 +2632,29 @@ enum coords_relative
> >  #define SET_USER_OBJECT_FLAGS 1
> >
> > +/* Register a hotkey */
> > +@REQ(register_hotkey)
> > - user_handle_t  window;        /* handle to the window */
> > - int            id;            /* hotkey identifier */
> > - int            flags;         /* modifier keys */
> > - int            vkey;          /* virtual key code */
> >
> > +@REPLY
> > - int            replaced;      /* did we replace an existing hotkey? */
> > - int            flags;         /* flags of replaced hotkey */
> > - int            vkey;          /* virtual key code of replaced hotkey */
> >
> > +@END
>
> The docs say that hotkeys are no longer replaced in recent Windows
> versions. It would be a good idea to have a test for that.
>
> --
> Alexandre Julliard
> [e-mail]