> 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

Reply via email to