Code review comment for lp:~widelands-dev/widelands/memory_leak

Revision history for this message
SirVer (sirver) wrote :

> Have you tested that this doesn't regress https://bugs.launchpad.net/widelands/+bug/1522564?

No, I did not. It very likely regresses that as blitrect_scale does not change the source rect when cropping. I'll look into that, but I'd like to get https://code.launchpad.net/~widelands-dev/widelands/fix_editor_transparency_issue/+merge/282874 in before that, since it touches on the same code and I do not want to deal with the conflicts.

« Back to merge proposal