widelands-dev team mailing list archive

Thread
Date

[Merge] lp:~widelands-dev/widelands/clang5-warnings-destructors into lp:widelands

To: mp+334286@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Thu, 30 Nov 2017 08:05:06 -0000
In-reply-to: <151170938461.13225.14455635854859608333.launchpad@ackee.canonical.com>
Reply-to: mp+334286@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

The proposal to merge lp:~widelands-dev/widelands/clang5-warnings-destructors into lp:widelands has been updated.

Commit Message changed to:

Fixed diverse new warnings by clang 5.

- -Wunused-lambda-capture
- -Wzero-as-null-pointer-constant
- Missing destructor overrides
- std::max with 0

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/clang5-warnings-destructors/+merge/334286
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/clang5-warnings-destructors into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/clang5-warnings-destructors into lp:widelands
From: GunChleoc, 2017-11-26