The proposal to merge lp:~widelands-dev/widelands/valgrind into lp:widelands
has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/valgrind/+merge/364213
--
Your team Widelands Developers is subscribed to branch
lp:~wide
Continuous integration builds have changed state:
Travis build 4576. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/504273300.
Appveyor build 4363. State: failed. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_valgrind-43
I think I have tested this sufficiently. Thanks for the review
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/valgrind/+merge/364213
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/valgrind.
_
Review: Approve code review
Code looks fine, will not do any further tests.
--
https://code.launchpad.net/~widelands-dev/widelands/valgrind/+merge/364213
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/valgrind.
___
GunChleoc has proposed merging lp:~widelands-dev/widelands/valgrind into
lp:widelands.
Commit message:
Update Valgrind suppressions and fix uninitialized variables reported by
Valgrind Memcheck
- Uninitialized variable in default AI
- Uninitialized variables in Building Statistics
- Exclude more
5 matches
Mail list logo
