]> git.pond.sub.org Git - empserver/blobdiff - src/lib/subs/attsub.c
Update copyright notice.
[empserver] / src / lib / subs / attsub.c
index 29f39059b600a5c63fa72e23c81b560bb5157637..45028d5bdc7063120b6b65941661c9789aaa5c31 100644 (file)
@@ -1,6 +1,6 @@
 /*
  *  Empire - A multi-player, client/server Internet based war game.
- *  Copyright (C) 1986-2006, Dave Pare, Jeff Bailey, Thomas Ruschak,
+ *  Copyright (C) 1986-2007, Dave Pare, Jeff Bailey, Thomas Ruschak,
  *                           Ken Stevens, Steve McClure
  *
  *  This program is free software; you can redistribute it and/or modify
  *  Known contributors to this file:
  *     Ken Stevens, 1995
  *     Steve McClure, 1996-2000
+ *     Markus Armbruster, 2006
  */
 
 #include <config.h>
 
+#include <ctype.h>
 #include <math.h>
-#include "misc.h"
-#include "player.h"
+#include "combat.h"
 #include "file.h"
-#include "plague.h"
-#include "sect.h"
-#include "ship.h"
-#include "path.h"
-#include "news.h"
-#include "treaty.h"
-#include "nat.h"
-#include "xy.h"
-#include "land.h"
-#include "nsc.h"
+#include "map.h"
+#include "misc.h"
 #include "mission.h"
-#include "combat.h"
-#include "item.h"
+#include "nsc.h"
 #include "optlist.h"
+#include "path.h"
+#include "plague.h"
+#include "player.h"
 #include "prototypes.h"
+#include "xy.h"
+#include "empobj.h"
+#include "unit.h"
 
 #define CASUALTY_LUMP  1       /* How big casualty chunks should be */
 
 static void ask_olist(int combat_mode, struct combat *off,
                      struct combat *def, struct emp_qelem *olist,
-                     s_char *land_answer, int *a_spyp, int *a_engineerp);
-static void take_move_in_mob(int combat_mode, struct llist *llp,
+                     char *land_answer, int *a_spyp, int *a_engineerp);
+static void take_move_in_mob(int combat_mode, struct ulist *llp,
                             struct combat *off, struct combat *def);
 static void move_in_land(int combat_mode, struct combat *off,
                         struct emp_qelem *olist, struct combat *def);
@@ -71,12 +69,13 @@ static int board_abort(struct combat *off, struct combat *def);
 static int land_board_abort(struct combat *off, struct combat *def);
 static int ask_off(int combat_mode, struct combat *off,
                   struct combat *def);
-static int get_dlist(struct combat *def, struct emp_qelem *list, int a_spy,
-                    int *d_spyp);
+static void get_dlist(struct combat *def, struct emp_qelem *list, int a_spy,
+                     int *d_spyp);
 static int get_ototal(int combat_mode, struct combat *off,
                      struct emp_qelem *olist, double osupport, int check);
 static int get_dtotal(struct combat *def, struct emp_qelem *list,
                      double dsupport, int check);
+static double att_calcodds(int, int);
 static int take_casualty(int combat_mode, struct combat *off,
                         struct emp_qelem *olist);
 
