widelands

Merge lp://qastaging/~widelands-dev/widelands/bug-1831583-fri02 into lp://qastaging/widelands

  1. bug-1831583-fri02
  2. Merge into trunk
Proposed by Benedikt Straub
Status: Merged
Merged at revision: 9142
Proposed branch: lp://qastaging/~widelands-dev/widelands/bug-1831583-fri02
Merge into: lp://qastaging/widelands
Diff against target: 31 lines (+4/-2)
1 file modified
data/campaigns/fri02.wmf/scripting/mission_thread.lua (+4/-2)
To merge this branch: bzr merge lp://qastaging/~widelands-dev/widelands/bug-1831583-fri02
Reviewer Review Type Date Requested Status
hessenfarmer Approve
GunChleoc Approve
Review via email: mp+368348@code.qastaging.launchpad.net

Commit message

Add some sleep time to costly loops in fri02 to prevent freezing

Description of the change

This is a Lua problem. gdb shows that masses of new Lua objects are created, and Lua doesn´t care to release the memory again. Hopefully, some additional sleep() will give the game and the garbage collector time to catch up.

I don´t want to fix this by increasing the base sleep time or the objective will be marked done only after an enormous delay.
Another way to fix the bug while perhaps improving performance would be to move the checks to C++, but I feel bad about having such specialised functions there.

To post a comment you must log in.
Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 5152. State: errored. Details: https://travis-ci.org/widelands/widelands/builds/541383420.
Appveyor build 4934. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1831583_fri02-4934.

Revision history for this message
GunChleoc (gunchleoc) wrote :

We have these sleep times a lot in our Lua scripts for checking loops, so let's have it :)

@bunnybot merge

review: Approve
Revision history for this message
bunnybot (widelandsofficial) wrote :

Refusing to merge, since Travis is not green. Use @bunnybot merge force for merging anyways.

Travis build 5152. State: errored. Details: https://travis-ci.org/widelands/widelands/builds/541383420.

Revision history for this message
hessenfarmer (stephan-lutz) wrote :

regarding these objectives a 1 second delay does not matter at all. for me even 2 or 3 seconds would be hardly recognizable by the player. so we could even go up with the sleep if necessary.

Revision history for this message
hessenfarmer (stephan-lutz) wrote :

Wow 3 times the inputqueues faiklure in one build seems to be a new record.

@bunnybot merge force

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: