C++ Pointers

[Naming is hard]

Hey, heres a peice of code that wont compile the error i get is,

"'minus' was not declared in this scope"

// pointer to functions
#include <iostream>
using namespace std;

int addition (int a, int b)
{ return (a+b); }

int subtraction (int a, int b)
{ return (a-b); }
int (*minus)(int,int) = subtraction;

int operation (int x, int y, int (*functocall)(int,int))
{
  int g;
  g = (*functocall)(x,y);
  return (g);
}

int main ()
{
  int m,n;
  m = operation (7, 5, addition);
  n = operation (20, m, minus);
  cout <<n;
  return 0;
}

Minus should be global...shouldn't it?

If I change:

int main ()
{
  int m,n;
  m = operation (7, 5, addition);
  n = operation (20, m, minus);
  cout <<n;
  return 0;
}

To:

int main ()
{
  int m,n;
  m = operation (7, 5, addition);
  n = operation (20, m, subtraction);
  cout <<n;
  return 0;
}

obivouly it works but thats not the point >_< why isn't minus global?

btw i got the code from a tutorial located at

http://www.cplusplus.com/doc/tutorial/pointers.html

[jcl]

Compiler and platform? Builds here with Visual C++ and almost with G++.

[Naming is hard]

Im using Dev-C++, and i tried in Code::Blocks

[jcl]

Ah. Try renaming minus and see what happens.

[Naming is hard]

Ah, that worked, dont get it but it worked.

[iccaros]

it looks like gcc is saying (code blocks and dev-c++ use minigw right which if I remember is gcc on windows)

that minus is already a funtion in the std namespace

this is the error I get

 also declared as `template<class _Tp> struct std::minus' here

so instead of using namespace std;

change it to using std::cout;

this way you won't pull in the entire namespace just to shortcut cout..

[jcl]

it looks like gcc is saying (code blocks and dev-c++ use minigw right which if I remember is gcc on windows)

that minus is already a funtion in the std namespace

Indeed. It's an STL (Standard Template Library) struct defining a functor (object that acts like a function).

so instead of using namespace std;

change it to using std::cout;

Or qualify the reference to minus in main as ::minus to indicate that you want the one in the global namespace.

[Naming is hard]

ahhh, alright thanks