Review: Approve review

Yep, this is basically my fix, will not compile or test this one, though.

See 
http://bazaar.launchpad.net/~widelands-dev/widelands/bug_1730204-crash/revision/8496

I added this code as well:

void BuildingWindow::die() {
    assert(!is_dying_); // Building Window must not die() twice

is this reasonable?

-- 
https://code.launchpad.net/~widelands-dev/widelands/fix_invalid_memory_access_on_dismantle/+merge/334428
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/fix_invalid_memory_access_on_dismantle.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to