accelerator keys in win32 API

ive tried long and hard to get accelerator keys working, but its proved very difficult. i just cant see what im doing wrong.

i created an accelerator table with the MSVC resource editor. i added an accelerator. i defined the IDs. i added code before the message loop to load the table, and modified the message loop to translate accelerator messages. then i set up a capture in WM_COMMAND for the ID of the key. but nothing happens, no message is sent at all when i press the accelerator key. what could be wrong?

Code:

IDR_ACCELERATOR1 ACCELERATORS DISCARDABLE 
BEGIN
    "Z",            ACCEL_QUIT,             VIRTKEY, CONTROL, ALT, NOINVERT
END

Code:

if ( (table=LoadAccelerators(hInstance,
    (LPCTSTR)MAKELONG(MAKEINTRESOURCE(IDR_ACCELERATOR1),0)))==NULL)
	        exit(500);
	

    // Step 3: The Message Loop
    while(GetMessage(&Msg, NULL, 0, 0) > 0)
    {
		if(!TranslateAccelerator(Msg.hwnd, table, &Msg))
		{
			TranslateMessage(&Msg);
			DispatchMessage(&Msg);
		}
    }

Code:

//these are at the very top of the source file
#define ACCEL_QUIT 10032
#define IDR_ACCELERATOR1 10033

Code:

case WM_COMMAND:
    {
          switch(LOWORD(wParam))
                {
                       case ACCEL_QUIT:
		MessageBox(hwnd,"Quit","!",MB_OK);
		PostQuitMessage(0);				return 0;

thats all the code involved. are there any glaring errors there, or anything im doing wrong?

You are using LoadAccelerators incorrectly, this is how it should be done:

Code:

LoadAccelerators (hInstance, MAKEINTRESOURCE (IDR_ACCELERATOR1));

This is as described in msdn for the API fn:

lpTableName
Pointer to a null-terminated string that names the accelerator table to load. Alternatively, this parameter can specify the resource identifier of an accelerator-table resource in the low-order word and zero in the high-order word. The MAKEINTRESOURCE macro can be used to create this value.

So you could rename your accelerator resource "IDR_ACCELERATOR" (with the quotes), remove the #defined numerical value for IDR_ACCELERATOR and:

Code:

LoadAccelerators(hInstance,TEXT("IDR_ACCELERATOR"));

as an alternative. You can, of course, give it any name you like.

i tried both those methods and nothing worked. with the first method, i received a NULL return from the LoadAccelerators function.
in the second method it seemed that the accelerator table loaded okay, but nothing happened when i pressed the accelerator. i then went into Spy++ and found this:

Code:

<02022> 00000FC4 S WM_COMMAND wNotifyCode:1 (sent from accelerator) wID:40001

i thought maybe it was sending 40001 as the id of the accelerator, but nothing came of that thought. is there still something that im doing wrong?