Why does it close here but not there?

Code:

---

int CALLBACK DlgProc(HWND hWndDlg, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
        switch (uMsg)
        {
        case WM_COMMAND:
                if (LOWORD(wParam) == IDC_BTN_TEST)
                {
                        iCount = ((GetTickCount() / 1000) / 60);
                        ltoa(iCount, szTicks, 10);
                        MessageBox(hWndDlg, szTicks, "Counter", MB_OK);
                }
                if(LOWORD(wParam) == IDC_BTN_CANCEL)
                {
                        MessageBox(hWndDlg, "Cancel pressed!", "Button", MB_OK);
                }
                if(LOWORD(wParam) == IDC_BTN_OK)
                {
                        MessageBox(hWndDlg, "OK pressed!", "Button", MB_OK);
                        return false;
                }
        case WM_CLOSE:
                EndDialog(hWndDlg, 0);
                PostQuitMessage(0);
                break;
        }
        return false;
}

---

If I click the IDC_BTN_TEST button, it tells me a number, like I want it to, then closes. If I click the IDC_BTN_OK button, it tells me "OK pressed!", but it doesn't close. Why is this?

Because you are not breaking or returning after the Test button's code. The case is falling through to the WM_CLOSE code.

Code:

---

int CALLBACK DlgProc(HWND hWndDlg, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
        switch (uMsg)
        {
        case WM_COMMAND:
                if (LOWORD(wParam) == IDC_BTN_TEST)
                {
                        iCount = ((GetTickCount() / 1000) / 60);
                        ltoa(iCount, szTicks, 10);
                        MessageBox(hWndDlg, szTicks, "Counter", MB_OK);
                                                return false;
                }
                if(LOWORD(wParam) == IDC_BTN_CANCEL)
                {
                        MessageBox(hWndDlg, "Cancel pressed!", "Button", MB_OK);
                }
                if(LOWORD(wParam) == IDC_BTN_OK)
                {
                        MessageBox(hWndDlg, "OK pressed!", "Button", MB_OK);
                        return false;
                }
        case WM_CLOSE:
                EndDialog(hWndDlg, 0);
                PostQuitMessage(0);
                break;
        }
        return false;
}

---

:)

*wonders why he didn't see that* Thanks a lot :)

why do you return false?
And why false instead of FALSE?
DlgProc has a predefined prototype with a BOOL return value, not bool. bool is a basic C++ data type, while BOOL is just an alias for int, and TRUE and FALSE are defined as 1 and 0.

TRUE = true = 1
FALSE = false = 0

BOOL is a win32 data type and bool is a standard c++ data type. That is the difference.

Megatron: Try this code:

Code:

---

bool cppbool;
BOOL winbool;

cppbool = true;
cppbool = FALSE;

winbool = false;
winbool = TRUE;

---

You'll get a warning at cppbool = FALSE. Only a warning - the compiler will do the type conversion, but it#s a "possible speed hazard". The compiler will also do a type conversion at winbool = false