← Back to team overview

widelands-dev team mailing list archive

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