I think that is an improvement. A better fix would be to change the m_animations vector to contain unique_ptr<> instead of raw pointers. The destructor would then not be needed.
If you do not want to do that, could you add a TODO next to m_animations in the .h file? -- https://code.launchpad.net/~widelands-dev/widelands/bug1438611/+merge/273449 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug1438611. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
