Quote:ScreenObjectsContainer.hpp:28: error: variable or field `render' declared void

#ifndef SCREENOBJECTSCONTAINER_HPP
#define SCREENOBJECTSCONTAINER_HPP

#include <iostream>
#include <list>

#include "ScreenObject.hpp"

using namespace std;

namespace client
{
    class Client;
    class GraphicModule;

    class ScreenObjectsContainer
    {
        typedef list<ScreenObject*> Screen_List;

        public:
            ScreenObjectsContainer  (Client *new_client, GraphicModule *new_graphic_module);
            ~ScreenObjectsContainer ();

            void register_item  (ScreenObject *new_scr_object);
            void remove_item    (ScreenObject *scr_object, int x_pos, int y_pos);
            void fire_action    (int x_pos, int y_pos);
            void arm_item       (int x_pos, int y_pos);
            void render         (client::GraphicModule::module_state new_state); //HERE is the enum as a parameter

        private:
            GraphicModule   *module;
            Client          *client;

            int screen_width;
            int screen_height;
            bool system_highlighted;

            Screen_List curr_list;
    };
}
#endif

Quote:GraphicModule.cpp:149: error: no matching function for call to `client::ScreenObjectsContainer::render(client::GraphicModule::module_state)'

Quote:Original post by Dancin_Fool
Well since you're using forward declaration, the compiler has no idea what's inside your GraphicModule, you would have to include the header where the enum is define, or define it in a seperate header.

Quote:Original post by JohnBolton
The problem is that at the line with the error, the compiler doesn't know what a GraphicModule::module_state is. Presumably, you have defined the enum in a header file named "GraphicModule.h". You should include that header file so that the compiler knows what a GraphicModule::module_state is when it compiles this header file.

Quote:Original post by JohnBolton
A couple more things you should do:

1. Do not include iostream, nothing in this header file uses it (unless I missed something).
2. You can forward-declare ScreenObject instead of including the header file.
   

Quote:Original post by JohnBolton

Do not put using namespace xxx; in a header outside of any scope. It leads to problems. Even though it may seem inconvenient, it is better to prefix the namespace everywhere in the header file (as in typedef list<ScreenObject*> Screen_List;).