Code review comment for lp:~widelands-dev/widelands/bug-1786613-10s-return-skipped

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

Ok I see your point about the stack. However your solution would need a lot of changes I the programs and additionally we would need to document the rule to implicitly avoid stats calculation by adding_work to the end of the program.
So I was thinking why not introduce a way to explicitly state that no stats update is required / foreseen for a work program. This could probably be done by defining a new return state. Alternatively I'm asking myself if we should not try to return=None or return= to avoid stats calculation without having a skipped state.

« Back to merge proposal