widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #10053
Re: [Merge] lp:~widelands-dev/widelands/fh1_width_and_mapobject_messages into lp:widelands
Found it - when buildings send player messages, the representative image is grabbed for its width, and that's done without player color. So, I changed the hash for the cashing in the animation manager.
I could have just changed the call in [1], but I figured that changing the hash is safer, although it comes at the cost of more memory usage. If you think that's code safety overkill, I'll be happy to do it the other way around.
[1] http://bazaar.launchpad.net/~widelands-dev/widelands/fh1_width_and_mapobject_messages/view/head:/src/logic/map_objects/tribes/building.cc#L743
--
https://code.launchpad.net/~widelands-dev/widelands/fh1_width_and_mapobject_messages/+merge/318189
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/fh1_width_and_mapobject_messages.
References