empire/empserver
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

(ask_olist): Let land units attack as long as they have positive

Browse source 
mobility, except for high-mobility terrain (mountains), where the
rules remain as they were: land units need to have all the mobility
charged for the attack, not counting combat and moving in to occupy.
Rationale: Making sure your land units reach attack positions with
enough mobility left is a pain in the neck.  Requiring only positive
mobility is friendlier, but allows rushing of mountains.
This commit is contained in:
Markus Armbruster 2006-06-17 14:28:59 +00:00
parent df6f365a71
commit 2673a258fe
1 changed files with 23 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

22
src/lib/subs/attsub.c
View file

@ -1037,14 +1037,30 @@ ask_olist(int combat_mode, struct combat *off, struct combat *def,
}
switch (combat_mode) {
case A_ATTACK:
mobcost = lnd_pathcost(&land,
att_mobcost(off->own, def,
lnd_mobtype(&land)));
/*
* We used to let land units attack only if they have the
* mobility consumed by the attack, not counting combat
* and moving in to occupy. Making sure your land units
* reach attack positions with enough mobility left is a
* pain in the neck. We now require positive mobility,
* just like for marching. Except we don't allow rushing
* of high-mobility sectors (mountains): for those we
* still require attack mobility.
*/
mobcost = att_mobcost(off->own, def, lnd_mobtype(&land));
if (mobcost < 1.0) {
if (land.lnd_mobil <= 0) {
pr("%s is out of mobility\n", prland(&land));
continue;
}
} else {
mobcost = lnd_pathcost(&land, mobcost);
if (land.lnd_mobil < mobcost) {
pr("%s does not have enough mobility (%d needed)\n",
prland(&land), (int)ceil(mobcost));
continue;
}
}
break;
case A_ASSAULT:
case A_BOARD: