← Back to team overview

widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/bug-1542238-log into lp:widelands

To: mp+285237@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Sat, 06 Feb 2016 08:48:00 -0000
In-reply-to: <20160205203124.9089.69159.launchpad@ackee.canonical.com>
Reply-to: mp+285237@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

LGTM, but I think we need some log output in the classes that call this function as well, in case the non-matching Immovable == nullptr.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1542238-log/+merge/285237
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1542238-log into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/bug-1542238-log into lp:widelands
From: TiborB, 2016-02-05