> Well you're right we do an assignment too much! I commented out the asignment
> in Clone() as it is already done in the new() call! Thanks for pointing me to
> this! I also tried to optimize the new() call for clone a bit.

Just my two cents:

Clone() _always_ should look like

        SomeThing * SomeThing::Clone() const
        {
                return new SomeThing(*this);
        }

and everything else should be handled by the copy constructor.

Andre'

-- 
André Pönitz ............................................. [EMAIL PROTECTED]
C++-Programmierer gesucht ... Naeheres unter http://mathematik.htwm.de/job

Reply via email to