Thread: WNDCLASS and class

I have the following code:

class XXX {
LRESULT CALLBACK mywndproc(HWND, UINT, LPARAM, WPARAM); // or WPARAM, LPARAM, doesn't matter

short createawindow();
};


short XXX::createawindow()
{
WNDCLASS wndclass;
...
wndclass.lpfnWndProc = mywindowproc; //!!!

return 0;
}

LRESULT CALLBACK XXX::mywndproc(args...)
{
code
}

so, what's wrong with the line marked with the !!!s?

Why do I get the error msg:
can't convert whatever(__stdcall XXX::*)whaterver to whatever whatever?
I mean, I know that the conversion is invalid, but I still want to use the callback function within my class.

this might just be a typo that you did when you posted to the board but I dont see what else would be wrong so could it be that

wndclass.lpfnWndProc = mywindowproc; //!!! -- should be wndclass.lpfnWndProc = mywndproc;

since both the prototype and function header are called wndproc not windowproc

nope, that's not the problem,
of course i ment mywndproc!
just a typo.

btw, my real code doesn't look that way, this is just a short "briefing" that leads to the problem i have.

the problem is because of the class.
as I said the error message is:

cant convert (something)(__stdcall XXX::*)(something) to (something)(something)

so the only difference between the two types is the (__stdcall XXX::*)

Your compiler may be picky about what you are doing here. Try adding changing short createawindow() to short createawindow(void) and changing short XXX::createwindow() to short XXX::createwindow(void). If this doesn't work then I don't know what will.

class XXX {
LRESULT CALLBACK mywndproc(HWND, UINT, LPARAM, WPARAM); // or WPARAM, LPARAM, doesn't matter
short createawindow();
};
short XXX::createawindow()
{
WNDCLASS wndclass;
...
wndclass.lpfnWndProc = mywindowproc; //!!!

The method is seen by the compiler as :
mywndproc(XXX* const this,HWND,UINT,WPARAM,LPARAM);
// the wParam should come first
so it can't convert the pointer, since it's declared to point to a function of proto :
func(HWND,UINT,WPARAM,LPARAM);

If you'd like to declare a pointer to your method, you'd do :
typedef LRESULT (XXX::*pfnMyMethod_t)(HWND,UINT,WPARAM,LPARAM);

and then : pfnMyMethod_t pfnProc = XXX_Instance.mywindowproc;

Hi
The types of the lpfnWndProc member of the WNDCLASS struct and your method of the class XXX are not the same.
Just each non-static method of the class has a 'hidden' first argument which value is used as 'this' - actually a pionter to the instance of the class(real object). so the above two types are not the same - just they looking the same.

Workaround of your problem may be to use the SetWindowLong
to set the ponter to the instance of the class (this), as a long value in the window's extra bytes, declare the static method and set it to the lpfnWindowProc member of the WNDCLASS struct when the window is created and then just to re-route the arguments to the desired function of the object which 'this' is in the extra window bytes.

pfnMyMethod_t pfnProc = XXX_Instance.mywindowproc;

I think this should be :

Code:

pfnMyMethod_t pfnProc = &XXX::mywindowproc;

and then call it :

Code:

(XXX_Instance.*pfnProc)(hWnd,wMsg,wParam,lParam);

Sorry for the inconvenience