Quote:Original post by Hodgman
I'm guessing that you're trying to have one constructor call another one?

What's happening is that your code is (almost) creating a temporary Wrapper object inside the constructor.*** Source Snippet Removed ***In C++, if you want two (or more) constructors to share some code, you've got to do it yourself. e.g.*** Source Snippet Removed ***Or in this case you can merge them into one constructor using default arguments:*** Source Snippet Removed ***

Quote:
on your last source bit, can you not use Wrapper(Cfg *cfg = new Cfg());?

I'm not sure about the limitations of default arguments as I've only ever used them for primitive data types.

#include <iostream>int main(){    int(i);    i = 10;    std::cout << i << '\n';}

#include <iostream>int main(){    int i;    (int (i));    i = 10;    std::cout << i << '\n';}

Quote:
I think it's actually interpreting that as a c-style cast.

(int)x;

std::vector<x> vec(std::istream_iterator<x>(fin), std::istream_iterator<x>());//that's a function declaration!! same as:std::vector<x> vec(std::istream_iterator<x> fin, std::istream_iterator<x>);

Quote:
You can create an unnamed temporary "Foo();" for example.

Quote:Original post by Burnt_Fyr

Quote:
on your last source bit, can you not use Wrapper(Cfg *cfg = new Cfg());?

I'm not sure about the limitations of default arguments as I've only ever used them for primitive data types.

you could, but that would making pasing a Cfg instance impossible, because it would be overwritten by the new operator. The code as originally written allows passing a Cfg object, or a null pointer. The constructor will use the object if passed, or create a default one if it receives the null pointer.