system tray help

[ZerOrDie]

case WM_TRAYNOTIFY:
		{						
			switch(lParam)				
			{				
			case WM_LBUTTONDOWN:				
				{
					ShowWindow(hwnd,SW_RESTORE);						
				}break;

			case WM_RBUTTONDOWN:
				{
					GetCursorPos(&pt);
					TrackPopupMenu(hPopupMenu, TPM_RIGHTALIGN, pt.x, pt.y, 0, hwnd, NULL);
				case IDMI_RESTORE:
					{
						MessageBox(0, "1", "1", MB_ICONSTOP | MB_OK);
						ShowWindow(hwnd,SW_RESTORE);
					}break;
				case IDMI_CLOSE:
					{
						MessageBox(0, "2", "2", MB_ICONSTOP | MB_OK);
						PostQuitMessage(0);
					}break;						
				}break;
			}
			
		}

the problem is that no matter what option i pick on my menu i always get 1 im sure its most likely a problem with my program flow...
i know i need another switch for the nested cases but what am i supossed to use as the switch?

[Codeplug]

Yes, you need another "switch() {}" to go with those cases.
IDMI_RESTORE isn't even in MSDN - so your on your own there.
Make sure you know what this is doing.

gg

[ZerOrDie]

err lol i feel like and idiot dammit :P
that should have been under the WM_COMMAND case

[NoOneHere]

Whats ub with ur breaks??
wouldnt a break be better placed inside the {}

//ur code
case Yada_Yada
{
      Blada Blada Blada
}break;

//should be:
case Yada_Yada
{
      Blada Blada Blada
      break;
}

[Hunter2]

Really? I thought it should be:

case Yada_Yada:
     Blada Blada Blada
     break;

Oh well, probably just another way of doing it.

[ZerOrDie]

Originally posted by NoOneHere
Whats ub with ur breaks??
wouldnt a break be better placed inside the {}

//ur code
case Yada_Yada
{
      Blada Blada Blada
}break;

//should be:
case Yada_Yada
{
      Blada Blada Blada
      break;
}

i like putting them there... not to mention i have seen many a w32 programmer place them in this fashion...

not to mention it does not matter if they are inside or outside...