Why does it close here but not there?

[Wynd]

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?

[crptcblade]

Because you are not breaking or returning after the Test button's code. The case is falling through to the WM_CLOSE 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;
}

[Wynd]

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

[CornedBee]

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.

[Megatron]

TRUE = true = 1
FALSE = false = 0

[Vlatko]

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

[CornedBee]

Megatron: Try this 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