← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/casern_workersqueue into lp:widelands

 

Gun pinged me to take a look at this.

First impression: Looking over the travis output there is clearly a bug in the code somewhere. It is flaky as it sometimes succeeds and sometimes fails. There is no correlation to the compiler (gcc/clang both are flaky) or the type of build (debug/release).

It is always the new casern test that is failing - but always at the very end. The test runs to completion: "All Tests passed.", but then the game seems to crash. I'll bet it is a segfault, since there is no further output. If I should guess, I would say there is invalid memory or double free in the shutdown code somewhere, but I did not investigate the code yet.
-- 
https://code.launchpad.net/~widelands-dev/widelands/casern_workersqueue/+merge/309763
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/casern_workersqueue into lp:widelands.


References