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

Reply via email to