On March 20, 2015 1:55:28 PM EDT, Islam Taha wrote:

Thank you again I think it’s solved right now, you can check it

Many people read this list without the benefit of a tool that presents thread history. This message quotes nothing from the previous message, so read by itself, it is essentially meaningless. ___ Rob (Sent from my portable computation engine)

On 2015-03-20 17:55, Islam Taha wrote:

Thank you again I think it’s solved right now, you can check it

You welcome, Islam. Yes, the copy constructor seems fine (maybe you can optimize it by preallocating the space in advance). The copy assignment operator is completely broken. It does not compile, and its declaration is wrong. Please review C++ idioms in general, writing a container implies them. You can also remove all that __ before your variables since you are not implementing the standard library (yet) and I believe doing so on user code is even forbidden. When you write your tests, you should ensure every single line of the source is hit. This is no sufficient condition for your code to be "tested", but it is a necessary one. Writing (copy/move) constructors and assignment operators is tricky, they are part of the class interface, and should be tested. Best Giacomo