Brush color

Hello
Can you check my code I can´t find out what is wrong with it..

Code:

LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
{
HDC              hDC;
PAINTSTRUCT      ps;
HBRUSH           hGreenBrush;
RECT             hRect;
        switch(message)
        {
          case WM_PAINT:
               hGreenBrush = CreateSolidBrush(RGB(0,255,0));
               hDC = BeginPaint(hWnd, &ps);
               &hRect = Rectangle(hDC, 10,10,500,200);
               FillRect(hDC, &hRect, hGreenBrush);
               EndPaint(hWnd, &ps);
               return 0;
          case WM_DESTROY:
          PostQuitMessage(0);
          return 0;
        }
        return DefWindowProc(hWnd, message, wParam, lParam);
}

There is supposed to be an error on the highlighted line. At least my compiler asserts it

Dev-c++

Thx for helping me

Try using SetRect() instead of Rectangle().

Originally Posted by Queatrix

Try using SetRect() instead of Rectangle().

Sorry does not work either... Should I change something else in the code, I am a beginner so I do not know the correct syntax of SetRect() exactly

Like:

Code:

SetRect(&hRect, 10, 10, 500, 200);

Rectangle()'s return type is bool not RECT* therefore you cannot set &hRect equal to the return value of Rectangle(). You will have set the values of hRect yourself or use the function mentioned by Queatrix.

And DeleteObject the brush handle returned from CreateSolidBrush when you're done with it.

Thank you very much now it seems to work the way it should...

you named your variable hRect, but it's not a handle, so it should be just rect for example.

Actually it really should be somthing like rcRect.

Originally Posted by Queatrix

Actually it really should be somthing like rcRect.

A variable name can start with an abbreviation of the type, but may also be just the struct name.