> Thanks for the fix! > > Looks good to me - can you please replace UI::RenderedText::Shared with > std::shared_ptr<UI::RenderedText> and get rid of the new using statement? I > think that would make the code easier to read.
Sure, I can do that. -- https://code.launchpad.net/~flegu/widelands/r8481-renderedtext-memory-leaks/+merge/333578 Your team Widelands Developers is requested to review the proposed merge of lp:~flegu/widelands/r8481-renderedtext-memory-leaks into lp:widelands. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp