Code review comment for lp:~widelands-dev/widelands/bug-1823612-llvm8-build-errors

Revision history for this message
GunChleoc (gunchleoc) wrote :

I have added a TODO to the AI code.

I have verified that no_stats is not affected by the bugfix in the production programs. I don't know if we ever entered the buggy state, but if we did, this will actually fix the stats: If duration was not set for these programs, Widelands would have picked a duration between 0 and 3 ms, depending on the ID of the current program state (Failed, Skipped, Completed..). Now, the duration is always 0 ms like it should be.

« Back to merge proposal