Update of patch #4885 (project freeciv): Status: Works For Me => In Progress
_______________________________________________________ Follow-up Comment #1: Seems that this needs better logic for EFT_DEFEND_BONUS, as otherwise Barracks III becomes redundant due to Airport. Looking at the relevant AI code, it seems the AI already thinks this is the case, so perhaps it makes sense to sort the AI first, and then use the same logic the AI will use for redundancy. I presume similar guidance ought apply for EFT_UNIT_RECOVER, EFT_VETERAN_BUILD, EFT_VETERAN_COMBAT, EFT_HP_REGEN, and EFT_UPKEEP_FACTOR, in that an improvement providing any of these effects should only be considered redundant if it fails to provide a bonus for any unit (considering class, type, and flags). If I've missed something here, please correct me, as otherwise I'll be using this list as the set for which unit checking is required. _______________________________________________________ Reply to this item at: <http://gna.org/patch/?4885> _______________________________________________ Message sent via/by Gna! http://gna.org/ _______________________________________________ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev