[wesnoth-commits] [wesnoth/wesnoth] 596eda: Some schema code reformatting as requested by @cel...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 596edaac0053d62c0beba49685a8a34029980e67
      https://github.com/wesnoth/wesnoth/commit/596edaac0053d62c0beba49685a8a34029980e67
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2018-03-05 (Mon, 05 Mar 2018)

  Changed paths:
    M src/serialization/schema_validator.cpp
    M src/serialization/schema_validator.hpp
    M src/serialization/tag.cpp
    M src/serialization/tag.hpp

  Log Message:
  -----------
  Some schema code reformatting as requested by @celticminstrel


  Commit: 8309ba63c1215fcb3823083edb5100ed94c6288f
      https://github.com/wesnoth/wesnoth/commit/8309ba63c1215fcb3823083edb5100ed94c6288f
  Author: Charles Dang <exodia339 at gmail.com>
  Date:   2018-03-05 (Mon, 05 Mar 2018)

  Changed paths:
    M src/serialization/schema_validator.cpp
    M src/serialization/tag.cpp
    M src/serialization/tag.hpp

  Log Message:
  -----------
  Serialization/Schema: deployed range-for and auto

Also removed a bunch of unnecessary typedefs:
* the value_type typedefs weren't since you can do map::value_type (granted, that gives you
  a const version, but they weren't used anywhere anyway)
* iterator typedefs weren't needed since we have range-for now. No need to construct a pair
  of iterators and then use an increment-for loop to iterate over the container.


Compare: https://github.com/wesnoth/wesnoth/compare/579bbc464ff0...8309ba63c121