Writing Raw Bitmap File

I've been doing win32 for a little while now and I've just encountered a bit of a problem making a program to take a screenshot. Even after finding a convenient msdn article: http://msdn.microsoft.com/library/de...tmaps_7zfp.asp I was still unable to do this. Currently with the help of these MSDN functions, it writes a corrupted (black?) bitmap file. Several other attempts at creating the headers and getting it all written out resulted in empty or corrupted. The souce is attached, originally I was just screwing around with the effect repeated BitBlt's GetDC(GetDesktopWindow()) but now I want to get that into a bitmap file.

Code:

---

void Screenshot(HDC hDC, HWND hwnd)
{
    SIZE size;
        RECT rect;
        GetClientRect(hwnd, &rect);
    size.cx = GetSystemMetrics(SM_CXSCREEN);
    size.cy = GetSystemMetrics(SM_CYSCREEN);
        HDC hdcMemory = GetDC(GetDesktopWindow());
    HBITMAP hBitmap = CreateCompatibleBitmap(hdcMemory, size.cx, size.cy);
    if(hBitmap) {
                BitBlt(hDC, 0, 0, size.cx, size.cy, hdcMemory, 0, 0, SRCCOPY);
                //Save(hwnd, CreateInfoHeader(hwnd, hBitmap), hBitmap, hDC);
        DeleteDC(hdcMemory);       
        DeleteObject(hBitmap);
    }
    ReleaseDC(NULL, hDC);
}

---

You're saving an empty bitmap! You copy the screen image to your window but not the memory bitmap. Here is an edited, commented but untested version of your function:

Code:

---

void Screenshot(HDC hDC, HWND hwnd)
{
        RECT rect;
        GetClientRect(hwnd, &rect);  /* Not currently used */

        /* Get the size of the screen */
        SIZE size;
        size.cx = GetSystemMetrics(SM_CXSCREEN);
        size.cy = GetSystemMetrics(SM_CYSCREEN);

        /* Get a device context for the screen */
        HDC hdcScreen = CreateDC(TEXT("DISPLAY"), NULL, NULL, NULL);

        /* Create a memory bitmap and put it in a memory DC so we can draw on it */
        HDC hdcShot = CreateCompatibleDC(hdcScreen);
        HBITMAP hBitmapShot = CreateCompatibleBitmap(hdcScreen, size.cx, size.cy);
        HBITMAP hBitmapOld = (HBITMAP) SelectObject(hdcShot, hBitmapShot);

        /* Copy the screen image to our memory bitmap */
        BitBlt(hdcShot, 0, 0, size.cx, size.cy, hdcScreen, 0, 0, SRCCOPY);

        /* You can copy the screen to your window (hDC) here if needed */
        /* BitBlt(hDC, 0, 0, size.cx, size.cy, hdcScreen, 0, 0, SRCCOPY); */

        /* Save our memory bitmap to file */
        Save(hwnd, CreateInfoHeader(hwnd, hBitmapShot), hBitmapShot, hdcShot);

        /* Cleanup memory bitmap and DC */
        SelectObject(hdcShot, hBitmapOld);
        DeleteObject(hBitmapShot);
        DeleteDC(hdcShot);

        /* Cleanup screen DC */
        DeleteDC(hdcScreen);
}

---