[Wesnoth-bugs] [patch #1046] unit map improvements

Update of patch #1046 (project wesnoth): Open/Closed:Open => Closed ___ Reply to this item at: ___ Nachricht

[Wesnoth-bugs] [patch #1046] unit map improvements

Update of patch #1046 (project wesnoth): Status:None => Done ___ Reply to this item at: ___ Message se

[Wesnoth-bugs] [patch #1046] unit map improvements

Follow-up Comment #6, patch #1046 (project wesnoth): So I had changed the ids to use a counter. But then you have the problem that you must save that counter value when the game is saved, otherwise you will get id clashes after loading. So, for now, I went back to the random padding. This patch

[Wesnoth-bugs] [patch #1046] unit map improvements

Update of patch #1046 (project wesnoth): Assigned to:None => boucman ___ Reply to this item at: ___ Message se

[Wesnoth-bugs] [patch #1046] unit map improvements

Follow-up Comment #5, patch #1046 (project wesnoth): Hey, I requested three different types of 'accessors', I was all fine with one type of iterator. ;) But an iterator is basically the same thing with additional feature++. And yes, it was needed. Now it just needs to be used / advertised.

[Wesnoth-bugs] [patch #1046] unit map improvements

Follow-up Comment #4, patch #1046 (project wesnoth): And a bit more... It would also be possible to do this just with a multimap * > > implemented basically the same. The downside to this would be that it doesn't allow fast lookup by unit_id, and I think it would be useful to add the ability to

[Wesnoth-bugs] [patch #1046] unit map improvements

Follow-up Comment #3, patch #1046 (project wesnoth): Yeah, I was going back and forth between having it set the id or not. Also, I have changed both the underlying id generation and the unit_id generation(in the unit_map) to use a counter rather than the random number generator. Just makes more s

[Wesnoth-bugs] [patch #1046] unit map improvements

Follow-up Comment #2, patch #1046 (project wesnoth): ok, some comments * generate_underlying_id() could directly set underlying_id_ instead of returning it... * there are two orthogonal changes here, one is the generate_underlying_id() which is good, the second is a bit more complicated, an nee

[Wesnoth-bugs] [patch #1046] unit map improvements

Follow-up Comment #1, patch #1046 (project wesnoth): to clarify, i append the two random numbers. This second patch cleans up the unit map code some more. Only changes from the previous are in unit_map.hpp and the changes are only to the iterator_counter. (file #4159) __

[Wesnoth-bugs] [patch #1046] unit map improvements

URL: Summary: unit map improvements Project: Battle for Wesnoth Submitted by: cjhopman Submitted on: Tuesday 04/08/2008 at 20:23 Priority: 5 - Normal Status: None