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:

Quote:

---

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?