Follow-up Comment #6, patch #3846 (project wesnoth):

No, I didn't notice that. From what Crab_ said, I understood that patch such
as the first of mine (and probably ayne's patch?) is duplication of the
carryover code to other places, and it should be handled just there to avoid
maintenance problems.

Judging by the code, I think that we already rely on one of those tags being
non-empty. For example:
-transfer_all_to (gamestatus.cpp) checks save_id or id
-teaminfo::read does the same

However, if there are so many ways to just set the side id, then what would
you suggest instead?

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?3846>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Wesnoth-bugs mailing list
Wesnoth-bugs@gna.org
https://mail.gna.org/listinfo/wesnoth-bugs

Reply via email to