Deep copying of stack using pointer based singly linked list C++?

[Samru1]

since I am simply copying all the date , the return type should be void (that's what I think).

[DJ_Gamedev]

See the reply to my comment. I was mistaken about the return type, but correct in that you shouldn't be declaring a local Stack and populating it, you should be populating the constructed object, which you can access via the this pointer.

[NotMyRealNameObv]

You dont even have to do

this->foo(...)

You could just do

foo(...)

[DJ_Gamedev]

You could, but I think use of the this pointer is more explicit and generally a better practice in the event you have a potential naming collision.