GET_X_LPARAM: identifier not found

[mrafcho001]

As the title says i get a bunch of errors saying that GET_X_LPARAM cannot be found, not even with argument-dependent lookup

#include <windows.h>


LRESULT CALLBACK WndProc (HWND, UINT, WPARAM, LPARAM) ;

int WINAPI WinMain (HINSTANCE hInstance, HINSTANCE hPrevInstance,
                    PSTR szCmdLine, int iCmdShow)
{
	static TCHAR szAppName[] = TEXT ("HelloWin") ;
	HWND         hwnd ;
	MSG          msg ;
	WNDCLASS     wndclass ;

	wndclass.style         = CS_HREDRAW | CS_VREDRAW ;
	wndclass.lpfnWndProc   = WndProc ;
	wndclass.cbClsExtra    = 0 ;
	wndclass.cbWndExtra    = 0 ;
	wndclass.hInstance     = hInstance ;
	wndclass.hIcon         = LoadIcon (NULL, IDI_APPLICATION) ;
	wndclass.hCursor       = LoadCursor (NULL, IDC_ARROW) ;
	wndclass.hbrBackground = (HBRUSH) GetStockObject (WHITE_BRUSH) ;
	wndclass.lpszMenuName  = NULL ;
	wndclass.lpszClassName = szAppName ;

	if (!RegisterClass (&wndclass))
	{
		MessageBox (NULL, TEXT ("This program requires Windows NT!"), 
					szAppName, MB_ICONERROR) ;
	return 0 ;
	}
	hwnd = CreateWindow (szAppName,                  // window class name
						TEXT ("Hello Windows"),     // window caption
						WS_OVERLAPPEDWINDOW,        // window style
						CW_USEDEFAULT,              // initial x position
						CW_USEDEFAULT,              // initial y position
						CW_USEDEFAULT,              // initial x size
						CW_USEDEFAULT,              // initial y size
						NULL,                       // parent window handle
						NULL,                       // window menu handle
						hInstance,                  // program instance handle
						NULL) ;                     // creation parameters


	ShowWindow (hwnd, iCmdShow) ;
	UpdateWindow (hwnd) ;
     
	while (GetMessage (&msg, NULL, 0, 0))
	{
		TranslateMessage (&msg) ;
		DispatchMessage (&msg) ;
	}
	return msg.wParam ;
}

LRESULT CALLBACK WndProc (HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
	HDC         hdc ;
	PAINTSTRUCT ps ;
	RECT        rect;
	POINT pt;
	static bool LButtonDown;
	switch (message)
	{
	case WM_CREATE:
		return 0 ;
	case WM_LBUTTONDOWN:
			LButtonDown = true;
			pt.x = GET_X_LPARAM(lParam);
			pt.y = GET_Y_LPARAM(lParam);
			return 0;
	case WM_LBUTTONUP:
			LButtonDown = false;
			return 0;
	case WM_MOUSEMOVE:
			if(LButtonDown == true)
			{
				hdc = GetDC (hwnd);
				MoveToEx(hdc, pt.x, pt.y, NULL);
				LineTo(hdc, GET_X_LPARAM(lParam), GET_Y_LPARAM(lParam));
				pt.x = GET_X_LPARAM(lParam);
				pt.y = GET_Y_LPARAM(lParam);
				ReleaseDC(hwnd, hdc);
				return 0;
			}
			else
				return 0;
						
	case WM_PAINT:
		 
		hdc = BeginPaint (hwnd, &ps) ;
		EndPaint (hwnd, &ps) ;
		return 0 ;
          
	case WM_DESTROY:
		PostQuitMessage (0) ;
		return 0 ;
	}
	return DefWindowProc (hwnd, message, wParam, lParam) ;
}

Quite simple code...

[bithub]

Use LOWORD() instead of GET_X_LPARAM() and use HIWORD instead of GET_Y_LPARAM.

[novacain]

GET_X_LPARAM is in Windowsx.h

[mrafcho001]

Thanks guys, works pefrectly.

Just another thing, why does it say that it cant find sprintf?

[Dante Shamest]

sprintf is in <stdio.h>

[novacain]

Just one more thing I noticed.....

You use the point to hold the location clicked. Make the point a static or it will have lost scope (the value will change) once the WM_LBUTTONDOWN msg has been processed.

You may also have problems with the drawing (speed and shearing). Search here for code (on drawing lines and 'double buffering').

[mrafcho001]

Just one more thing I noticed.....

You use the point to hold the location clicked. Make the point a static or it will have lost scope (the value will change) once the WM_LBUTTONDOWN msg has been processed.

You may also have problems with the drawing (speed and shearing). Search here for code (on drawing lines and 'double buffering').

Yearh i realized that ones i tested, and fixed it.

Thanks though.

[Queatrix]

Somthing else that you could have done to fix it without editing any code is by including the header file windowsx.h