← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/ai_for_mods into lp:widelands

 

My only concern is division of integers and consequent rounding

Dňa ne 22. 1. 2017, 21:52 GunChleoc <fios@xxxxxxxxxxxxxxxxxxx> napísal(a):

> Actually, the divisions by 100 cancel each other out in
>
>     final += (attack * health) / (defense * evade);
>
> so I have removed them. Less code is good code :)
>
> @bunnybot merge
> --
>
> https://code.launchpad.net/~widelands-dev/widelands/ai_for_mods/+merge/315311
> You are reviewing the proposed merge of
> lp:~widelands-dev/widelands/ai_for_mods into lp:widelands.
>

-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_for_mods/+merge/315311
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_for_mods.


References