views:

107

answers:

2

Hi,

I have the following member of class foo.

foo &foo::bar()
{
   return this;
}

But I am getting compiler errors. What stupid thing am I doing wrong?

Compiler error (gcc): error: invalid initialization of non-const reference of type 'foo&' from a temporary of type 'foo* const'

+10  A: 

this is a pointer. So it should be return *this;

Naveen
Thanks I should know that :)
Thomas
+3  A: 

As Naveen points out, you need to return *this.

Just a quick tip though: a way to figure out what somewhat obscure compiler errors mean is to try compiling on a different compiler to see if there is a better message. For example, you can use Comeau online.

In this case it gives:

"ComeauTest.c", line 7: error: initial value of reference to non-const must be an
          lvalue
     return this;
            ^

Not sure it's better in this case - but in some cases the messages are way better.

JRL
+1 for the quick tip. It can be really clarifying.
Francesco
Michael Burr