Why does this header file not get processed properly?

Asked Jul 24 '13 at 09:30

Active Jul 24 '13 at 10:00

Viewed 107 times

I have a file, in which I include a header file. However on compilation I am getting lots of was not declared in this scope errors, which is odd as I declare/include everything I need in the header file. Here is the header file

#ifndef INPUT_H
#define INPUT_H
#include <map>
#include <functional>
#include <SDL/SDL.h>
using namespace std;
multimap <SDL_EventType, function<void(SDL_Event&)>> functions;

template <typename T>
void registerInput(SDL_EventType type, T&& functionPointer);
#endif

And here is the source file:

#include "input.h"
template <typename T>
void registerInput(SDL_EventType type, T&& functionPointer){
    functions.insert(pair<SDL_EventType, function<void(SDL_Event&)> >(type, functionPointer));
}

The compile error:

input.cpp:3:20: error: ‘SDL_EventType’ was not declared in this scope.

What am I doing wrong?

edited Jul 24 '13 at 09:52

Daniel Daranas

22,454
9
63
116

asked Jul 24 '13 at 09:30

w4etwetewtwet

1,330
2
10
20

Show the compile errors. – Daniel Daranas Jul 24 '13 at 09:31
@DanielDaranas input.cpp:3:20: error: ‘SDL_EventType’ was not declared in this scope. Which is odd because SDL_EventTYpe is declared in SDL/SDL.h which I include in input.h – w4etwetewtwet Jul 24 '13 at 09:32
2

and no "using namespace" in a header! – alexbuisson Jul 24 '13 at 09:37
@alexbuisson Ah, I didn't know that was a bad idea, I'll remove it. – w4etwetewtwet Jul 24 '13 at 09:38
@alexbuisson Out of interest why is it a bad idea? – w4etwetewtwet Jul 24 '13 at 09:40
2

Won't solve the problem, but I think you need to put the definition of `registerInput` in the header file, or you won't be able to call it, since it's a template function, and template functions are only instantiated when they are called (by most compilers at least). – Mats Petersson Jul 24 '13 at 09:44
@handuel, Agree it's not the "Answer" to the question, that's why it's just a comment, but before saying "Out of interest " Ask Google or read http://stackoverflow.com/questions/5849457/using-namespace-in-c-headers to known why it's not a good idea to put 'using namespace' in header file! – alexbuisson Jul 24 '13 at 09:46
2

@alexbuisson "Out of interest" does not mean "This has no interest". It means "I'm asking because I'm interested in this subject." – Daniel Daranas Jul 24 '13 at 09:54
1

DanielDaranas Thank you, Sorry @handuel, I'm not enough skilled in English and sometime I misunderstand. – alexbuisson Jul 24 '13 at 10:05
1

@alexbuisson It is fine. Thank you for helping me. – w4etwetewtwet Jul 24 '13 at 10:15
@MatsPetersson Thankyou, that helps with this issue, however, as you stated, the problem still remains with other functions etc. – w4etwetewtwet Jul 24 '13 at 10:21
For some reason the file is not included! I added tons of errors to the header, and they aren't picked up, the compiler doesn't process them. – w4etwetewtwet Jul 24 '13 at 10:23
1

Got it! I accidentally had a precompiled version of the header in the directory. – w4etwetewtwet Jul 24 '13 at 10:24
1

precompiled headers - I turn it off whenever possible! – Mats Petersson Jul 24 '13 at 10:25
@MatsPetersson It was from a mistyped compilation command. – w4etwetewtwet Jul 24 '13 at 10:27

Why does this header file not get processed properly?

0 Answers0