🚀 go-pugleaf

Better to not use a cast.

Instead:

What HOOKPROC is (from winuser.h)

typedef LRESULT (CALLBACK* HOOKPROC)(
     int code,
     WPARAM wParam,
     LPARAM lParam);

Change your function to be

LRESULT CALLBACK KeyboardProc(int ncode, wParam wparam, lParam lparam)

so that it matches the type for which SetWindowsHookEx is supposed to 
receive.  You are passing an HINSTANCE to the wParam argument so cast that 
calling parameter.  Do not cast the whole function as that masks changes 
made in any future edits and they can have devastating effects, for example: 
forgetting to label the function as CALLBACK or __stdcall.

.  Ed

> amol wrote in message
> news:47e0a645$1@newsgroups.borland.com...
>
> Hi there,
>
> I am getting following error messages while using SetWindowsHookEx:
>
> "cannot convert 'long (__stdcall *)(int,unsigned int,long)'
> to 'int(__stdcall *)()' "
>
> "Type mismatch in parameter 'lpfn' (wanted 'int(__stdcall *)()', got 
> 'long(__stdcall *) (int,unsigned int,long)')"
>
>
> I am calling the "SetWindowsHookEx" as follow:
>
> hook = SetWindowsHookEx(WH_KEYBOARD,KeyboardProc,hinstance,NULL);
> LRESULT CALLBACK KeyboardProc(int ncode,WPARAM wparam,LPARAM lparam)
> {
>  return ( CallNextHookEx(hook,ncode,wparam,lparam) );
> }
>
> As per the documentation "KeyboardProc" should take three parameters, but 
> it is giving me error.
> I am using C++ Builder 6.