Code review comment for lp:~flegu/widelands/fix_some_memory_leaks

Thanks for cleaning up after me :)

Merging this with bunnybot is no problem.

I have added you to widelands-dev, so you we can work together on branches in the future. Just don't push directly to trunk :)

I think we can have a std::unique_ptr here for all the variables that you touched. It is clear who owns it, so we don't need shared_ptr, and raw pointers are something we want to get rid of.

The only raw pointers that are OK are for UI::Panel and its subclasses, because the pointers are managed well within UI::Panel and it would be a huge job to refactor that.