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