Code review comment for lp://staging/~widelands-dev/widelands/game_end_summary

Revision history for this message
cghislai (charlyghislain) wrote :

I went for an enum for the result (won/lost/resigned).
I also changed the report_result function to only take 3 arguments : player, result, and the info string.
I added some comment to show allowed values for that string, and parsed some of them in the game_summary screen.

« Back to merge proposal