Code review comment for lp:~widelands-dev/widelands/terrain_affinity_as_int

Revision history for this message
GunChleoc (gunchleoc) wrote :

Thanks for the excellent review :)

I have addressed most of your comments and left replies to the rest.

One of our math wizards spent a long time tweaking the system and the function, so I'm hesitant to change any semantics here. There is an e function involved, so there is no way to avoid floating point completely unless we radically change the system.

« Back to merge proposal