I'm using the example code here , but getting errors in this area. for (i = 0; i < CPENS; i++) { phbmp[i] = CreateCompatibleBitmap( hdcMain, wLineX, wLineY); if (i == 0) { hbmpOld = SelectObject(hdcLines, phbmp[i]); } else { SelectObject(hdcLines, phbmp[i]); } ExtFloodFill( hdcLines, 0, 0, crMenuClr, FLOODFILLBORDER); } Code: phbmp[i] = CreateCompatibleBitmap(hdcMain, wLineX, wLineY); Error: no operator "=" matches these operands operand types are HBITMAP__=HBITMAP Code: hbmpOld = SelectObject(hdcLines, phbmp[i]); Error: a value of type "HGDIOBJ" cannot be assigned to an entity of type "HBITMAP" no suitable conversion function from "HBITMAP__" to "HGDIOBJ" exists

How do I fix the following errors?

Accepted answer

Viorel 118.5K

Probably some characters were lost. To fix the first error, try to add “[ ]” to these parameters: HPEN phpen[], HBITMAP phbmp[].

To fix the conversion problems, try to use HGDIOBJ instead of HBRUSH and HBITMAP for corresponding local variables, for example: HGDIOBJ hbmpOld.

Maybe you can use the Feedback button to report the found documentation errors.

CDev-8220 220

Yes, Thanks.

Following those suggestion, fixed all the errors, and the code works.

I had to adjust the forward declaration as well.

VOID MakeLineMenu(HWND, HPEN hp[], HBITMAP hb[]);

VOID MakeLineMenu(HWND hwnd, HPEN phpen[], HBITMAP phbmp[]) {
    HMENU hmenuLines = nullptr;     // handle to Lines menu       
    HMENU hmenu = nullptr;          // handle to main menu               
    COLORREF crMenuClr = 0;         // menu-item background color        
    HBRUSH hbrBackground = nullptr; // handle to background brush        
    //HBRUSH hbrOld = nullptr;        // handle to previous brush          
    HGDIOBJ hbrObj = nullptr;       // handle to previous brush          
    WORD wLineX = 0;                // width of line bitmaps             
    WORD wLineY = 0;                // height of line bitmaps            
    HDC hdcMain = nullptr;          // handle to main window's DC        
    HDC hdcLines = nullptr;         // handle to compatible DC           
    //HBITMAP hbmpOld = nullptr;      // handle to previous bitmap 
    HGDIOBJ hbmpObj = nullptr;      // handle to previous bitmap          
    HBITMAP__ hbp{};
    int i = 0;                      // loop counter                      
    string str = "&Lines";
    // Initializing an object of wstring
    wstring temp = wstring(str.begin(), str.end());
    // Applying c_str() method on temp
    LPCWSTR wideString = temp.c_str();
    // Create the Lines menu. Add it to the menu bar
    hmenu = GetMenu(hwnd);
    hmenuLines = CreatePopupMenu();
    AppendMenu(
        hmenu, 
        MF_STRING | MF_POPUP,
        (UINT)hmenuLines, 
        wideString
    );
    // Create a brush for the menu-item background color.
    crMenuClr = GetSysColor(COLOR_MENU);
    hbrBackground = CreateSolidBrush(crMenuClr);
    /* Create a compatible device context for the line bitmaps,  
    and then select the background brush into it. */
    hdcMain = GetDC(hwnd);
    hdcLines = CreateCompatibleDC(hdcMain);
    hbrObj = SelectObject(hdcLines, hbrBackground);
    /* Get the dimensions of the check-mark bitmap. 
    The width of the line bitmaps will be five times the 
    width of the check-mark bitmap */
    wLineX = GetSystemMetrics(SM_CXMENUCHECK) * (WORD)5;
    wLineY = GetSystemMetrics(SM_CYMENUCHECK);
    /* Create the bitmaps and select them, one at a time, into 
    the compatible device context. Initialize each bitmap by 
    filling it with the menu-item background color. */
    for (i = 0; i < CPENS; i++) {
        phbmp[i] = CreateCompatibleBitmap(
            hdcMain, wLineX, wLineY);
        if (i == 0) {
            hbmpObj = SelectObject(hdcLines, phbmp[i]);
        }
        else {
            SelectObject(hdcLines, phbmp[i]);
        }
        ExtFloodFill(
            hdcLines, 0, 0, crMenuClr, FLOODFILLBORDER);
    }
    // Create the pens. 
    phpen[0] = CreatePen(PS_SOLID, 1, RGB(0, 0, 0));
    phpen[1] = CreatePen(PS_DOT, 1, RGB(0, 0, 0));
    phpen[2] = CreatePen(PS_DASH, 1, RGB(0, 0, 0));
    phpen[3] = CreatePen(PS_DASHDOT, 1, RGB(0, 0, 0));
    phpen[4] = CreatePen(PS_DASHDOTDOT, 1, RGB(0, 0, 0));
    /* Select a pen and a bitmap into the compatible device  
    context, draw a line into the bitmap, and then append 
    the bitmap as an item in the Lines menu. */
    for (i = 0; i < CPENS; i++) {
        SelectObject(hdcLines, phbmp[i]);
        SelectObject(hdcLines, phpen[i]);
        MoveToEx(hdcLines, 0, wLineY / 2, NULL);
        LineTo(hdcLines, wLineX, wLineY / 2);
        AppendMenu(
            hmenuLines, 
            MF_BITMAP, 
            static_cast<UINT_PTR>(i) + 1,
            (LPCTSTR)phbmp[i]
        );
    }
    /* Release the main window's device context and destroy the  
    compatible device context. Also, destroy the background  
    brush. */
    ReleaseDC(hwnd, hdcMain);
    SelectObject(hdcLines, hbrObj);
    DeleteObject(hbrBackground);
    SelectObject(hdcLines, hbmpObj);
    DeleteDC(hdcLines);
    return;
}

I'll use the feedback button to point to this thread.

One thing, if it's okay to ask here. What's the best way to avoid the warning:

warning C4311: 'type cast': pointer truncation from 'HMENU' to 'UINT'

in this block of code:

hmenuLines = CreatePopupMenu();
    AppendMenu(
        hmenu, 
        MF_STRING | MF_POPUP,
        (UINT)hmenuLines, 
        wideString
    );

Viorel 118.5K Reputation points

2024-04-03T19:26:01.79+00:00

Try (UINT_PTR)hmenuLines.
CDev-8220 220 Reputation points

2024-04-04T10:54:13.86+00:00

That did it! Thanks.

Share via

How do I fix the following errors?

0 additional answers

Your answer