[Commits] [wesnoth/wesnoth-old] 337fe6: Rename some variables to reduce confusion.

GitHub noreply at github.com
Sun Feb 23 03:48:44 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth-old
  Commit: 337fe6535cd76846538babbbbc8dd39c3324c7b6
      https://github.com/wesnoth/wesnoth-old/commit/337fe6535cd76846538babbbbc8dd39c3324c7b6
  Author: JaMiT <jt_coding at zoho.com>
  Date:   2014-02-22 (Sat, 22 Feb 2014)

  Changed paths:
    M src/display.cpp

  Log Message:
  -----------
  Rename some variables to reduce confusion.

The variable 'n' was declared within a scope that had already
declared 'n'. While each variable is only used in the line in which
it is defined (they are loop control variables for 'for' statements),
it's just bad style to have that potential confusion.

Plus, 'n' is not a descriptive name. (Not that the new names are
much better.)


  Commit: 46b87250cd7a12bdae451846bc66551d5673f1a0
      https://github.com/wesnoth/wesnoth-old/commit/46b87250cd7a12bdae451846bc66551d5673f1a0
  Author: JaMiT <jt_coding at zoho.com>
  Date:   2014-02-22 (Sat, 22 Feb 2014)

  Changed paths:
    A data/core/images/terrain/fog/fog-all.png
    M src/display.cpp

  Log Message:
  -----------
  Better support of removing fog from a single hex.

Fixes bug #21643, although artists might find room for improvement.


Compare: https://github.com/wesnoth/wesnoth-old/compare/b995ae5003f7...46b87250cd7a


More information about the Commits mailing list