@@ -85,7 +84,7 @@ static int take_def(int combat_mode, struct emp_qelem *list,
                    struct combat *off, struct combat *def);
 
 static int get_land(int combat_mode, struct combat *def, int uid,
-                   struct llist *llp, int victim_land);
+                   struct ulist *llp, int victim_land);
 
 char *att_mode[] = {
     /* must match combat types in combat.h */
@@ -102,7 +101,7 @@ char *att_mode[] = {
  * the type of the object is determined by combat->type which can take the
  * values EF_SECTOR, EF_SHIP, EF_PLANE, or EF_BAD.  Another important parameter
  * which is often passed to these functions is combat_mode.  This can take
- * the value A_DEFENSE, A_ATTACK, A_ASSAULT, A_PARA, A_BOARD and A_LBOARD.
+ * the value A_DEFEND, A_ATTACK, A_ASSAULT, A_PARA, A_BOARD and A_LBOARD.
  * As these six modes of being in combat affect things like mobcost and combat
  * value, there are often switches made on combat_mode.  Note that in all cases
  * no mobility is taken from sectors, ships, or land units until the player
@@ -338,9 +337,9 @@ put_combat(struct combat *com)
        sect.sct_type = com->sct_type;
        deff = sect.sct_effic - com->eff;
        if (deff > 0) {
-           sect.sct_road -= (sect.sct_road * deff / 100.0);
-           sect.sct_rail -= (sect.sct_rail * deff / 100.0);
-           sect.sct_defense -= (sect.sct_defense * deff / 100.0);
+           sect.sct_road -= sect.sct_road * deff / 100.0;
+           sect.sct_rail -= sect.sct_rail * deff / 100.0;
+           sect.sct_defense -= sect.sct_defense * deff / 100.0;
            if (sect.sct_road <= 0)
                sect.sct_road = 0;
            if (sect.sct_rail <= 0)
@@ -349,8 +348,6 @@ put_combat(struct combat *com)
                sect.sct_defense = 0;
        }
        sect.sct_effic = com->eff;
-       if (!opt_DEFENSE_INFRA)
-           sect.sct_defense = sect.sct_effic;
        if (com->mobcost) {
            if (opt_MOB_ACCESS) {
                if ((com->mob - com->mobcost) < -127)
@@ -382,13 +379,13 @@ put_combat(struct combat *com)
            if (com->mob - com->mobcost < -127)
                land.lnd_mobil = -127;
            else
-               land.lnd_mobil = (s_char)(com->mob - com->mobcost);
+               land.lnd_mobil = (signed char)(com->mob - com->mobcost);
        }
-       makelost(EF_LAND, land.lnd_own, land.lnd_uid, land.lnd_x,
-                land.lnd_y);
+       makelost(EF_LAND, land.lnd_own, land.lnd_uid,
+                land.lnd_x, land.lnd_y);
        land.lnd_own = com->own;
-       makenotlost(EF_LAND, land.lnd_own, land.lnd_uid, land.lnd_x,
-                   land.lnd_y);
+       makenotlost(EF_LAND, land.lnd_own, land.lnd_uid,
+                   land.lnd_x, land.lnd_y);
        if (com->plague) {
            if (land.lnd_pstage == PLG_HEALTHY)
                land.lnd_pstage = PLG_EXPOSED;
@@ -410,13 +407,13 @@ put_combat(struct combat *com)
            if (com->mob - com->mobcost < -127)
                ship.shp_mobil = -127;
            else
-               ship.shp_mobil = (s_char)(com->mob - com->mobcost);
+               ship.shp_mobil = (signed char)(com->mob - com->mobcost);
        }
-       makelost(EF_SHIP, ship.shp_own, ship.shp_uid, ship.shp_x,
-                ship.shp_y);
+       makelost(EF_SHIP, ship.shp_own, ship.shp_uid,
+                ship.shp_x, ship.shp_y);
        ship.shp_own = com->own;
-       makenotlost(EF_SHIP, ship.shp_own, ship.shp_uid, ship.shp_x,
-                   ship.shp_y);
+       makenotlost(EF_SHIP, ship.shp_own, ship.shp_uid,
+                   ship.shp_x, ship.shp_y);
        if (com->plague) {
            if (ship.shp_pstage == PLG_HEALTHY)
                ship.shp_pstage = PLG_EXPOSED;
@@ -515,9 +512,9 @@ att_abort(int combat_mode, struct combat *off, struct combat *def)
        return land_board_abort(off, def);
     }
 
-    if (off && def->sct_dcp->d_mcst <= 0) {
-       pr("You can't %s a %s sector!\n", att_mode[combat_mode],
-          def->sct_dcp->d_name);
+    if (off && def->sct_dcp->d_mob0 < 0) {
+       pr("You can't %s a %s sector!\n",
+          att_mode[combat_mode], def->sct_dcp->d_name);
        return abort_attack();
     }
     if (!off || off->relations_checked)
@@ -766,7 +763,7 @@ att_ask_offense(int combat_mode, struct combat *off, struct combat *def,
                struct emp_qelem *olist, int *a_spyp, int *a_engineerp)
 {
     int n;
-    s_char land_answer[1024];
+    char land_answer[256];
 
     emp_initque(olist);
     if (att_abort(combat_mode, off, def))
@@ -784,22 +781,51 @@ att_ask_offense(int combat_mode, struct combat *off, struct combat *def,
     return 0;
 }
 
+/*
+ * Return path cost for ATTACKER to enter sector given by DEF.
+ * MOBTYPE is a mobility type accepted by sector_mcost().
+ */
+static double
+att_mobcost(natid attacker, struct combat *def, int mobtype)
+{
+    struct sctstr sect;
+    int ok;
+
+    if (CANT_HAPPEN(def->type != EF_SECTOR))
+       return -1.0;
+    ok = getsect(def->x, def->y, &sect);
+    if (CANT_HAPPEN(!ok))
+       return -1.0;
+
+    /*
+     * We want the cost to move/march into the sector.  If we just
+     * called sector_mcost(), we'd get the defender's cost.  The
+     * attacker's cost is higher unless he's the old-owner.  Note: if
+     * there are no civilians, a victorious attacker will become the
+     * old-owner.  But he isn't now.
+     */
+    sect.sct_own = attacker;
+    sect.sct_mobil = 0;
+    return sector_mcost(&sect, mobtype);
+}
+
 /* How many mil is off allowed to attack with when it attacks def? */
 
 static int
 get_mob_support(int combat_mode, struct combat *off, struct combat *def)
 {
     int mob_support;
+    double mobcost;
 
     switch (combat_mode) {
     case A_ATTACK:
-       mob_support = off->mob / sector_mcost(getsectp(def->x, def->y),
-                                             MOB_ROAD);
-       if (mob_support < 0)
-           mob_support = 0;
-/*             mob_support = off->mob / sector_mcost(def->sct_type, def->eff);*/
+       mobcost = att_mobcost(off->own, def, MOB_MOVE);
+       if (mobcost < 0 || off->mob <= 0)
+           return 0;
+       mob_support = off->mob / mobcost;
        if (mob_support < off->troops)
-           pr("Sector %s has %d mobility which can only support %d mil,\n", xyas(off->x, off->y, player->cnum), off->mob, mob_support);
+           pr("Sector %s has %d mobility which can only support %d mil,\n",
+              xyas(off->x, off->y, player->cnum), off->mob, mob_support);
        else
            mob_support = off->troops;
        return mob_support;
@@ -852,10 +878,9 @@ calc_mobcost(int combat_mode, struct combat *off, struct combat *def,
        return;
     switch (combat_mode) {
     case A_ATTACK:
-       off->mobcost +=
-           MAX(1,
-               (int)(attacking_mil *
-                     sector_mcost(getsectp(def->x, def->y), MOB_ROAD)));
+       off->mobcost += MAX(1,
+                           (int)(attacking_mil
+                                 * att_mobcost(off->own, def, MOB_MOVE)));
        break;
     case A_LBOARD:
        off->mobcost += MAX(1, attacking_mil / 5);
@@ -920,9 +945,9 @@ static char
 att_prompt(char *prompt, char army)
 {
     char buf[1024];
-    char *p = buf;
+    char *p;
 
-    if (army == ' ')
+    if (!army)
        army = '~';
     for (;;) {
        p = getstring(prompt, buf);
@@ -947,15 +972,15 @@ att_prompt(char *prompt, char army)
 
 static void
 ask_olist(int combat_mode, struct combat *off, struct combat *def,
-         struct emp_qelem *olist, s_char *land_answer, int *a_spyp,
+         struct emp_qelem *olist, char *land_answer, int *a_spyp,
          int *a_engineerp)
 {
     struct nstr_item ni;
     struct lndstr land;
     double mobcost;
-    struct llist *llp;
+    struct ulist *llp;
     struct lchrstr *lcp;
-    int att_val;
+    double att_val;
     int count = 0;
     int maxland = 0;
     int first_time = 1;
@@ -1010,12 +1035,29 @@ ask_olist(int combat_mode, struct combat *off, struct combat *def,
        }
        switch (combat_mode) {
        case A_ATTACK:
-           mobcost =
-               lnd_mobcost(&land, getsectp(def->x, def->y), MOB_NONE);
-           if (land.lnd_mobil < mobcost) {
-               pr("%s does not have enough mobility (%d needed)\n",
-                  prland(&land), (int)ceil(mobcost));
-               continue;
+           /*
+            * 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:
@@ -1023,9 +1065,13 @@ ask_olist(int combat_mode, struct combat *off, struct combat *def,
            mobcost = 0;
            if (!(lcp->l_flags & L_ASSAULT))
                continue;
+           break;
+       default:
+           CANT_REACH();
+           return;
        }
        att_val = attack_val(combat_mode, &land);
-       if (!att_val) {
+       if (att_val < 1.0) {
            pr("%s has no offensive strength\n", prland(&land));
            continue;
        }
@@ -1040,7 +1086,7 @@ ask_olist(int combat_mode, struct combat *off, struct combat *def,
            first_time = 0;
            pr("You may board with a maximum of %d land units\n", maxland);
        }
-       pr("%s has a base %s value of %d\n",
+       pr("%s has a base %s value of %.0f\n",
           prland(&land), att_mode[combat_mode], att_val);
        if (land_answer[(int)land.lnd_army] != 'Y') {
            sprintf(prompt,
@@ -1048,7 +1094,7 @@ ask_olist(int combat_mode, struct combat *off, struct combat *def,
                    att_mode[combat_mode],
                    prland(&land),
                    prcom(1, off),
-                   land.lnd_army == ' ' ? '~' : land.lnd_army,
+                   land.lnd_army ? land.lnd_army : '~',
                    land.lnd_effic);
            land_answer[(int)land.lnd_army] =
                att_prompt(prompt, land.lnd_army);
@@ -1058,19 +1104,19 @@ ask_olist(int combat_mode, struct combat *off, struct combat *def,
                land_answer[(int)land.lnd_army] != 'Y')
                continue;
        }
-       if (!(llp = malloc(sizeof(struct llist)))) {
+       if (!(llp = malloc(sizeof(struct ulist)))) {
            logerror("Malloc failed in attack!\n");
            abort_attack();
            return;
        }
-       memset(llp, 0, sizeof(struct llist));
+       memset(llp, 0, sizeof(struct ulist));
        emp_insque(&llp->queue, olist);
        llp->mobil = mobcost;
        if (!get_land(combat_mode, def, land.lnd_uid, llp, 0))
            continue;
        if (lnd_spyval(&land) > *a_spyp)
            *a_spyp = lnd_spyval(&land);
-       if (llp->lcp->l_flags & L_ENGINEER)
+       if (((struct lchrstr *)llp->chrp)->l_flags & L_ENGINEER)
            ++*a_engineerp;
        if (def->type == EF_SHIP && ++count >= maxland)
            break;
@@ -1093,27 +1139,18 @@ att_combat_eff(struct combat *com)
            eff = 1.0 + ((str - 1.0) * eff);
        } else
            eff = sector_strength(getsectp(com->x, com->y));
-/*                     str = com->sct_dcp->d_dstr;*/
     } else if (com->type == EF_SHIP && com->own != player->cnum) {
        getship(com->shp_uid, &ship);
-       eff = (1.0 + ship.shp_armor / 100.0);
+       eff = 1.0 + ship.shp_armor / 100.0;
     }
     return eff;
 }
 
-/*
- * Estimate the defense strength and give the attacker a chance to abort
- * if the odds are less than 50%
- */
-
 int
-att_estimate_defense(int combat_mode, struct combat *off,
-                    struct emp_qelem *olist, struct combat *def,
-                    int a_spy)
+att_get_offense(int combat_mode, struct combat *off,
+               struct emp_qelem *olist, struct combat *def)
 {
     int ototal;
-    int estimate;
-    int odds;
 
     /*
      * Get the attacker units & mil again in case they changed while the
@@ -1126,30 +1163,6 @@ att_estimate_defense(int combat_mode, struct combat *off,
     if (combat_mode == A_PARA)
        return ototal;
     pr("\n             Initial attack strength: %8d\n", ototal);
-
-    estimate = att_combat_eff(def) * roundintby(def->troops, 10);
-    estimate += att_combat_eff(def) * get_dlist(def, 0, a_spy, 0);
-
-    /*
-     * Calculate the initial (pre-support) attack odds.  If they're less
-     * than 50%, ask for a confirmation.
-     */
-
-    odds = (int)(att_calcodds(ototal, estimate) * 100);
-
-/*
-       if (odds < 50) {
-               pr("          Estimated defense strength: %8d\n", estimate);
-               pr("                      Estimated odds: %8d%%\n\n",odds);
-               sprintf(prompt, "Are you sure you want to %s [yn]? ",
-                       att_mode[combat_mode]);
-               if (!confirm(prompt))
-                       return abort_attack();
-               ototal = get_ototal(combat_mode, off, olist,1.0,1);
-               if (att_empty_attack(combat_mode, ototal, def))
-                       return abort_attack();
-       }
- */
     return ototal;
 }
 
@@ -1160,7 +1173,7 @@ att_get_defense(struct emp_qelem *olist, struct combat *def,
 {
     int d_spy = 0;
     struct emp_qelem *qp;
-    struct llist *llp;
+    struct ulist *llp;
     int dtotal;
     int old_dtotal;
 
@@ -1176,8 +1189,8 @@ att_get_defense(struct emp_qelem *olist, struct combat *def,
        att_reacting_units(def, dlist, a_spy, &d_spy, ototal);
 
     for (qp = olist->q_forw; qp != olist; qp = qp->q_forw) {
-       llp = (struct llist *)qp;
-       intelligence_report(def->own, &llp->land, d_spy,
+       llp = (struct ulist *)qp;
+       intelligence_report(def->own, &llp->unit.land, d_spy,
                            "Scouts report attacking unit:");
     }
 
@@ -1191,14 +1204,13 @@ att_get_defense(struct emp_qelem *olist, struct combat *def,
 
 /* Get the defensive land units in the sector or on the ship */
 
-static int
+static void
 get_dlist(struct combat *def, struct emp_qelem *list, int a_spy,
          int *d_spyp)
 {
     struct nstr_item ni;
-    struct llist *llp;
+    struct ulist *llp;
     struct lndstr land;
-    int estimate = 0;
 
 /* In here is where you need to take out spies and trains from the defending
    lists.  Spies try to hide, trains get trapped and can be boarded. */
@@ -1218,17 +1230,16 @@ get_dlist(struct combat *def, struct emp_qelem *list, int a_spy,
        if (def->type == EF_LAND && land.lnd_land != def->lnd_uid)
            continue;
        if (!list) {            /* Just estimating the enemy strength */
-           estimate += intelligence_report(player->cnum,
-                                           &land, a_spy,
-                                           "Scouts report defending unit:");
+           intelligence_report(player->cnum, &land, a_spy,
+                               "Scouts report defending unit:");
            continue;
        }
-       if (!(llp = malloc(sizeof(struct llist)))) {
+       if (!(llp = malloc(sizeof(struct ulist)))) {
            logerror("Malloc failed in attack!\n");
            abort_attack();
-           return 0;
+           return;
        }
-       memset(llp, 0, sizeof(struct llist));
+       memset(llp, 0, sizeof(struct ulist));
        emp_insque(&llp->queue, list);
        llp->supplied = has_supply(&land);
        if (!get_land(A_DEFEND, def, land.lnd_uid, llp, 1))
@@ -1236,7 +1247,6 @@ get_dlist(struct combat *def, struct emp_qelem *list, int a_spy,
        if (lnd_spyval(&land) > *d_spyp)
            *d_spyp = lnd_spyval(&land);
     }
-    return estimate;
 }
 
 /* Calculate the total offensive strength */
@@ -1247,7 +1257,7 @@ get_ototal(int combat_mode, struct combat *off, struct emp_qelem *olist,
 {
     double ototal = 0.0;
     struct emp_qelem *qp, *next;
-    struct llist *llp;
+    struct ulist *llp;
     int n, w;
 
     /*
@@ -1268,26 +1278,26 @@ get_ototal(int combat_mode, struct combat *off, struct emp_qelem *olist,
 
     for (qp = olist->q_forw; qp != olist; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
-       if (check && !get_land(combat_mode, 0, llp->land.lnd_uid, llp, 0))
+       llp = (struct ulist *)qp;
+       if (check && !get_land(combat_mode, 0, llp->unit.land.lnd_uid, llp, 0))
            continue;
        if (combat_mode == A_ATTACK) {
            w = -1;
            for (n = 0; n <= off->last; ++n) {
                if (off[n].type == EF_BAD)
                    continue;
-               if ((off[n].x == llp->land.lnd_x) &&
-                   (off[n].y == llp->land.lnd_y))
+               if ((off[n].x == llp->unit.land.lnd_x) &&
+                   (off[n].y == llp->unit.land.lnd_y))
                    w = n;
            }
            if (w < 0) {
                lnd_delete(llp, "is in a sector not owned by you");
                continue;
            }
-           ototal += attack_val(combat_mode, &llp->land) *
+           ototal += attack_val(combat_mode, &llp->unit.land) *
                att_combat_eff(off + w);
        } else {
-           ototal += attack_val(combat_mode, &llp->land);
+           ototal += attack_val(combat_mode, &llp->unit.land);
        }
     }
     ototal *= osupport;
@@ -1303,7 +1313,7 @@ get_dtotal(struct combat *def, struct emp_qelem *list, double dsupport,
 {
     double dtotal = 0.0, eff = 1.0, d_unit;
     struct emp_qelem *qp, *next;
-    struct llist *llp;
+    struct ulist *llp;
 
     if (check && att_get_combat(def, 1) < 0)
        return 0;
@@ -1317,13 +1327,12 @@ get_dtotal(struct combat *def, struct emp_qelem *list, double dsupport,
 
     for (qp = list->q_forw; qp != list; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
-       d_unit = 0.0;
-       if (check && !get_land(A_DEFEND, def, llp->land.lnd_uid, llp, 1))
+       llp = (struct ulist *)qp;
+       if (check && !get_land(A_DEFEND, def, llp->unit.land.lnd_uid, llp, 1))
            continue;
-       d_unit = defense_val(&llp->land);
+       d_unit = defense_val(&llp->unit.land);
        if (!llp->supplied)
-           d_unit = ((double)defense_val(&llp->land) / 2.0);
+           d_unit /= 2.0;
        dtotal += d_unit * eff;
     }
 
@@ -1338,18 +1347,18 @@ get_dtotal(struct combat *def, struct emp_qelem *list, double dsupport,
  */
 
 static int
-get_land(int combat_mode, struct combat *def, int uid, struct llist *llp,
+get_land(int combat_mode, struct combat *def, int uid, struct ulist *llp,
         int victim_land)
 {
-    struct lndstr *lp = &llp->land;
+    struct lndstr *lp = &llp->unit.land;
     char buf[512];
 
     getland(uid, lp);
 
-    if (!llp->lcp) {           /* first time */
-       llp->x = llp->land.lnd_x;
-       llp->y = llp->land.lnd_y;
-       llp->lcp = &lchr[(int)llp->land.lnd_type];
+    if (!llp->chrp) {          /* first time */
+       llp->x = llp->unit.land.lnd_x;
+       llp->y = llp->unit.land.lnd_y;
+       llp->chrp = (struct empobj_chr *)&lchr[(int)llp->unit.land.lnd_type];
     } else {                   /* not first time */
        if (lp->lnd_effic < LAND_MINEFF) {
            sprintf(buf, "was destroyed and is no longer a part of the %s",
@@ -1385,7 +1394,7 @@ get_land(int combat_mode, struct combat *def, int uid, struct llist *llp,
            }
        }
     }
-    llp->eff = llp->land.lnd_effic;
+    llp->eff = llp->unit.land.lnd_effic;
 
     return 1;
 }
@@ -1401,13 +1410,13 @@ static void
 kill_land(struct emp_qelem *list)
 {
     struct emp_qelem *qp, *next;
-    struct llist *llp;
+    struct ulist *llp;
 
     for (qp = list->q_forw; qp != list; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
-       if (llp->land.lnd_ship >= 0) {
-           llp->land.lnd_effic = 0;
+       llp = (struct ulist *)qp;
+       if (llp->unit.land.lnd_ship >= 0) {
+           llp->unit.land.lnd_effic = 0;
            lnd_delete(llp, "cannot return to the ship, and dies!");
        }
     }
@@ -1417,15 +1426,15 @@ static void
 att_infect_units(struct emp_qelem *list, int plague)
 {
     struct emp_qelem *qp, *next;
-    struct llist *llp;
+    struct ulist *llp;
 
     if (!plague)
        return;
     for (qp = list->q_forw; qp != list; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
-       if (llp->land.lnd_pstage == PLG_HEALTHY)
-           llp->land.lnd_pstage = PLG_EXPOSED;
+       llp = (struct ulist *)qp;
+       if (llp->unit.land.lnd_pstage == PLG_HEALTHY)
+           llp->unit.land.lnd_pstage = PLG_EXPOSED;
     }
 }
 
@@ -1433,21 +1442,21 @@ static void
 put_land(struct emp_qelem *list)
 {
     struct emp_qelem *qp, *next;
-    struct llist *llp;
+    struct ulist *llp;
 
     for (qp = list->q_forw; qp != list; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
-       llp->land.lnd_mission = 0;
-       llp->land.lnd_harden = 0;
-       llp->land.lnd_mobil -= (int)llp->mobil;
+       llp = (struct ulist *)qp;
+       llp->unit.land.lnd_mission = 0;
+       llp->unit.land.lnd_harden = 0;
+       llp->unit.land.lnd_mobil -= (int)llp->mobil;
        llp->mobil = 0.0;
-       putland(llp->land.lnd_uid, &llp->land);
-       if (llp->land.lnd_own != player->cnum) {
+       putland(llp->unit.land.lnd_uid, &llp->unit.land);
+       if (llp->unit.land.lnd_own != player->cnum) {
            emp_remque((struct emp_qelem *)llp);
            free(llp);
        } else
-           get_land(A_ATTACK, 0, llp->land.lnd_uid, llp, 0);
+           get_land(A_ATTACK, 0, llp->unit.land.lnd_uid, llp, 0);
     }
 }
 
@@ -1456,42 +1465,30 @@ put_land(struct emp_qelem *list)
  * Note that the "strength" command also calls this routine.
  */
 
-int
+double
 att_reacting_units(struct combat *def, struct emp_qelem *list, int a_spy,
                   int *d_spyp, int ototal)
 {
     struct nstr_item ni;
     struct lndstr land;
     struct sctstr sect, dsect;
-    struct llist *llp;
+    struct ulist *llp;
     int dtotal;
-    int new_land = 0;
+    double new_land = 0;
     double mobcost;
-    double move_cost;
-    int supply_state;
+    double pathcost;
     int dist;
     int radius;
     int origx, origy;
     double eff = att_combat_eff(def);
-    char *p;
     char buf[1024];
 
-    /*
-     *
-     * All units that are within their reaction radius and not damaged
-     * below their morale value now get to react to the threatened sect.
-     * Once we've sent enough to counter the threat, stop sending them.
-     *
-     * Not anymore.  All units get to react. :)
-     */
-
     if (list)
        dtotal = get_dtotal(def, list, 1.0, 1);
     else
        dtotal = 0;
     snxtitem_all(&ni, EF_LAND);
-    while (nxtitem(&ni, &land) &&
-          (dtotal + new_land * eff < (int)(1.2 * (float)ototal))) {
+    while (nxtitem(&ni, &land) && dtotal + new_land * eff < 1.2 * ototal) {
        if (!land.lnd_own)
            continue;
        if (!land.lnd_rad_max)
@@ -1502,17 +1499,15 @@ att_reacting_units(struct combat *def, struct emp_qelem *list, int a_spy,
            continue;
        if (land.lnd_ship >= 0)
            continue;
-       if (!defense_val(&land))
+       if (land.lnd_land >= 0)
+           continue;
+       if (defense_val(&land) < 1.0)
            continue;
-/*
-               if (land.lnd_effic <= land.lnd_retreat)
-                       continue;
- */
        if (!lnd_can_attack(&land))
            continue;
 
        /* Only supplied units can react */
-       if (!(supply_state = has_supply(&land)))
+       if (!has_supply(&land))
            continue;
 
        dist = mapdist(land.lnd_x, land.lnd_y, def->x, def->y);
@@ -1531,15 +1526,11 @@ att_reacting_units(struct combat *def, struct emp_qelem *list, int a_spy,
            continue;
 
        getsect(def->x, def->y, &dsect);
-       if (!(p = BestLandPath(buf, &sect, &dsect, &move_cost, MOB_ROAD)))
+       if (!BestLandPath(buf, &sect, &dsect, &pathcost,
+                         lnd_mobtype(&land)))
            continue;
 
-       mobcost = land.lnd_effic * 0.01 * lchr[(int)land.lnd_type].l_spd;
-       if (mobcost < 0.01)
-           mobcost = 0.01;
-       mobcost = 480.0 / (mobcost + techfact(land.lnd_tech, mobcost));
-       mobcost *= (move_cost * 5.0);
-
+       mobcost = lnd_pathcost(&land, pathcost);
        if (land.lnd_mobil < mobcost)
            continue;
 
@@ -1558,15 +1549,14 @@ att_reacting_units(struct combat *def, struct emp_qelem *list, int a_spy,
        wu(0, land.lnd_own, "%s reacts to %s.\n",
           prland(&land), xyas(land.lnd_x, land.lnd_y, land.lnd_own));
 
-       llp = (struct llist *)
-           malloc(sizeof(struct llist));
+       llp = malloc(sizeof(struct ulist));
 
-       memset(llp, 0, sizeof(struct llist));
-       llp->supplied = supply_state;
+       memset(llp, 0, sizeof(struct ulist));
+       llp->supplied = 1;
        llp->x = origx;
        llp->y = origy;
-       llp->lcp = &lchr[(int)land.lnd_type];
-       llp->land = land;
+       llp->chrp = (struct empobj_chr *)&lchr[(int)land.lnd_type];
+       llp->unit.land = land;
        emp_insque(&llp->queue, list);
        if (lnd_spyval(&land) > *d_spyp)
            *d_spyp = lnd_spyval(&land);
@@ -1590,25 +1580,25 @@ get_osupport(char *outs, struct combat *def, int fort_sup, int ship_sup,
     af = as = au = ap = 0.0;
     if (fort_sup) {
        dam = dd(def->own, player->cnum, def->x, def->y, 0, 0);
-       af = ((double)dam / 100.0);
+       af = dam / 100.0;
        osupport += af;
     }
     if (ship_sup) {
        dam = sd(def->own, player->cnum, def->x, def->y, 0, 0, 0);
 
-       as = ((double)dam / 100.0);
+       as = dam / 100.0;
        osupport += as;
     }
 
     if (land_sup) {
        dam = lnd_support(def->own, player->cnum, def->x, def->y, 0);
-       au = ((double)dam / 100.0);
+       au = dam / 100.0;
        osupport += au;
     }
 
     if (plane_sup) {
        dam = off_support(def->x, def->y, def->own, player->cnum);
-       ap = (((double)dam) / 100.0);
+       ap = dam / 100.0;
        osupport += ap;
     }
     sprintf(outs, "attacker\t%1.2f\t%1.2f\t%1.2f\t%1.2f\n", af, as, au,
@@ -1634,25 +1624,25 @@ get_dsupport(char *outs, struct emp_qelem *list, struct combat *def,
        good = 1;
     } else {
        dam = dd(player->cnum, def->own, def->x, def->y, 0, 1);
-       df = ((double)dam / 100.0);
+       df = dam / 100.0;
        dsupport += df;
 
        dtotal = get_dtotal(def, list, dsupport, 0);
        if (dtotal < 1.2 * ototal) {
            dam = sd(player->cnum, def->own, def->x, def->y, 0, 1, 0);
-           ds = ((double)dam / 100.0);
+           ds = dam / 100.0;
            dsupport += ds;
            dtotal = get_dtotal(def, list, dsupport, 0);
        }
        if (dtotal < 1.2 * ototal) {
            dam = lnd_support(player->cnum, def->own, def->x, def->y, 1);
-           du = ((double)dam / 100.0);
+           du = dam / 100.0;
            dsupport += du;
            dtotal = get_dtotal(def, list, dsupport, 1);
        }
        if (dtotal < 1.2 * ototal) {
            dam = def_support(def->x, def->y, player->cnum, def->own);
-           dp = (((double)dam) / 100.0);
+           dp = dam / 100.0;
            dsupport += dp;
        }
     }
@@ -1688,7 +1678,7 @@ get_mine_dsupport(struct combat *def, int a_engineer)
     if (sect.sct_oldown != player->cnum) {
        mines = MIN(sect.sct_mines, 20);
        if (a_engineer)
-           mines = ldround(((double)mines / 2.0), 1);
+           mines = ldround(mines / 2.0, 1);
        if (mines > 0) {
            if (def->own)
                wu(0, def->own, "Defending mines add %1.2f\n",
@@ -1719,8 +1709,8 @@ att_get_support(int combat_mode, int ofort, int oship, int oland,
 
     /*
      * I need to put a 1 at the end of the next four total_stren calls
-     * becauase units & mil may have been damaged by collateral damage or
-     * neclear warheads from the offensive & defensive support.
+     * because units & mil may have been damaged by collateral damage or
+     * nuclear warheads from the offensive & defensive support.
      */
 
     ototal = get_ototal(combat_mode, off, olist, *osupportp, 1);
@@ -1770,13 +1760,13 @@ count_bodies(struct combat *off, struct emp_qelem *list)
     int n;
     int bodies = 0;
     struct emp_qelem *qp;
-    struct llist *llp;
+    struct ulist *llp;
 
     for (n = 0; n <= off->last; ++n)
        bodies += off[n].troops;
     for (qp = list->q_forw; qp != list; qp = qp->q_forw) {
-       llp = (struct llist *)qp;
-       bodies += total_mil(&llp->land);
+       llp = (struct ulist *)qp;
+       bodies += llp->unit.land.lnd_item[I_MILIT];
     }
     return bodies;
 }
@@ -1801,6 +1791,7 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
     int news_item;
     int recalctime;
     double odds;
+    int newmob;
     char *action;
 
     ototal = get_ototal(combat_mode, off, olist, osupport,
@@ -1862,7 +1853,8 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
     att_infect_units(olist, off->plague);
     att_infect_units(dlist, def->plague);
 
-    /* Fighting is slightly random.  There is always that last little 
+    /*
+     * Fighting is slightly random.  There is always that last little 
      * effort you see people put in.  Or the stray bullet that takes out
      * an officer and the rest go into chaos.  Things like that.
      * Thus, we have added a very slight random factor that will sometimes
@@ -1870,8 +1862,9 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
      * (either +- 5%) to account for this randomness.  We also only
      * recalculate the odds every 8-50 casualties, not every cacsualty,
      * since a single dead guy normally wouldn't cause a commander to
-     * rethink his strategies, but 50 dead guys might. */
-    odds = odds + (double)((double)((random() % 11) - 5) / 100.0);
+     * rethink his strategies, but 50 dead guys might.
+     */
+    odds += (random() % 11 - 5) / 100.0;
     if (odds < 0.0)
        odds = 0.1;
     if (odds > 1.0)
@@ -1894,7 +1887,7 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
        if (recalctime-- <= 0) {
            recalctime = 8 + (random() % 43);
            odds = att_calcodds(ototal, dtotal);
-           odds = odds + (double)((double)((random() % 11) - 5) / 100.0);
+           odds += (random() % 11 - 5) / 100.0;
            if (odds < 0.0)
                odds = 0.1;
            if (odds > 1.0)
@@ -1907,12 +1900,12 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
        def->mil = 0;
     else {
        if (def->type == EF_SECTOR && d_mil && d_cas) {
-           int tmob;
-
-           /* Make sure we use a positive mobility here */
-           tmob = ((def->mob < 0) ? -(def->mob) : (def->mob));
-           def->mobcost =
-               MIN(20, MIN(1, tmob - damage(tmob, 100 * d_cas / d_mil)));
+           if (def->mob < 0)
+               def->mobcost = 0;
+           else {
+               newmob = damage(def->mob, 100 * d_cas / d_mil);
+               def->mobcost = MIN(20, def->mob - newmob);
+           }
        }
        def->mil = def->troops;
     }
@@ -1920,12 +1913,14 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
     /* update attack mobility & mil */
     for (n = 0; n <= off->last; ++n)
        if (off[n].type != EF_BAD && off[n].troops < a_troops[n]) {
-           if (off[n].type == EF_SECTOR && off[n].mil)
-               off[n].mobcost +=
-                   MIN(20,
-                       off[n].mob - damage(off[n].mob,
-                                           100 * (a_troops[n] - off[n].troops)
-                                           / off[n].mil));
+           if (off[n].type == EF_SECTOR && off[n].mil) {
+               if (!CANT_HAPPEN(off[n].mob < 0)) {
+                   newmob = damage(off[n].mob,
+                                   100 * (a_troops[n] - off[n].troops)
+                                   / off[n].mil);
+                   off[n].mobcost += MIN(20, off[n].mob - newmob);
+               }
+           }
            off[n].mil -= a_troops[n] - off[n].troops;
        }
 
@@ -1970,6 +1965,10 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
            pr("We have boarded %s, sir!\n", prcom(0, def));
            action = "boarding";
            break;
+       default:
+           CANT_REACH();
+           news_item = 0;
+           action = "defeating";
        }
     } else {
        switch (combat_mode) {
@@ -2001,6 +2000,10 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
            kill_land(olist);
            action = "trying to board";
            break;
+       default:
+           CANT_REACH();
+           news_item = 0;
+           action = "fighting";
        }
     }
     nreport(player->cnum, news_item, def->own, 1);
@@ -2050,7 +2053,7 @@ att_fight(int combat_mode, struct combat *off, struct emp_qelem *olist,
 
 /* What percentage of the combat forces going head-to-head are we? */
 
-double
+static double
 att_calcodds(int ototal, int dtotal)
 {
     double odds;
@@ -2061,7 +2064,7 @@ att_calcodds(int ototal, int dtotal)
     else if (dtotal <= 0)
        odds = 1.0;
     else
-       odds = ((double)ototal) / (dtotal + ototal);
+       odds = (double)ototal / (dtotal + ototal);
 
     return odds;
 }
@@ -2075,7 +2078,7 @@ take_casualty(int combat_mode, struct combat *off, struct emp_qelem *olist)
     int biggest_troops = 0, index = -1;
     int n, tot_troops = 0, biggest_mil, cas;
     struct emp_qelem *qp, *biggest;
-    struct llist *llp;
+    struct ulist *llp;
 
     for (n = 0; n <= off->last; ++n) {
        if (off[n].type != EF_BAD) {
@@ -2132,17 +2135,17 @@ take_casualty(int combat_mode, struct combat *off, struct emp_qelem *olist)
     biggest = NULL;
     biggest_mil = -1;
     for (qp = olist->q_forw; qp != olist; qp = qp->q_forw) {
-       llp = (struct llist *)qp;
+       llp = (struct ulist *)qp;
 
-       if (total_mil(&llp->land) > biggest_mil) {
-           biggest_mil = total_mil(&llp->land);
+       if (llp->unit.land.lnd_item[I_MILIT] > biggest_mil) {
+           biggest_mil = llp->unit.land.lnd_item[I_MILIT];
            biggest = qp;
        }
     }
     if (biggest == NULL)
        return CASUALTY_LUMP - to_take;
 
-    llp = (struct llist *)biggest;
+    llp = (struct ulist *)biggest;
     cas = lnd_take_casualty(combat_mode, llp, to_take);
     return CASUALTY_LUMP - (to_take - cas);
 }
@@ -2153,17 +2156,18 @@ static void
 send_reacting_units_home(struct emp_qelem *list)
 {
     struct emp_qelem *qp, *next;
-    struct llist *llp;
+    struct ulist *llp;
     char buf[1024];
 
     for (qp = list->q_forw; qp != list; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
-       if ((llp->land.lnd_x != llp->x) || (llp->land.lnd_y != llp->y)) {
+       llp = (struct ulist *)qp;
+       if ((llp->unit.land.lnd_x != llp->x) ||
+           (llp->unit.land.lnd_y != llp->y)) {
            sprintf(buf, "returns to %s",
-                   xyas(llp->x, llp->y, llp->land.lnd_own));
-           llp->land.lnd_x = llp->x;
-           llp->land.lnd_y = llp->y;
+                   xyas(llp->x, llp->y, llp->unit.land.lnd_own));
+           llp->unit.land.lnd_x = llp->x;
+           llp->unit.land.lnd_y = llp->y;
            lnd_delete(llp, buf);
        }
     }
@@ -2198,7 +2202,7 @@ take_def(int combat_mode, struct emp_qelem *list, struct combat *off,
 {
     int n;
     int occuppied = 0;
-    struct llist *llp, *delete_me = 0;
+    struct ulist *llp, *delete_me = 0;
     char buf[1024];
     struct sctstr sect;
     struct shpstr ship;
@@ -2230,16 +2234,16 @@ take_def(int combat_mode, struct emp_qelem *list, struct combat *off,
                   pr_com(2, def, def->own));
            return 0;
        } else {
-           llp = (struct llist *)list->q_forw;
-           llp->land.lnd_x = def->x;
-           llp->land.lnd_y = def->y;
+           llp = (struct ulist *)list->q_forw;
+           llp->unit.land.lnd_x = def->x;
+           llp->unit.land.lnd_y = def->y;
            take_move_in_mob(combat_mode, llp, off, def);
            if (def->type == EF_SHIP) {
-               llp->land.lnd_ship = def->shp_uid;
+               llp->unit.land.lnd_ship = def->shp_uid;
                sprintf(buf, "boards %s", prcom(0, def));
                delete_me = llp;
            } else {
-               llp->land.lnd_ship = -1;
+               llp->unit.land.lnd_ship = -1;
                sprintf(buf, "moves in to occupy %s",
                        xyas(def->x, def->y, player->cnum));
                lnd_delete(llp, buf);
@@ -2280,11 +2284,11 @@ ask_move_in(struct combat *off, struct emp_qelem *olist,
 {
     int n;
     struct emp_qelem *qp, *next;
-    struct llist *llp;
+    struct ulist *llp;
     char buf[512];
     char prompt[512];
-    s_char land_answer[1024];
-    s_char *answerp;
+    char land_answer[256];
+    char *answerp;
 
     for (n = 0; n <= off->last; ++n)
        if (off[n].type != EF_BAD && off[n].troops > 0)
@@ -2299,29 +2303,30 @@ ask_move_in(struct combat *off, struct emp_qelem *olist,
     memset(land_answer, 0, sizeof(land_answer));
     for (qp = olist->q_forw; qp != olist; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
-       answerp = &land_answer[(int)llp->land.lnd_army];
+       llp = (struct ulist *)qp;
+       answerp = &land_answer[(int)llp->unit.land.lnd_army];
        if (player->aborted || att_get_combat(def, 0) < 0)
            *answerp = 'N';
        if (*answerp == 'Y')
            continue;
        if (*answerp != 'N') {
-           if (!get_land(A_ATTACK, def, llp->land.lnd_uid, llp, 0))
+           if (!get_land(A_ATTACK, def, llp->unit.land.lnd_uid, llp, 0))
                continue;
            sprintf(prompt, "Move in with %s (%c %d%%) [ynYNq?] ",
-                   prland(&llp->land),
-                   llp->land.lnd_army == ' ' ? '~' : llp->land.lnd_army,
-                   llp->land.lnd_effic);
-           *answerp = att_prompt(prompt, llp->land.lnd_army);
+                   prland(&llp->unit.land),
+                   llp->unit.land.lnd_army ? llp->unit.land.lnd_army :  '~',
+                   llp->unit.land.lnd_effic);
+           *answerp = att_prompt(prompt, llp->unit.land.lnd_army);
            if (player->aborted || att_get_combat(def, 0) < 0)
                *answerp = 'N';
-           if (!get_land(A_ATTACK, def, llp->land.lnd_uid, llp, 0))
+           if (!get_land(A_ATTACK, def, llp->unit.land.lnd_uid, llp, 0))
                continue;
        }
        if (*answerp == 'y' || *answerp == 'Y')
            continue;
        sprintf(buf, "stays in %s",
-               xyas(llp->land.lnd_x, llp->land.lnd_y, player->cnum));
+               xyas(llp->unit.land.lnd_x, llp->unit.land.lnd_y,
+                    player->cnum));
        lnd_delete(llp, buf);
     }
     if (QEMPTY(olist))
@@ -2329,11 +2334,12 @@ ask_move_in(struct combat *off, struct emp_qelem *olist,
     if (att_get_combat(def, 0) < 0) {
        for (qp = olist->q_forw; qp != olist; qp = next) {
            next = qp->q_forw;
-           llp = (struct llist *)qp;
-           if (!get_land(A_ATTACK, def, llp->land.lnd_uid, llp, 0))
+           llp = (struct ulist *)qp;
+           if (!get_land(A_ATTACK, def, llp->unit.land.lnd_uid, llp, 0))
                continue;
            sprintf(buf, "stays in %s",
-                   xyas(llp->land.lnd_x, llp->land.lnd_y, player->cnum));
+                   xyas(llp->unit.land.lnd_x, llp->unit.land.lnd_y,
+                        player->cnum));
            lnd_delete(llp, buf);
        }
        return;
@@ -2350,23 +2356,23 @@ move_in_land(int combat_mode, struct combat *off, struct emp_qelem *olist,
             struct combat *def)
 {
     struct emp_qelem *qp, *next;
-    struct llist *llp;
+    struct ulist *llp;
     char buf[512];
 
     if (QEMPTY(olist))
        return;
     for (qp = olist->q_forw; qp != olist; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
-       if (!get_land(combat_mode, def, llp->land.lnd_uid, llp, 0))
+       llp = (struct ulist *)qp;
+       if (!get_land(combat_mode, def, llp->unit.land.lnd_uid, llp, 0))
            continue;
        take_move_in_mob(combat_mode, llp, off, def);
-       llp->land.lnd_x = def->x;
-       llp->land.lnd_y = def->y;
+       llp->unit.land.lnd_x = def->x;
+       llp->unit.land.lnd_y = def->y;
        if (def->type == EF_SHIP)
-           llp->land.lnd_ship = def->shp_uid;
+           llp->unit.land.lnd_ship = def->shp_uid;
        else
-           llp->land.lnd_ship = -1;
+           llp->unit.land.lnd_ship = -1;
     }
     if (QEMPTY(olist))
        return;
@@ -2383,7 +2389,7 @@ move_in_land(int combat_mode, struct combat *off, struct emp_qelem *olist,
        return;
     for (qp = olist->q_forw; qp != olist; qp = next) {
        next = qp->q_forw;
-       llp = (struct llist *)qp;
+       llp = (struct ulist *)qp;
        lnd_print(llp, buf);
     }
     if (QEMPTY(olist))
@@ -2460,15 +2466,13 @@ ask_move_in_off(struct combat *off, struct combat *def)
        return;
     if (off->own != player->cnum)
        return;
-    d = sector_mcost(getsectp(def->x, def->y), MOB_ROAD);
+    d = att_mobcost(off->own, def, MOB_MOVE);
     if ((mob_support = MIN(off->troops, (int)(off->mob / d))) <= 0)
        return;
     sprintf(prompt, "How many mil to move in from %s (%d max)? ",
            xyas(off->x, off->y, player->cnum), mob_support);
-    if (!(p = getstring(prompt, buf)) || !*p || (num_mil = atoi(p)) <= 0) {
-       num_mil = 0;
+    if (!(p = getstring(prompt, buf)) || !*p || (num_mil = atoi(p)) <= 0)
        return;
-    }
 /* Make sure we don't move in more than we can support mobility-wise */
     if (num_mil > mob_support)
        num_mil = mob_support;
@@ -2487,7 +2491,7 @@ ask_move_in_off(struct combat *off, struct combat *def)
     off->troops -= num_mil;
     put_combat(off);
     left = num_mil;
-    weight = num_mil * ichr[I_MILIT].i_lbs;
+    weight = (double)num_mil * ichr[I_MILIT].i_lbs;
     if (opt_INTERDICT_ATT && chance(weight / 200.0)) {
        if (chance(weight / 100.0))
            dam +=
@@ -2516,7 +2520,7 @@ ask_move_in_off(struct combat *off, struct combat *def)
 /* Charge land units for moving into a sector or onto a ship */
 
 static void
-take_move_in_mob(int combat_mode, struct llist *llp, struct combat *off,
+take_move_in_mob(int combat_mode, struct ulist *llp, struct combat *off,
                 struct combat *def)
 {
     int mobcost;
@@ -2524,36 +2528,39 @@ take_move_in_mob(int combat_mode, struct llist *llp, struct combat *off,
 
     switch (combat_mode) {
     case A_ATTACK:
-       mobcost =
-           lnd_mobcost(&llp->land, getsectp(def->x, def->y), MOB_NONE);
-       new = llp->land.lnd_mobil - mobcost;
+       mobcost = lnd_pathcost(&llp->unit.land,
+                              att_mobcost(off->own, def,
+                                          lnd_mobtype(&llp->unit.land)));
+       new = llp->unit.land.lnd_mobil - mobcost;
        if (new < -127)
            new = -127;
-       llp->land.lnd_mobil = new;
+       llp->unit.land.lnd_mobil = new;
        break;
     case A_ASSAULT:
        if (off->shp_mcp->m_flags & M_LAND) {
-           if (llp->lcp->l_flags & L_MARINE)
-               llp->land.lnd_mobil -=
+           if (((struct lchrstr *)llp->chrp)->l_flags & L_MARINE)
+               llp->unit.land.lnd_mobil -=
                    (float)etu_per_update * land_mob_scale * 0.5;
            else
-               llp->land.lnd_mobil -= (float)etu_per_update * land_mob_scale;
+               llp->unit.land.lnd_mobil -= (float)etu_per_update *
+                   land_mob_scale;
        } else {
-           if (llp->lcp->l_flags & L_MARINE)
-               llp->land.lnd_mobil = 0;
+           if (((struct lchrstr *)llp->chrp)->l_flags & L_MARINE)
+               llp->unit.land.lnd_mobil = 0;
            else
-               llp->land.lnd_mobil = -(float)etu_per_update * land_mob_scale;
+               llp->unit.land.lnd_mobil = -(float)etu_per_update *
+                   land_mob_scale;
        }
        break;
     case A_BOARD:
        /* I arbitrarily chose the numbers 10 and 40 below -KHS */
-       if (llp->lcp->l_flags & L_MARINE)
-           llp->land.lnd_mobil -= 10;
+       if (((struct lchrstr *)llp->chrp)->l_flags & L_MARINE)
+           llp->unit.land.lnd_mobil -= 10;
        else
-           llp->land.lnd_mobil -= 40;
+           llp->unit.land.lnd_mobil -= 40;
        break;
     }
-    llp->land.lnd_harden = 0;
+    llp->unit.land.lnd_harden = 0;
 }
 
 static void
@@ -2590,19 +2597,13 @@ att_free_lists(struct emp_qelem *olist, struct emp_qelem *dlist)
 double
 sector_strength(struct sctstr *sp)
 {
-    double d;
-
-    d = 1.0;
-
-    if (sp->sct_type == SCT_MOUNT)
-       d = 2.0;
-
-    d = d + ((double)(dchr[sp->sct_type].d_dstr - d) *
-            ((double)sp->sct_defense / 100.0));
+    double def = SCT_DEFENSE(sp) / 100.0;
+    double base = sp->sct_type == SCT_MOUNT ? 2.0 : 1.0;
+    double d = base + (dchr[sp->sct_type].d_dstr - base) * def;
 
     if (d > dchr[sp->sct_type].d_dstr)
        d = dchr[sp->sct_type].d_dstr;
-    if (d < 0.1)
-       d = 0.1;
+    if (d < base)
+       d = base;
     return d;
 }