Originally Posted by
Elysia
You are trying to pass a temporary to the constructor which doesn't accept temporaries.
Is the Link going to modify the object? If not, then make it take const God&. If it's going to modify the object, you have to construct it first:
God blah(...);
Link(blah, ...);
But why are you creating a linked list in the first place? Why not use std::list?
Furthermore, you should be using smart pointers, eg:
std::shared_ptr<Link> norse_gods(God("Odin", "Norse", "Eight legged horse"), 0, 0);
Also, your names are bloody poor:
string n, string m, string v = "", string w = ""
What is n? What is m? What is v? And what the heck is w!?!?!?!?!?!!!????!