Icon Trouble

I am following the Forger's tutorial and have a problem with adding my icon, the error is only in two lines and i think i know where it is i just don't know how to get rid.

Code:

---

hIcon = LoadImage(NULL, "menu_one.ico", IMAGE_ICON, 32, 32, LR_LOADFROMFILE);

hIconSm = LoadImage(NULL, "menu_one.ico", IMAGE_ICON, 16, 16, LR_LOADFROMFILE);

---

The error is the same in both lines and it says:
29 untitled18.cpp
ANSI C++ forbids implicit conversion from `void *' in assignment

I am using Dev C++

In C++, you will have to use a cast. This is because LoadImage can return objects of several types (HICON, HBITMAP) so returns a void*. In C++ (but not C), a cast is required to go from a void* to another type. The Forger tutorials are written for C so there will be a few minor changes needed to compile with C++.

Code:

---

hIcon = (HICON) LoadImage(NULL, "menu_one.ico", IMAGE_ICON, 32, 32, LR_LOADFROMFILE);

hIconSm = (HICON) LoadImage(NULL, "menu_one.ico", IMAGE_ICON, 16, 16, LR_LOADFROMFILE);

---

Thanks

When you are writing for Windows after you have the code for creating the window, is everything else done in the WinProc area? (obviously apart from changing things like window size)

Generally speaking, yes. Unless you're doing something funky with multiple threads, then nearly all of the code you're gonna write for a GUI program will occur in or branch from a window procedure. ;)

I now have a problem creating a dialogue the error is in:

Code:

---

GROUPBOX        "About this program...",IDC_STATIC,7,7,225,52

---

oh it is a parse error

You need to define IDC_STATIC after your includes (in the .rc file):

Code:

---

#include <windows.h>
#define IDC_STATIC -1

---

MAKEINTRESOURCE(IDD_ABOUT), hwnd, AboutDlgProc);

I nwo have a problewm with that line: parse error before ), it doesnt work in C or C++

That means that you don't have IDD_ABOUT defined. You should have a "resource.h" file with the ID defines in it. This file is included in both the rc file and the .c or .cpp file.

Ok i thought I had tried define IDD_ABOUT

#include <windows.h>
#define IDC_STATIC -1
#define IDD_ABOUT

that is my dialog header file and that is included in the main file

Any help?

You're not supposed to bump threads.

Quote:

---

You should have a "resource.h" file with the ID defines in it. This file is included in both the rc file and the .c or .cpp file.

---

Do you?

Quote:

---

Code:

---

MAKEINTRESOURCE(IDD_ABOUT), hwnd, AboutDlgProc);

---

I nwo have a problewm with that line: parse error before ), it doesnt work in C or C++

---

Of course not. You need this:

Code:

---

MAKEINTRESOURCE(IDD_ABOUT /* nothing here */, hwnd, AboutDlgProc);

---

Bump Threads?

And what do i do about my problem because I don't know what that is that you just showed me

Quote:

---

Bump Threads?

---

Yeah, you're not supposed to say anything like "Anyone know the answer?" just to put your thread higher on the list.

Quote:

---

And what do i do about my problem because I don't know what that is that you just showed me

---

This closing parenthesis

Code:

---

MAKEINTRESOURCE(IDD_ABOUT), hwnd, AboutDlgProc);

---

should not be there. Get rid of it and see what happens.

dwks, the complete line (which bumfluff should have posted) will look something like this:

Code:

---

DialogBox(hInstance, MAKEINTRESOURCE(IDD_ABOUT), hwnd, AboutDlgProc);

---

so the closing bracket for MAKEINTRESOURCE is correct.

The problem is that IDD_ABOUT has not been defined correctly. It must be given a value:

Code:

---

#define IDD_ABOUT 1

---