[Commits] [wesnoth/wesnoth] cd91cc: Restrict singleton checks in video setters to the ...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: cd91cc224c66d1963bd9e0f24e1a4cda0482b3dd
      https://github.com/wesnoth/wesnoth/commit/cd91cc224c66d1963bd9e0f24e1a4cda0482b3dd
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2016-01-14 (Thu, 14 Jan 2016)

  Changed paths:
    M src/video.cpp

  Log Message:
  -----------
  Restrict singleton checks in video setters to the singleton callers only


  Commit: b7c507453c9b4e4a6549c3bb74c7fe78aff60902
      https://github.com/wesnoth/wesnoth/commit/b7c507453c9b4e4a6549c3bb74c7fe78aff60902
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2016-01-14 (Thu, 14 Jan 2016)

  Changed paths:
    M src/attack_prediction_display.cpp

  Log Message:
  -----------
  Use (U+00D7) instead of * to indicate multiples in damage calculation dialog


  Commit: ddd2a6a96c71cc7aa125b8fab39c0a45346de8c2
      https://github.com/wesnoth/wesnoth/commit/ddd2a6a96c71cc7aa125b8fab39c0a45346de8c2
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2016-01-14 (Thu, 14 Jan 2016)

  Changed paths:
    M src/game_launcher.cpp

  Log Message:
  -----------
  Set stored res values directly when starting game with -r

Previously, passing -r called video.set_resolution, where a display::get_singleton check
would only pass the config setter block. However, with recent refactors, a display singleton
no longer exists in areas such as the titlescreen and the check had to be removed. Rather
than add a third bool argument to video.set_resolution, I save time and set the prefs values
directly.


Compare: https://github.com/wesnoth/wesnoth/compare/d98b74d3ad7c...ddd2a6a96c71