winAPI hello world won't quit !need help to solve it

I just started learning the Win API,
this is the first code i wrote by referring some various tutorials i found in the internet...

i have a problem, the program won't close. when i click the X on the corner the window is destroyed, but the process still keeps running ( i can see it on the Task Manager )...

how do i make the process end/quit... dosen't WM_QUIT do that??

Code:

---

#include <Windows.h>

// message handler
LRESULT CALLBACK WndProc(HWND hWindow, UINT iMessage,WPARAM wParam, LPARAM lParam);



int WINAPI WinMain(
                HINSTANCE hInstance, // Handle to the current instance
                HINSTANCE hPrevInst, // Handle to the previous instance
                LPSTR lpCmdLine,                // Pointer to the command line
                int nShowCmd                        // Shows the state of the window
                )
{
        WNDCLASS kWndClass;        // window object

        // set visual properties of the window
        kWndClass.hCursor = LoadCursor(NULL, IDC_ARROW);
        kWndClass.hIcon = LoadIcon(NULL, IDI_APPLICATION);
        kWndClass.hbrBackground = (HBRUSH)GetStockObject(WHITE_BRUSH);

        // set system properties of the window
        kWndClass.hInstance = hInstance;
        kWndClass.lpfnWndProc = (WNDPROC)WndProc;
        kWndClass.lpszClassName = TEXT("01 Basic Window");

        // set extra properties
        kWndClass.lpszMenuName = NULL;
        kWndClass.cbClsExtra = NULL;
        kWndClass.cbWndExtra = NULL;
        kWndClass.style = NULL;

        // register the window object
        if(!RegisterClass(&kWndClass))
        {
                MessageBox(NULL, TEXT("Window Registration Failed!"), TEXT("Error!"),
            MB_ICONEXCLAMATION | MB_OK);
                return -1;
        }

        HWND hWindow;        // The window

        hWindow = CreateWindow(
                                TEXT("01 Basic Window"), // Registered class name
                                TEXT("A Blank Window"),        // Application window name
                                WS_OVERLAPPEDWINDOW | WS_VISIBLE,        // Window style
                                CW_USEDEFAULT,  // Horizontal position of the window
                                CW_USEDEFAULT,  // Vertical position of the window
                                CW_USEDEFAULT,  // Window width
                                CW_USEDEFAULT,  // Window height
                                NULL,          // Handle to the parent window
                                NULL,          // Handle to the menu the identifier
                                  hInstance,      // Handle to the application instance
                                NULL            // Pointer to the window-creation data
                                );

        // Check if it failed to create the window successfully
        if(!hWindow)
                return FALSE;

        ShowWindow(hWindow,nShowCmd);
        UpdateWindow(hWindow);

        MSG kMessage;        // Message

        // Message Loop
        while( GetMessage(&kMessage,hWindow,0,0)!= FALSE )
        {
                TranslateMessage(&kMessage);
                DispatchMessage(&kMessage);
        }

        return kMessage.wParam;
}


LRESULT CALLBACK WndProc(HWND hWindow, UINT iMessage,WPARAM wParam, LPARAM lParam)
{
        switch(iMessage)
    {
        case WM_CLOSE:
            DestroyWindow(hWindow);
        break;

        case WM_DESTROY:
            PostQuitMessage(0);
        break;

        default:
            return DefWindowProc(hWindow,iMessage,wParam,lParam);
    }

    return 0;
}

---

--------------------------------------------------------------------------------------------------------------------------------------
by the way,
i am using windows 7 32 bit.

do i have to use WNDCLASSEX other than WNDCLASS in win 7 ??

i now tried the code from this site,
Tutorial: A Simple Window

and it worked correctly... :(

That code is using WNDCLASSEX and relevent "Ex" functions....
--------------------------------------------------------------------------------------------------------------------------------------

Check the return value of GetMessage or take a look at PeekMessage, Strange the window messgaes look fine, can you verify that WM_DESTROY is being called.

WNDCLASS should be fine msdn states: "You can still use WNDCLASS and RegisterClass if you do not need to set the small icon associated with the window class.". but if it works now maybe it was a msvc prob or a breakpoint

Quote:

---

Originally Posted by pYro

Check the return value of GetMessage or take a look at PeekMessage, Strange the window messgaes look fine, can you verify that WM_DESTROY is being called

---

thanks pYro,

yes i tried that, i am using VC++ too. so i put some breakpoints and followed the execution of code... WM_CLOSE and WM_DESTROY are passed... but the process dosent end.. only the window gets destroyed... and sometimes when i right click on the title bar and select close from the drop down menu, it works... but the X on the corner dosent work like that...

Oki,
I found that if i replace

Code:

---

GetMessage(&kMessage,hWindow,0,0)

---

by this,

Code:

---

GetMessage(&kMessage,NULL,0,0)

---

it works....

but is it OK to do it... will then the other messages work correctly after putting it like that???

i tried putting

Code:

---

hWindow=NULL;

---

in WM_CLOSE and WM_DESTROY, but they both didn't work...

or i can use PeekMessage and check

Code:

---

kMessage.message==WM_DESTROY

---

before i put the kMessage to the GetMessage() rite...?

gotit, GetMessage returns -1 after window close, in the loop check for getmessage error like msdn example

Code:

---

        // Message Loop
        BOOL  b_Ret = FALSE;
        while( ( b_Ret = GetMessage(&kMessage,hWindow,0,0) )!= FALSE )
        {
                if (b_Ret == -1){

                        break;
                }
                TranslateMessage(&kMessage);
                DispatchMessage(&kMessage);
        }

---

yes it is fine to replace with NULL, gets messages for all windows. placing the window handle in has caused the call to fail after the window is destroyed... because the hWindow is invalid and cannot get messages for it.

Yah... Thanks pYro. :)

no worries :), stumped me too for a bit; so was good to find a solution.