From b7455d3bd43e4591daccbb1a7665f0d0f0cc188c Mon Sep 17 00:00:00 2001 From: Celtic Minstrel Date: Tue, 6 Sep 2016 10:29:52 -0400 Subject: [PATCH] Remove suokko AI remnants --- src/ai/default/aspect_attacks.cpp | 17 ----------------- src/ai/default/attack.cpp | 6 ------ src/ai/default/contexts.cpp | 5 ----- 3 files changed, 28 deletions(-) diff --git a/src/ai/default/aspect_attacks.cpp b/src/ai/default/aspect_attacks.cpp index 5cd4f340a83..b240025a695 100644 --- a/src/ai/default/aspect_attacks.cpp +++ b/src/ai/default/aspect_attacks.cpp @@ -310,30 +310,13 @@ void aspect_attacks_base::do_attack_analysis( // If this is a position with equal defense to another position, // but more vulnerability then we don't want to use it. -#ifdef SUOKKO - //FIXME: this code was in sukko's r29531 Correct? - // scale vulnerability to 60 hp unit - if(cur_position >= 0 && rating < best_rating - && (vulnerability/surround_bonus*30.0)/unit_itor->second.hitpoints() - - (support*surround_bonus*30.0)/unit_itor->second.max_hitpoints() - > best_vulnerability - best_support) { - continue; - } -#else if(cur_position >= 0 && rating == best_rating && vulnerability/surround_bonus - support*surround_bonus >= best_vulnerability - best_support) { continue; } -#endif cur_position = j; best_rating = rating; -#ifdef SUOKKO - //FIXME: this code was in sukko's r29531 Correct? - best_vulnerability = (vulnerability/surround_bonus*30.0)/unit_itor->second.hitpoints(); - best_support = (support*surround_bonus*30.0)/unit_itor->second.max_hitpoints(); -#else best_vulnerability = vulnerability/surround_bonus; best_support = support*surround_bonus; -#endif } if(cur_position != -1) { diff --git a/src/ai/default/attack.cpp b/src/ai/default/attack.cpp index 644dcac863e..e59218f08e8 100644 --- a/src/ai/default/attack.cpp +++ b/src/ai/default/attack.cpp @@ -279,14 +279,8 @@ double attack_analysis::rating(double aggression, const readonly_context& ai_obj // into sub-optimal terrain. // Calculate the 'exposure' of our units to risk. -#ifdef SUOKKO - //FIXME: this code was in sukko's r29531 Correct? - const double exposure_mod = uses_leader ? ai_obj.current_team().caution()* 8.0 : ai_obj.current_team().caution() * 4.0; - const double exposure = exposure_mod*resources_used*((terrain_quality - alternative_terrain_quality)/10)*vulnerability/std::max(0.01,support); -#else const double exposure_mod = uses_leader ? 2.0 : ai_obj.get_caution(); const double exposure = exposure_mod*resources_used*(terrain_quality - alternative_terrain_quality)*vulnerability/std::max(0.01,support); -#endif LOG_AI << "attack option has base value " << value << " with exposure " << exposure << ": " << vulnerability << "/" << support << " = " << (vulnerability/std::max(support,0.1)) << "\n"; value -= exposure*(1.0-aggression); diff --git a/src/ai/default/contexts.cpp b/src/ai/default/contexts.cpp index a237a92305a..22ae1d9ba0a 100644 --- a/src/ai/default/contexts.cpp +++ b/src/ai/default/contexts.cpp @@ -164,12 +164,7 @@ std::vector default_ai_context_impl::find_targets(const move_map& enemy_ assert(threats.empty() == false); -#ifdef SUOKKO - //FIXME: suokko's revision 29531 included this change. Correct? - const double value = threat*get_protect_leader()/leader->second.hitpoints(); -#else const double value = threat/double(threats.size()); -#endif for(std::set::const_iterator i = threats.begin(); i != threats.end(); ++i) { LOG_AI << "found threat target... " << *i << " with value: " << value << "\n"; targets.push_back(target(*i,value,target::TYPE::THREAT));