Follow-up Comment #6, patch #1020 (project wesnoth): I am glad to see someone tackle this idea finally.
Unfortunately, you missed the most important part of the proposal: throwing a unit_map_iterator_invalid exception when someone tries to access an invalid unit iterator. Another big head-scratcher is why you are keeping a revision number if you don't use it to avoid unnecessary work in find_pair()? As an extension to this idea, it would be nice to add support for unit_map_observers who get notified when a refresh (a.k.a. find_pair with new revision number) occurs. _______________________________________________________ Reply to this item at: <http://gna.org/patch/?1020> _______________________________________________ Message sent via/by Gna! http://gna.org/ _______________________________________________ Wesnoth-bugs mailing list Wesnoth-bugs@gna.org https://mail.gna.org/listinfo/wesnoth-bugs