Thread: LoadBitmap function call is constantly failing for some reason

This is my code so far for trying to load a few bitmaps into my program, although at the moment, it's not going well, lol. For some reason, the function calls to LoadBitmap are constantly failing, and I can't find where the problem is.

In my main.cpp file:

Code:

case WM_CREATE:
                // Create handles for the bitmaps. These will be used in the Button replace function
                hbmCross = LoadBitmap(hThisInstance, "ID_BMP_CROSS");
                hbmNaught = LoadBitmap(hThisInstance, "ID_BMP_NAUGHT");
                if(!hbmCross || !hbmNaught) {
                          sprintf(Info.buffer, "Could not load resources from file");
                          MessageBox(hwnd, Info.buffer, "Error Loading Resources", MB_ICONERROR);
                          SendMessage(hwnd, WM_DESTROY, 0, 0);
                          return -1;
                          }

The important line is the LoadBitmap function calls, since these are the one(s) which are always failing.

included in my images.rc file:

Code:

/* images.rc which contains the IDs for the image files used */
ID_BMP_CROSS BITMAP "cross.bmp"
ID_BMP_NAUGHT BITMAP "naught.bmp"

I've added my images.rc file to the project. At first, I thought that the problem was different names used in the program, from the names of the files in the directory, but that's still not working. I'm out of ideas as to what the problem might be.

[edit]I've also tried using MAKEINTRESOURCE, but that didn't work either[/edit]

If you are loading a resource rather than a file, your second argument to LoadBitmap should be something like:

Code:

MAKEINTRESOURCE(ID_BMP_CROSS)

Right now you are trying to load a bitmap from a file called "ID_BMP_CROSS", and my guess is that this is not a valid bitmap file in your system.

You could, potentially use "cross.bmp" instead, but then you wouldn't need to have them in the resource file - so my guess is that you really want to use the MAKEINTRESOURCE variant.

--
Mats

And what does "GetLastError" say about it? What is your current code?

--
Mats

Originally Posted by Swarvy

Is that not the same as putting in the resource file:

Code:

hbmCross BITMAP "cross.bmp"
hbmNaught BITMAP "naught.bmp"

If so, then yeah, i've done that. lol. I've also got defines in the main header file for them, but that hasn't worked so far.

So, again, what does "GetLastError" return?

--
Mats

Originally Posted by tabstop

Well, it's one of those "is your printer on"-type questions that works way more often than it should. (I'm tempted to say you should copy cross.bmp into every directory on your computer just to be sure, but I won't.) Can you find your other resources, or is it LoadBitmap specific?

I also have a menu in my program, but that works fine. Both the bitmaps and the menu are defined in the same resource file, so I don't think that's the problem. Would it cause a problem if both are defined in the same resource file? I can't see how it should, but I'm no expert.

[edit] I've attached all the program files to do with the program (some are a little unnecessary, but I've included them anyway). The program is a little rough round the edges at the moment, but I was planning on cleaning it up when I had something which worked as I wanted it to. [/edit]

Code from resource.rc (cant upload it):

Code:

/* This is the main resource file which is used in opensource tictactoe */
#include "resource.h"

/* images.rc which contains the IDs for the image files used */
ID_BMP_CROSS BITMAP "cross.bmp"
ID_BMP_NAUGHT BITMAP "naught.bmp"

IDD_MENU MENU
BEGIN
    POPUP "File"
    BEGIN
        MENUITEM "New Game", ID_FILE_NEW
        MENUITEM SEPARATOR
        MENUITEM "Exit", ID_FILE_EXIT
    END
    POPUP "About"
    BEGIN
         MENUITEM "About Opensource TicTacToe", ID_ABOUT_ABOUT
    END
END

The part of interest is the WM_CREATE case Callback prodecure. It is there that the LoadBitmap() functions are called.