widelands-dev team mailing list archive

[ypopezios <ypopezios@xxxxxxxx>]

ypopezios has proposed merging lp:~widelands-dev/widelands/previous_song_exclusion into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #1735938 in widelands: "Random song selection should exclude the previous song"
  https://bugs.launchpad.net/widelands/+bug/1735938

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/previous_song_exclusion/+merge/347404

- Changed current_song_ from iterator into simple index.
- Changed calculation of random song to exclude current_song_ from the possible results.
- Unified handling of random and linear selection.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/previous_song_exclusion into lp:widelands.