Re: SetWindowHookEx

> [e-mail] writes:
> > My app uses SetWindowHookEx to install a WH_KEYBOARD_LL hook.
> > Currently, the wine implementation tells me
> > "fixme:hook:set_windows_hook system hook WH_KEYBOARD_LL won't work right"
> > when I try to use it.  I'd like to implement this so I can get my app to
> > run, but I'm not sure where to start, since I don't know why it
> > "won't work right."  Any hints/suggestions?
>
> I'm working on it. Basically it won't work right across processes at
> this point; if you have a single process it should be OK.

It's returning NULL for me, which implies it isn't working.  Is there some
sort of trace that I can send you to help you out?

-Steve

SetWindowHookEx

[Alexandre Julliard]

[e-mail] writes:
> It's returning NULL for me, which implies it isn't working.  Is there some
> sort of trace that I can send you to help you out?

Oops you are right, this should fix it:

Index: server/hook.c
===================================================================
RCS file: /opt/cvs-commit/wine/server/hook.c,v
retrieving revision 1.1
diff -u -r1.1 hook.c
--- server/hook.c	29 Oct 2002 00:41:42 -0000	1.1
+++ server/hook.c	30 Oct 2002 19:51:20 -0000
@@ -227,7 +227,8 @@
         set_error( STATUS_INVALID_PARAMETER );
         return;
     }
-    if (!(thread = get_thread_from_id( req->tid ))) return;
+    if (!req->tid) thread = (struct thread *)grab_object( current );
+    else if (!(thread = get_thread_from_id( req->tid ))) return;

if ((hook = add_hook( thread, req->id - WH_MINHOOK )))
     {