tresinformal / drakkar

The tresinformal video game called 'Drakkar'

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Refactor: game::get_v_player -> game::get_players

richelbilderbeek opened this issue · comments

Currently, the game has a member function called get_v_player that returns the players, where v stands for vector.
However, NL.5: Avoid encoding type information in names, hence this should be simply renamed to get_players.

To transition, both function names will be temporarily supported:

  #ifdef FIX_ISSUE_662
  {
    const game g;
    const auto players{g.get_players()};
    const auto v_player{g.get_v_player()};
    assert(players == v_player);
  }
  #endif // FIX_ISSUE_662

If this is done, go to #663