C++ Resource Problems?

_whelp

New Member
I tried to compile a .cpp file, but there were errors in the resource file.

What is wrong with it?

Code:
#include "resource.h"

IDR_MAINMENU MENU {
    POPUP "&File" {
        MENUITEM "E&xit", ID_FILE_EXIT
    }
}
The errors were:

error: expected unqualified-id before numeric constant
error: expected `,' or `;' before numeric constant
[Those are only in line 3...]
 
JASS:
#include "resource.h"

IDR_MAINMENU MENU {
    POPUP "&File" {
        MENUITEM "E&xit", ID_FILE_EXIT;
    }
}


?
 

Samael88

Evil always finds a way
It is a constant. The error is stating that it want the semicolon before the constant:p

Code:
#include "resource.h"

IDR_MAINMENU MENU {
    POPUP "&File" {
        MENUITEM "E&xit"; ID_FILE_EXIT
    }
}
I am not sure if this is meant to be like that at all:eek:

All I can find when googling it is this:
Code:
IDR_MYMENU MENU
BEGIN
     POPUP "&File"
     BEGIN
          MENUITEM "E&xit", ID_FILE_EXIT
     END
And that is part of a header file.
Are you sure that it is supposed to be done that way?
 

_whelp

New Member
The way you found in google doesn't work in my compiler, so I use {}...

Even with the ; there, it doesn't work.
 

_whelp

New Member
I was actually using that tutorial while making it, but it still doesn't work even if the code is like almost exactly the same...
 

Samael88

Evil always finds a way
The tutorial says windows:) It is windows.h they are using dude;)

Code:
case WM_CREATE:
    {
        HMENU hMenu, hSubMenu;
        HICON hIcon, hIconSm;

        hMenu = CreateMenu();

        hSubMenu = CreatePopupMenu();
        AppendMenu(hSubMenu, MF_STRING, ID_FILE_EXIT, "E&xit");
        AppendMenu(hMenu, MF_STRING | MF_POPUP, (UINT)hSubMenu, "&File");

        hSubMenu = CreatePopupMenu();
        AppendMenu(hSubMenu, MF_STRING, ID_STUFF_GO, "&Go");
        AppendMenu(hMenu, MF_STRING | MF_POPUP, (UINT)hSubMenu, "&Stuff");

        SetMenu(hwnd, hMenu);


        hIcon = LoadImage(NULL, "menu_two.ico", IMAGE_ICON, 32, 32, LR_LOADFROMFILE);
        if(hIcon)
            SendMessage(hwnd, WM_SETICON, ICON_BIG, (LPARAM)hIcon);
        else
            MessageBox(hwnd, "Could not load large icon!", "Error", MB_OK | MB_ICONERROR);

        hIconSm = LoadImage(NULL, "menu_two.ico", IMAGE_ICON, 16, 16, LR_LOADFROMFILE);
        if(hIconSm)
            SendMessage(hwnd, WM_SETICON, ICON_SMALL, (LPARAM)hIconSm);
        else
            MessageBox(hwnd, "Could not load small icon!", "Error", MB_OK | MB_ICONERROR);
    }
    break;
This is the code you want to use:)

Code:
#include "resource.h"

IDR_MYMENU MENU
BEGIN
    POPUP "&File"
    BEGIN
        MENUITEM "E&xit", ID_FILE_EXIT
    END

    POPUP "&Stuff"
    BEGIN
        MENUITEM "&Go", ID_STUFF_GO
        MENUITEM "G&o somewhere else", 0, GRAYED
    END
END

IDI_MYICON ICON "menu_one.ico"
This is what goes into a file called ".rc", I am not familiar with that particular file type, I think it is a configuration file of some sort tho.
It is supposed to be included into the project:
"You will want to add the .rc file to your project or makefile depending on what tools you are using. "'

Edit: The problem with that tutorial is that it does not have the whole code:( It is much easier when you can download and take a look at everything yourself.
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • The Helper The Helper:
    I am not complaining though Hurricanes are not fun I have been through alot of them
  • Blackveiled Blackveiled:
    I evacuated for it just incase since I live in La Porte pretty much right on the water (in Houston area). Didn't see anything. It's a good thing for me anyways.
  • Ghan Ghan:
    It's too bad that Blizzard messed up so terribly with Reforged.
  • Ghan Ghan:
    Still has a 0.6 user metacritic score.
  • Ghan Ghan:
    (Out of 10)
  • Varine Varine:
    I haven't even tried it yet because everyone said it sucked. How do you get .6 out of ten?
  • The Helper The Helper:
    I am waiting for Dwarf Fortress to come out on Steam then I am on it.
  • Accname Accname:
    I play Dwarf Fortress from time to time.
  • Accname Accname:
    Its Okay.
  • Accname Accname:
    But the performance is sub-par. Does not seem to be well optimized.
  • Varine Varine:
    Is the Steam version coming with a graphical overlay or is it using the ASCII graphics still?
  • tom_mai78101 tom_mai78101:
    Steam version has the graphical overlay.
  • The Helper The Helper:
    Steam Version has graphics, a new interface and will be integrated in the steam mod stuff
  • Blackveiled Blackveiled:
    I didn't think WC3 Reforged was that bad. It pretty much did what it stated (to me at least), but then again I can care less about reforged campaigns and all that. I just care about gameplay.
  • Varine Varine:
    I think people were hoping for a resurgence in mod support. I have little interest in the actual game tbh
  • Varine Varine:
    Plus I know they added that Dota precedent clause where Blizzard can steal your shit which isn't cool.
  • Ghan Ghan:
    They sold Reforged to the community as primarily a huge graphical uplift with high res textures and redone cinematics.
  • Ghan Ghan:
    They even showed one of the redone cinematics at Blizzcon and that very cinematic wasn't even in the released game.
  • Ghan Ghan:
    So I think folks were angry about that.
  • Varine Varine:
    Didn't the backwards compatability have issues too?
  • Accname Accname:
    I never had any hopes for that. Blizzard is not the same company it was 10 years ago.
  • The Helper The Helper:
    It happens in corporations. They just absorb the companies they buy and it is not about the love of making awesome games it is how much money can we make with the least amount of cost.
  • The Helper The Helper:
    Blizzard is watered down now hopefully they can pull it together
  • The Helper The Helper:
    they got a server engineer job opening :)

    Members online

    No members online now.

    Affiliates

    Hive Workshop
    Top