]> git.pond.sub.org Git - empserver/blobdiff - src/lib/subs/plnsub.c
Clean up superfluous includes
[empserver] / src / lib / subs / plnsub.c
index 0735fba58a38a31eab6b2bfa8260c1a15eb7def5..cdc0c4828766bad864fa786108f0e4ffef7397ff 100644 (file)
@@ -1,11 +1,11 @@
 /*
  *  Empire - A multi-player, client/server Internet based war game.
- *  Copyright (C) 1986-2006, Dave Pare, Jeff Bailey, Thomas Ruschak,
- *                           Ken Stevens, Steve McClure
+ *  Copyright (C) 1986-2011, Dave Pare, Jeff Bailey, Thomas Ruschak,
+ *                Ken Stevens, Steve McClure, Markus Armbruster
  *
- *  This program is free software; you can redistribute it and/or modify
+ *  Empire is free software: you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation; either version 2 of the License, or
+ *  the Free Software Foundation, either version 3 of the License, or
  *  (at your option) any later version.
  *
  *  This program is distributed in the hope that it will be useful,
@@ -14,8 +14,7 @@
  *  GNU General Public License for more details.
  *
  *  You should have received a copy of the GNU General Public License
- *  along with this program; if not, write to the Free Software
- *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
  *
  *  ---
  *
  *  ---
  *
  *  plnsub.c: Plane subroutine stuff
- * 
+ *
  *  Known contributors to this file:
  *     Dave Pare, 1986
  *     Ken Stevens, 1995
  *     Steve McClure, 1998-2000
- *     Markus Armbruster, 2004-2005
+ *     Markus Armbruster, 2004-2010
  */
 
 #include <config.h>
 
-#include <math.h>
+#include "file.h"
+#include "item.h"
+#include "land.h"
+#include "map.h"
 #include "misc.h"
+#include "nat.h"
+#include "nsc.h"
+#include "nuke.h"
+#include "optlist.h"
+#include "plane.h"
 #include "player.h"
+#include "prototypes.h"
 #include "sect.h"
 #include "ship.h"
-#include "land.h"
-#include "item.h"
-#include "plane.h"
 #include "xy.h"
-#include "nsc.h"
-#include "file.h"
-#include "nat.h"
-#include "path.h"
-#include "prototypes.h"
-#include "optlist.h"
 
-static int pln_equip(struct plist *, struct ichrstr *, int, char);
+static int fit_plane_on_ship(struct plnstr *, struct shpstr *);
+
+/*
+ * Get planes and escorts argument.
+ * Read planes into *NI_BOMB, and (optional) escorts into *NI_ESC.
+ * If INPUT_BOMB is not empty, use it, else prompt for more input.
+ * Same for INPUT_ESC.
+ * If we got a plane argument, initialize *NI_BOMB and *NI_ESC, and
+ * return 0.
+ * Else return -1 (*NI_BOMB and *NI_ESC may be modified).
+ */
+int
+get_planes(struct nstr_item *ni_bomb, struct nstr_item *ni_esc,
+          char *input_bomb, char *input_esc)
+{
+    if (!snxtitem(ni_bomb, EF_PLANE, input_bomb, NULL))
+       return -1;
+    if (!snxtitem(ni_esc, EF_PLANE, input_esc, "escort(s)? ")) {
+       if (player->aborted)
+           return -1;
+       pr("No escorts...\n");
+    }
+    return 0;
+}
 
 /*
  * Get assembly point argument.
@@ -77,8 +99,7 @@ get_assembly_point(char *input, struct sctstr *ap_sect, char *buf)
        return NULL;
 
     /* over own or allied sector is fine */
-    if (ap_sect->sct_own == player->cnum
-       || getrel(getnatp(ap_sect->sct_own), player->cnum) == ALLIED)
+    if (relations_with(ap_sect->sct_own, player->cnum) == ALLIED)
        return ap_sect;
 
     /* over own or allied ship is fine */
@@ -86,8 +107,7 @@ get_assembly_point(char *input, struct sctstr *ap_sect, char *buf)
     while (nxtitem(&ni, &ship)) {
        if (ship.shp_effic < SHIP_MINEFF || ship.shp_own == 0)
            continue;
-       if (ship.shp_own == player->cnum
-           || getrel(getnatp(ship.shp_own), player->cnum) == ALLIED)
+       if (relations_with(ship.shp_own, player->cnum) == ALLIED)
            return ap_sect;
     }
 
@@ -100,7 +120,7 @@ pln_onewaymission(struct sctstr *target, int *shipno, int *flagp)
 {
     int nships;
     int cno;
-    int flags;
+    int flags, fl;
     struct shpstr ship;
     char buf[1024];
     char *p;
@@ -108,17 +128,19 @@ pln_onewaymission(struct sctstr *target, int *shipno, int *flagp)
     flags = *flagp;
 
     /* offer carriers */
-    nships = carriersatxy(target->sct_x, target->sct_y,
-                         M_FLY | M_CHOPPER, 0, player->cnum);
+    nships = carriersatxy(target->sct_x, target->sct_y, player->cnum);
     if (nships) {
        for (;;) {
-           if (!(p = getstarg(0, "Carrier #? ", buf)) || !*p)
+           p = getstring("Carrier #? ", buf);
+           if (!p)
+               return -1;
+           if (!*p)
                break;
            cno = atoi(p);
            if (cno < 0
                || !getship(cno, &ship)
                || (!player->owner
-                   && (getrel(getnatp(ship.shp_own), player->cnum)
+                   && (relations_with(ship.shp_own, player->cnum)
                        != ALLIED))) {
                pr("Not yours\n");
                continue;
@@ -128,22 +150,14 @@ pln_onewaymission(struct sctstr *target, int *shipno, int *flagp)
                   xyas(target->sct_x, target->sct_y, player->cnum));
                continue;
            }
-           if ((!(mchr[(int)ship.shp_type].m_flags & M_FLY)
-                && !(mchr[(int)ship.shp_type].m_flags & M_XLIGHT)
-                && !(mchr[(int)ship.shp_type].m_flags & M_CHOPPER))
-               || ship.shp_effic < SHP_AIROPS_EFF) {
+           fl = carrier_planes(&ship, 0);
+           if (fl == 0) {
                pr("Can't land on %s.\n", prship(&ship));
                continue;
            }
-
            /* clear to land on ship#CNO */
            pr("landing on carrier %d\n", cno);
-           if (mchr[(int)ship.shp_type].m_flags & M_FLY)
-               flags |= P_L;
-           if (mchr[(int)ship.shp_type].m_flags & M_CHOPPER)
-               flags |= P_K;
-           if (mchr[(int)ship.shp_type].m_flags & M_XLIGHT)
-               flags |= P_E;
+           flags |= fl;
            *shipno = cno;
            *flagp = flags;
            return 0;
@@ -151,8 +165,7 @@ pln_onewaymission(struct sctstr *target, int *shipno, int *flagp)
     }
 
     /* try to land at sector */
-    if (target->sct_own != player->cnum
-       && getrel(getnatp(target->sct_own), player->cnum) != ALLIED) {
+    if (relations_with(target->sct_own, player->cnum) != ALLIED) {
        pr("Nowhere to land at sector %s!\n",
           xyas(target->sct_x, target->sct_y, player->cnum));
        return -1;
@@ -177,46 +190,21 @@ pln_oneway_to_carrier_ok(struct emp_qelem *bomb_list,
 {
     struct emp_qelem *list, *qp;
     struct plist *plp;
-    struct plchrstr *pcp;
     struct shpstr ship;
-    struct mchrstr *mcp;
-    int nchoppers, nxlight, nplane;
 
     if (cno < 0 || !getship(cno, &ship))
        return 0;
 
-    count_planes(&ship);
-    nchoppers = ship.shp_nchoppers;
-    nxlight = ship.shp_nxlight;
-    nplane = ship.shp_nplane;
-    mcp = &mchr[(int)ship.shp_type];
-
     /* for both lists */
     for (list = bomb_list;
         list;
         list = list == bomb_list ? esc_list : NULL) {
        for (qp = list->q_forw; qp != list; qp = qp->q_forw) {
-           /* FIXME duplicates put_plane_on_ship() logic; refactor */
            plp = (struct plist *)qp;
-           pcp = &plchr[(int)plp->plane.pln_type];
            if (plp->plane.pln_ship == ship.shp_uid)
                continue;
-           /* try chopper space */
-           if ((pcp->pl_flags & P_K) && (mcp->m_flags & M_CHOPPER)
-               && nchoppers < mcp->m_nchoppers)
-               ++nchoppers;
-           /* try xlight space */
-           else if ((pcp->pl_flags & P_E) && (mcp->m_flags & M_XLIGHT)
-                    && nxlight < mcp->m_nxlight)
-               ++nxlight;
-           /* try plane space */
-           else if ((((pcp->pl_flags & P_L) && (mcp->m_flags & M_FLY))
-                     || ((pcp->pl_flags & P_M) && (pcp->pl_flags & P_L)
-                         && (mcp->m_flags & M_MSL)))
-                    && nplane < mcp->m_nplanes)
-               ++nplane;
-           else
-               return 0;               /* won't be able to land */
+           if (!fit_plane_on_ship(&plp->plane, &ship))
+               return 0;
        }
     }
     return 1;
@@ -235,20 +223,17 @@ pln_newlanding(struct emp_qelem *list, coord tx, coord ty, int cno)
     for (qp = list->q_forw; qp != list; qp = qp->q_forw) {
        plp = (struct plist *)qp;
        if (cno >= 0) {
-           count_planes(&ship);
-           if (!can_be_on_ship(plp->plane.pln_uid, ship.shp_uid))
+           if (!could_be_on_ship(&plp->plane, &ship, 0, 0, 0, 0))
                pr("\t%s cannot land on ship #%d! %s aborts!\n",
                   prplane(&plp->plane), cno, prplane(&plp->plane));
            else if (!put_plane_on_ship(&plp->plane, &ship))
-               pr("\tNo room on ship #%d! %s aborts!\n", cno,
-                  prplane(&plp->plane));
+               pr("\tNo room on ship #%d! %s aborts!\n",
+                  cno, prplane(&plp->plane));
            else {
                if (plp->plane.pln_own != ship.shp_own) {
-/*                                     plp->plane.pln_own = ship.shp_own;*/
-                   wu(0, ship.shp_own,
-                      "%s %s lands on your %s\n",
-                      cname(player->cnum),
-                      prplane(&plp->plane), prship(&ship));
+                   wu(0, ship.shp_own, "%s %s lands on your %s\n",
+                      cname(player->cnum), prplane(&plp->plane),
+                      prship(&ship));
                }
            }
        } else {
@@ -256,7 +241,6 @@ pln_newlanding(struct emp_qelem *list, coord tx, coord ty, int cno)
            plp->plane.pln_y = ty;
            getsect(tx, ty, &sect);
            if (plp->plane.pln_own != sect.sct_own) {
-/*                             plp->plane.pln_own = sect.sct_own;*/
                wu(0, sect.sct_own,
                   "%s %s lands at your sector %s\n",
                   cname(player->cnum),
@@ -265,133 +249,304 @@ pln_newlanding(struct emp_qelem *list, coord tx, coord ty, int cno)
            plp->plane.pln_ship = cno;
        }
     }
-    if (cno >= 0)
-       putship(ship.shp_uid, &ship);
 }
 
 void
 pln_dropoff(struct emp_qelem *list, struct ichrstr *ip, coord tx, coord ty,
-           void *ptr, int type)
+           int cno)
 {
     struct emp_qelem *qp;
     struct plist *plp;
     int amt;
-    struct sctstr *sectp;
-    struct shpstr *sp;
+    struct sctstr sect;
+    struct shpstr ship;
     int there;
     int max;
-    struct mchrstr *mp;
 
-    if (ip == 0)
+    if (!ip)
        return;
     amt = 0;
     for (qp = list->q_forw; qp != list; qp = qp->q_forw) {
        plp = (struct plist *)qp;
-       amt += plp->misc;
+       amt += plp->load;
     }
-    if (type == EF_SECTOR) {
-       sectp = ptr;
-       if (!sectp->sct_own) {
-           if (sectp->sct_type == SCT_WATER)
+    if (cno < 0) {
+       getsect(tx, ty, &sect);
+       if (!sect.sct_own) {
+           if (sect.sct_type == SCT_WATER)
                pr("Your %s sink like a rock!\n", ip->i_name);
            else
                pr("Your %s vanish without a trace.\n", ip->i_name);
            return;
        }
-       if (sectp->sct_own != player->cnum
-           && getrel(getnatp(sectp->sct_own), player->cnum) != ALLIED) {
+       if (relations_with(sect.sct_own, player->cnum) != ALLIED) {
            pr("You don't own %s!  Cargo jettisoned...\n",
               xyas(tx, ty, player->cnum));
            return;
        }
-       if (ip->i_uid == I_CIVIL && sectp->sct_own != sectp->sct_oldown) {
+       if (ip->i_uid == I_CIVIL && sect.sct_own != sect.sct_oldown) {
            pr("%s is occupied.  Your civilians suffer from identity crisis and die.\n",
               xyas(tx, ty, player->cnum));
            return;
        }
-       there = sectp->sct_item[ip->i_uid];
+       there = sect.sct_item[ip->i_uid];
        max = ITEM_MAX;
     } else {
-       sp = ptr;
-       there = sp->shp_item[ip->i_uid];
-       mp = &mchr[(int)sp->shp_type];
-       max = mp->m_item[ip->i_uid];
+       getship(cno, &ship);
+       there = ship.shp_item[ip->i_uid];
+       max = mchr[ship.shp_type].m_item[ip->i_uid];
     }
     there += amt;
     if (there > max) {
-       pr("%d excess %s discarded\n", max - there, ip->i_name);
-       amt = max - there;
+       pr("%d excess %s discarded\n", there - max, ip->i_name);
+       amt -= there - max;
        there = max;
     }
     pr("%d %s landed safely", amt, ip->i_name);
-    if (type == EF_SECTOR) {
-       sectp = ptr;
-       sectp->sct_item[ip->i_uid] = there;
-       if (sectp->sct_own != player->cnum)
-           wu(0, sectp->sct_own, "%s planes drop %d %s in %s\n",
+    if (cno < 0) {
+       sect.sct_item[ip->i_uid] = there;
+       if (sect.sct_own != player->cnum)
+           wu(0, sect.sct_own, "%s planes drop %d %s in %s\n",
               cname(player->cnum), amt, ip->i_name,
-              xyas(sectp->sct_x, sectp->sct_y, sectp->sct_own));
+              xyas(tx, ty, sect.sct_own));
        pr(" at %s\n", xyas(tx, ty, player->cnum));
-       putsect((struct sctstr *)ptr);
+       putsect(&sect);
     } else {
-       struct shpstr *sp = (struct shpstr *)ptr;
-       sp->shp_item[ip->i_uid] = there;
-       if (sp->shp_own != player->cnum)
-           wu(0, sp->shp_own, "%s planes land %d %s on carrier %d\n",
-              cname(player->cnum), amt, ip->i_name, sp->shp_uid);
-       pr(" on carrier #%d\n", sp->shp_uid);
-       putship(sp->shp_uid, sp);
+       ship.shp_item[ip->i_uid] = there;
+       if (ship.shp_own != player->cnum)
+           wu(0, ship.shp_own, "%s planes land %d %s on carrier %d\n",
+              cname(player->cnum), amt, ip->i_name, ship.shp_uid);
+       pr(" on carrier #%d\n", ship.shp_uid);
+       putship(ship.shp_uid, &ship);
     }
 }
 
 void
-pln_mine(struct emp_qelem *list, struct sctstr *sectp)
+pln_mine(struct emp_qelem *list, coord tx, coord ty)
 {
     struct emp_qelem *qp;
     struct plist *plp;
     int amt;
+    struct sctstr sect;
 
     amt = 0;
     for (qp = list->q_forw; qp != list; qp = qp->q_forw) {
        plp = (struct plist *)qp;
-       amt += plp->misc;
+       amt += plp->load;
 
     }
     if (amt > 0) {
-       if (sectp->sct_type != SCT_WATER) {
+       getsect(tx, ty, &sect);
+       if (sect.sct_type != SCT_WATER) {
            pr("Your seamines have no effect here.\n");
            return;
        }
-       sectp->sct_mines = MIN(sectp->sct_mines + amt, MINES_MAX);
-       pr("%d mines laid in %s.\n", amt,
-          xyas(sectp->sct_x, sectp->sct_y, player->cnum));
-       if (map_set(player->cnum, sectp->sct_x, sectp->sct_y, 'X', 0))
+       sect.sct_mines = MIN(sect.sct_mines + amt, MINES_MAX);
+       pr("%d mines laid in %s.\n", amt, xyas(tx, ty, player->cnum));
+       if (map_set(player->cnum, tx, ty, 'X', 0))
            writemap(player->cnum);
-       putsect(sectp);
+       putsect(&sect);
     }
 }
 
+/*
+ * Has PP's type capabilities satisfying WANTFLAGS and NOWANTFLAGS?
+ * A plane type is capable unless
+ * - it lacks all of the P_B, P_T in WANTFLAGS, or
+ * - it lacks all of the P_F, P_ESC in WANTFLAGS, or
+ * - it lacks all of the P_E, P_L, P_K in WANTFLAGS, or
+ * - it lacks any of the other capabilities in WANTFLAGS, or
+ * - it has any of the capabilities in NOWANTFLAGS.
+ */
+int
+pln_capable(struct plnstr *pp, int wantflags, int nowantflags)
+{
+    int flags = plchr[(int)pp->pln_type].pl_flags;
+
+    if (wantflags & (P_B | P_T)) {
+       if ((flags & wantflags & (P_B | P_T)) == 0)
+           return 0;
+       wantflags &= ~(P_B | P_T);
+    }
+
+    if (wantflags & (P_F | P_ESC)) {
+       if ((flags & wantflags & (P_F | P_ESC)) == 0)
+           return 0;
+       wantflags &= ~(P_F | P_ESC);
+    }
+
+    if (wantflags & (P_E | P_L | P_K)) {
+       if ((flags & wantflags & (P_E | P_L | P_K)) == 0)
+           return 0;
+       wantflags &= ~(P_E | P_L | P_K);
+    }
+
+    if ((flags & wantflags) != wantflags)
+       return 0;
+
+    if (flags & nowantflags)
+       return 0;
+
+    return 1;
+}
+
+/*
+ * Return union of capabilities of planes in LIST.
+ */
+int
+pln_caps(struct emp_qelem *list)
+{
+    struct emp_qelem *qp;
+    struct plist *plp;
+    int fl;
+
+    fl = 0;
+    for (qp = list->q_forw; qp != list; qp = qp->q_forw) {
+       plp = (struct plist *)qp;
+       fl |= plp->pcp->pl_flags;
+    }
+
+    return fl;
+}
+
+/*
+ * Find plane types that can operate from carrier SP.
+ * If MSL find missile types, else non-missile types.
+ * Return a combination of P_L, P_K, P_E.
+ * It's zero if SP can't support air operations due to its type or
+ * state (low efficiency).
+ */
+int
+carrier_planes(struct shpstr *sp, int msl)
+{
+    struct mchrstr *mcp = mchr + sp->shp_type;
+    int res;
+
+    if (sp->shp_effic < SHP_AIROPS_EFF)
+       return 0;
+
+    res = 0;
+    if (mcp->m_flags & M_FLY)
+       res |= P_L;
+    if ((mcp->m_flags & M_MSL) && msl)
+       res |= P_L;
+    if (mcp->m_nchoppers && !msl)
+       res |= P_K;
+    if (mcp->m_nxlight)
+       res |= P_E;
+    return res;
+}
+
+int
+pln_airbase_ok(struct plnstr *pp, int oneway, int noisy)
+{
+    struct shpstr ship;
+    struct lndstr land;
+    struct sctstr sect;
+    struct plchrstr *pcp = plchr + pp->pln_type;
+
+    if (CANT_HAPPEN(noisy && pp->pln_own != player->cnum))
+       noisy = 0;
+
+    if (pp->pln_ship >= 0) {
+       /* ship: needs to be own or allied, efficient */
+       if (!getship(pp->pln_ship, &ship)) {
+           CANT_REACH();
+           return 0;
+       }
+       if (relations_with(ship.shp_own, pp->pln_own) != ALLIED) {
+           if (noisy)
+               pr("(note) An ally does not own the ship %s is on\n",
+                  prplane(pp));
+           return 0;
+       }
+       if (!(carrier_planes(&ship, pcp->pl_flags & P_M) & pcp->pl_flags))
+           return 0;
+
+    } else if (pp->pln_land >= 0) {
+       /* land: needs to be own or allied, efficient, not embarked */
+       if (!getland(pp->pln_land, &land)) {
+           CANT_REACH();
+           return 0;
+       }
+       if (relations_with(land.lnd_own, pp->pln_own) != ALLIED) {
+           if (noisy)
+               pr("(note) An ally does not own the unit %s is on\n",
+                  prplane(pp));
+           return 0;
+       }
+       if (land.lnd_effic < LND_AIROPS_EFF || !(pcp->pl_flags & P_E))
+           return 0;
+       if (land.lnd_ship >= 0 || land.lnd_land >= 0)
+           return 0;
+
+    } else {
+       /* sector: needs to be own or allied, efficient airfield */
+       if (!getsect(pp->pln_x, pp->pln_y, &sect)) {
+           CANT_REACH();
+           return 0;
+       }
+
+       if (relations_with(sect.sct_own, pp->pln_own) != ALLIED) {
+           if (noisy)
+               pr("(note) An ally does not own the sector %s is in\n",
+                  prplane(pp));
+           return 0;
+       }
+       /* need airfield unless VTOL */
+       if ((pcp->pl_flags & P_V) == 0) {
+           if (sect.sct_type != SCT_AIRPT) {
+               if (noisy)
+                   pr("%s not at airport\n", prplane(pp));
+               return 0;
+           }
+           if (sect.sct_effic < 40) {
+               if (noisy)
+                   pr("%s is not 40%% efficient, %s can't take off from there.\n",
+                      xyas(sect.sct_x, sect.sct_y, player->cnum),
+                      prplane(pp));
+               return 0;
+           }
+           if (!oneway && sect.sct_effic < 60) {
+               if (noisy)
+                   pr("%s is not 60%% efficient, %s can't land there.\n",
+                      xyas(sect.sct_x, sect.sct_y, player->cnum),
+                      prplane(pp));
+               return 0;
+           }
+       }
+    }
+
+    return 1;
+}
+
 void
 pln_sel(struct nstr_item *ni, struct emp_qelem *list, struct sctstr *ap,
        int ap_to_target, int rangemult, int wantflags, int nowantflags)
 {
     struct plnstr plane;
-    struct shpstr ship;
-    struct lndstr land;
-    struct sctstr sect;
     int range;
     struct plchrstr *pcp;
     struct plist *plp;
-    int y;
-    int bad, bad1;
-    unsigned x;
 
     emp_initque(list);
     while (nxtitem(ni, &plane)) {
-       if (!player->owner)
+       /*
+        * It would be nice to let deities fly foreign planes, but
+        * much of the code assumes that only the plane's owner can
+        * fly it.
+        */
+       if (!plane.pln_own || plane.pln_own != player->cnum)
            continue;
        if (plane.pln_mobil <= 0)
            continue;
+       if (plane.pln_effic < 40) {
+           pr("%s not efficient enough (must be 40%%)\n",
+              prplane(&plane));
+           continue;
+       }
+       if (!pln_capable(&plane, wantflags, nowantflags))
+           continue;
        if (opt_MARKET) {
            if (ontradingblock(EF_PLANE, &plane)) {
                pr("plane #%d inelligible - it's for sale.\n",
@@ -405,216 +560,63 @@ pln_sel(struct nstr_item *ni, struct emp_qelem *list, struct sctstr *ap,
            pr("%s too far from assembly point\n", prplane(&plane));
            continue;
        }
-       if (plane.pln_effic < 40) {
-           pr("%s not efficient enough (must be 40%%)\n",
-              prplane(&plane));
-           continue;
-       }
        range += ap_to_target;
        range *= rangemult;
        pcp = &plchr[(int)plane.pln_type];
-       bad = 0;
-       bad1 = 0;
-       if (wantflags) {
-           for (x = 0; x < sizeof(wantflags) * 8; x++) {
-               y = (1 << x);
-               if ((wantflags & y) == y)
-                   if ((pcp->pl_flags & y) != y) {
-                       switch (y) {
-                       case P_F:
-                       case P_ESC:
-                           bad1 = 2;
-                           break;
-                       case P_E:
-                       case P_L:
-                       case P_K:
-                           bad1 = 1;
-                           break;
-                       default:
-                           bad = 1;
-                       }
-                   }
-           }
-           if (bad)
-               continue;
-           if (bad1 == 2) {
-               if ((pcp->pl_flags & P_ESC) || (pcp->pl_flags & P_F))
-                   bad1 = 0;
-           }
-           if (bad1 == 1) {
-               if ((wantflags & P_L) && (pcp->pl_flags & P_L))
-                   bad1 = 0;
-               if ((wantflags & P_K) && (pcp->pl_flags & P_K))
-                   bad1 = 0;
-               if ((wantflags & P_E) && (pcp->pl_flags & P_E))
-                   bad1 = 0;
-           }
-           if (bad1)
-               continue;
-       }
-       bad = 0;
-       bad1 = 0;
-       if (nowantflags) {
-           for (x = 0; x < sizeof(nowantflags) * 8; x++) {
-               y = (1 << x);
-               if ((nowantflags & y) == y)
-                   if ((pcp->pl_flags & y) == y)
-                       bad = 1;
-           }
-           if (bad)
-               continue;
-       }
        if (plane.pln_range < range) {
            pr("%s out of range (%d:%d)\n",
               prplane(&plane), plane.pln_range, range);
            continue;
        }
-       if (plane.pln_ship >= 0) {
-           if (!getship(plane.pln_ship, &ship) ||
-               plane.pln_own != player->cnum) {
-             shipsunk:
-               plane.pln_effic = 0;
-               pr("(note) ship not valid for %s\n", prplane(&plane));
-               putplane(plane.pln_uid, &plane);
-               continue;
-           }
-           if (!can_be_on_ship(plane.pln_uid, ship.shp_uid))
-               goto shipsunk;
-           if (ship.shp_effic < SHIP_MINEFF)
-               goto shipsunk;
-           if (ship.shp_effic < SHP_AIROPS_EFF)
-               continue;
-           /* Can't fly off non-owned ships or non-allied ship */
-           if ((ship.shp_own != player->cnum) &&
-               (getrel(getnatp(ship.shp_own), player->cnum) != ALLIED)) {
-               pr("(note) An ally does not own the ship %s is on\n",
-                  prplane(&plane));
-               continue;
-           }
-       }
-       if (plane.pln_land >= 0) {
-           if (!getland(plane.pln_land, &land) ||
-               (plane.pln_own != player->cnum)) {
-             landdead:
-               plane.pln_effic = 0;
-               pr("(note) land unit not valid for %s\n", prplane(&plane));
-               putplane(plane.pln_uid, &plane);
-               continue;
-           }
-           if (!(plchr[(int)plane.pln_type].pl_flags & P_E))
-               goto landdead;
-           if (land.lnd_effic < LAND_MINEFF)
-               goto landdead;
-           if (land.lnd_effic < LND_AIROPS_EFF)
-               continue;
-           /* Can't fly off units in ships or other units */
-           if ((land.lnd_ship >= 0) || (land.lnd_land >= 0))
-               continue;
-           /* Can't fly off non-owned units or non-allied unit */
-           if ((land.lnd_own != player->cnum) &&
-               (getrel(getnatp(land.lnd_own), player->cnum) != ALLIED)) {
-               pr("(note) An ally does not own the unit %s is on\n",
-                  prplane(&plane));
-               continue;
-           }
-       }
-       /* Now, check the sector status if not on a plane or unit */
-       if ((plane.pln_ship < 0) && (plane.pln_land < 0)) {
-           if (!getsect(plane.pln_x, plane.pln_y, &sect))
-               continue;
-           /* First, check allied status */
-           /* Can't fly from non-owned sectors or non-allied sectors */
-           if ((sect.sct_own != player->cnum) &&
-               (getrel(getnatp(sect.sct_own), player->cnum) != ALLIED)) {
-               pr("(note) An ally does not own the sector %s is in\n",
-                  prplane(&plane));
-               continue;
-           }
-           /* non-vtol plane */
-           if ((pcp->pl_flags & P_V) == 0) {
-               if (sect.sct_type != SCT_AIRPT) {
-                   pr("%s not at airport\n", prplane(&plane));
-                   continue;
-               }
-               if (sect.sct_effic < 40) {
-                   pr("%s is not 40%% efficient, %s can't take off from there.\n", xyas(sect.sct_x, sect.sct_y, plane.pln_own), prplane(&plane));
-                   continue;
-               }
-               if (rangemult == 2 && sect.sct_effic < 60) {
-                   pr("%s is not 60%% efficient, %s can't land there.\n",
-                      xyas(sect.sct_x, sect.sct_y, plane.pln_own),
-                      prplane(&plane));
-                   continue;
-               }
-           }
-       }
+       if (!pln_airbase_ok(&plane, rangemult != 2, 1))
+           continue;
        pr("%s standing by\n", prplane(&plane));
        plane.pln_mission = 0;
        putplane(plane.pln_uid, &plane);
        plp = malloc(sizeof(struct plist));
-       plp->misc = 0;
-       plp->bombs = 0;
+       plp->load = 0;
        plp->pcp = pcp;
        plp->plane = plane;
        emp_insque(&plp->queue, list);
     }
 }
 
-int
-pln_arm(struct emp_qelem *list, int dist, char mission, struct ichrstr *ip,
-       int flags, int mission_flags)
+void
+pln_arm(struct emp_qelem *list, int dist, char mission, struct ichrstr *ip)
 {
     struct emp_qelem *qp;
     struct emp_qelem *next;
     struct plist *plp;
+    struct plnstr *pp;
 
     for (qp = list->q_forw; qp != list; qp = next) {
        next = qp->q_forw;
        plp = (struct plist *)qp;
-       if (pln_equip(plp, ip, flags, mission) < 0) {
+       pp = &plp->plane;
+       getplane(pp->pln_uid, pp);
+       if ((pp->pln_flags & PLN_LAUNCHED)
+           || pln_equip(plp, ip, mission) < 0) {
            emp_remque(qp);
            free(qp);
            continue;
        }
-       if (flags & (P_S | P_I)) {
-           if (plp->pcp->pl_flags & P_S)
-               mission_flags |= P_S;
-           if (plp->pcp->pl_flags & P_I)
-               mission_flags |= P_I;
-       }
-       if (!(plp->pcp->pl_flags & P_H))
-           /* no stealth on this mission */
-           mission_flags &= ~P_H;
-       if (!(plp->pcp->pl_flags & P_X))
-           /* no stealth on this mission */
-           mission_flags &= ~P_X;
-       if (!(plp->pcp->pl_flags & P_A)) {
-           /* no asw on this mission */
-           mission_flags &= ~P_A;
-           /* FIXME escorts turn ASW patrol into ordinary recon */
-       }
-       if (!(plp->pcp->pl_flags & P_MINE)) {
-           /* no asw on this mission */
-           mission_flags &= ~P_MINE;
-           /* FIXME no effect */
-       }
-       plp->plane.pln_mobil -= pln_mobcost(dist, &plp->plane, flags);
-       pr("%s equipped\n", prplane(&plp->plane));
+       pp->pln_flags |= PLN_LAUNCHED;
+       pp->pln_mobil -= pln_mobcost(dist, pp, mission);
+       putplane(pp->pln_uid, pp);
+       pr("%s equipped\n", prplane(pp));
     }
-    return mission_flags;
 }
 
-static int
-pln_equip(struct plist *plp, struct ichrstr *ip, int flags, char mission)
+int
+pln_equip(struct plist *plp, struct ichrstr *ip, char mission)
 {
     struct plchrstr *pcp;
     struct plnstr *pp;
-    int needed;
+    int load, needed;
     struct lndstr land;
     struct shpstr ship;
     struct sctstr sect;
     i_type itype;
-    int rval;
     short *item;
     int own;
     int abandon_needed;
@@ -634,82 +636,100 @@ pln_equip(struct plist *plp, struct ichrstr *ip, int flags, char mission)
        item = sect.sct_item;
        own = sect.sct_oldown;
     }
-    if (ip) {
-       if (ip->i_uid == I_CIVIL) {
-           if (pp->pln_own != own) {
-               pr("You don't control those civilians!\n");
-               return -1;
-           }
-       }
-    }
     if (pcp->pl_fuel > item[I_PETROL]) {
        pr("%s not enough petrol there!\n", prplane(pp));
        return -1;
     }
     item[I_PETROL] -= pcp->pl_fuel;
-    rval = 0;
-    if ((flags & P_F) == 0) {
-       itype = I_NONE;
-       needed = 0;
-       switch (mission) {
-       case 's':
-       case 'p':
-           if (pp->pln_nuketype == -1) {
-               itype = I_SHELL;
-               needed = pp->pln_load;
-           }
+    load = pln_load(pp);
+    itype = I_NONE;
+    switch (mission) {
+    case 's':          /* strategic bomb */
+    case 'p':          /* pinpoint bomb */
+       itype = I_SHELL;
+       break;
+    case 't':          /* transport */
+       if (!(pcp->pl_flags & P_C) || !ip)
            break;
-       case 't':
-       case 'd':
-           if ((pcp->pl_flags & P_C) == 0 || ip == 0)
-               break;
-           itype = ip->i_uid;
-           needed = (pp->pln_load * 2) / ip->i_lbs;
+       itype = ip->i_uid;
+       load *= 2;
+       break;
+    case 'm':          /* mine */
+       if ((pcp->pl_flags & P_MINE) == 0)
            break;
-       case 'm':
-           if ((pcp->pl_flags & P_MINE) == 0)
-               break;
-           itype = I_SHELL;
-           needed = (pp->pln_load * 2) / ip->i_lbs;
+       itype = I_SHELL;
+       load *= 2;
+       break;
+    case 'd':          /* drop */
+       if (!(pcp->pl_flags & P_C) || CANT_HAPPEN(!ip))
            break;
-       case 'a':
-           if ((pcp->pl_flags & (P_V | P_C)) == 0)
-               break;
-           itype = I_MILIT;
-           needed = pp->pln_load / ip->i_lbs;
+       itype = ip->i_uid;
+       if (pcp->pl_flags & P_V)
+           load *= 2;
+       break;
+    case 'a':          /* paradrop */
+       if (!(pcp->pl_flags & P_P))
            break;
-       case 'n':
-           if (pp->pln_nuketype == -1)
-               rval = -1;
+       itype = I_MILIT;
+       if (pcp->pl_flags & P_V)
+           load *= 2;
+       break;
+    case 'r':          /* reconnaissance */
+    case 'e':          /* escort */
+       load = 0;
+       break;
+    case 'i':          /* missile interception */
+       if (CANT_HAPPEN(!(pcp->pl_flags & P_M)
+                       || !(pcp->pl_flags & (P_N | P_O))))
            break;
-       default:
-           break;
-       }
-       if (rval < 0 || (itype != I_NONE && needed <= 0)) {
+       if (load)
+           itype = I_SHELL;
+       break;
+    default:
+       CANT_REACH();
+       load = 0;
+    }
+
+    if (itype != I_NONE) {
+       needed = load / ichr[itype].i_lbs;
+       if (needed <= 0) {
            pr("%s can't contribute to mission\n", prplane(pp));
            return -1;
        }
-       if (itype != I_NONE) {
-#if 0
-           /* Supply is broken somewhere, so don't use it for now */
-           if (itype == I_SHELL && item[itype] < needed)
-               item[itype] += supply_commod(plp->plane.pln_own,
-                                            plp->plane.pln_x,
-                                            plp->plane.pln_y,
-                                            I_SHELL, needed);
-#endif
-           abandon_needed = !!would_abandon(&sect, itype, needed, NULL);
-           if (item[itype] < needed + abandon_needed) {
-               pr("Not enough %s for %s\n", ichr[itype].i_name, prplane(pp));
+       if (nuk_on_plane(pp) >= 0) {
+           if (mission == 's' || mission == 't')
+               needed = 0;
+           else {
+               pr("%s can't fly this mission"
+                  " while it is carrying a nuclear weapon",
+                  prplane(pp));
                return -1;
            }
-           item[itype] -= needed;
        }
-       if (itype == I_SHELL && (mission == 's' || mission == 'p'))
-           plp->bombs = needed;
+       if (itype == I_CIVIL && pp->pln_own != own) {
+           pr("You don't control those civilians!\n");
+           return -1;
+       }
+#if 0
+       /* Supply is broken somewhere, so don't use it for now */
+       if (itype == I_SHELL && item[itype] < needed)
+           item[itype] += supply_commod(plp->plane.pln_own,
+                                        plp->plane.pln_x,
+                                        plp->plane.pln_y,
+                                        I_SHELL, needed);
+#endif
+       if (pp->pln_ship >= 0 || pp->pln_land >= 0)
+           abandon_needed = 0;
        else
-           plp->misc = needed;
+           abandon_needed = !!would_abandon(&sect, itype, needed, NULL);
+       if (item[itype] < needed + abandon_needed) {
+           pr("Not enough %s for %s\n", ichr[itype].i_name, prplane(pp));
+           return -1;
+       }
+       item[itype] -= needed;
+       plp->load = needed;
     }
+
     if (pp->pln_ship >= 0) {
        if (pp->pln_own != ship.shp_own) {
            wu(0, ship.shp_own,
@@ -732,247 +752,161 @@ pln_equip(struct plist *plp, struct ichrstr *ip, int flags, char mission)
        }
        putsect(&sect);
     }
-    return rval;
+    return 0;
 }
 
 void
 pln_put(struct emp_qelem *list)
 {
-    struct emp_qelem *qp;
-    struct emp_qelem *newqp;
-    struct plist *plp;
-    struct plnstr *pp;
-    struct shpstr ship;
-    struct sctstr sect;
+    struct emp_qelem *qp, *next;
 
-    /* Here is where planes return home from bombing runs.
-       We need to make sure they still have somewhere to return
-       home to! */
-    qp = list->q_forw;
-    while (qp != list) {
-       plp = (struct plist *)qp;
-       pp = &plp->plane;
-       /* Ok, check out where it wants to land */
-       if (pp->pln_ship >= 0) {
-           /* It is landing on a carrier */
-           getship(pp->pln_ship, &ship);
-           /* We should do more, like make sure it's really
-              a carrier, etc. but for now just make sure it's
-              not sunk. */
-           if (ship.shp_effic < SHIP_MINEFF) {
-               mpr(pp->pln_own,
-                   "Ship #%d has been sunk, plane #%d has nowhere to land, and\n"
-                   "splashes into the sea.\n",
-                   pp->pln_ship, pp->pln_uid);
-               pp->pln_effic = 0;
-           }
-       } else {
-           /* Presume we are landing back in a sector. */
-           getsect(pp->pln_x, pp->pln_y, &sect);
-           if (sect.sct_type == SCT_WATER || sect.sct_type == SCT_WASTE) {
-               mpr(pp->pln_own,
-                   "Nowhere to land at %s, plane #%d crashes and burns...\n",
-                   xyas(pp->pln_x, pp->pln_y, pp->pln_own), pp->pln_uid);
-               pp->pln_effic = 0;
-           }
-       }
-       putplane(pp->pln_uid, pp);
-       newqp = qp->q_forw;
-       emp_remque(qp);
-       free(qp);
-       qp = newqp;
+    for (qp = list->q_forw; qp != list; qp = next) {
+       next = qp->q_forw;
+       pln_put1((struct plist *)qp);
     }
 }
 
 void
-pln_removedupes(struct emp_qelem *bomb_list, struct emp_qelem *esc_list)
+pln_put1(struct plist *plp)
 {
-    struct emp_qelem *bomb;
-    struct emp_qelem *esc;
-    struct plist *bombp;
-    struct plist *escp;
+    struct plnstr *pp;
+    struct shpstr ship;
+    struct sctstr sect;
 
-    if (QEMPTY(bomb_list) || QEMPTY(esc_list))
-       return;
-    bomb = bomb_list->q_forw;
-    while (bomb != bomb_list) {
-       if (QEMPTY(esc_list)) {
-           bomb = bomb_list;
-           continue;
+    pp = &plp->plane;
+
+    if (CANT_HAPPEN((pp->pln_flags & PLN_LAUNCHED)
+                   && (plchr[pp->pln_type].pl_flags & P_M)
+                   && pp->pln_effic >= PLANE_MINEFF))
+       pp->pln_effic = 0;   /* bug: missile launched but not used up */
+
+    if (!(pp->pln_flags & PLN_LAUNCHED))
+       ;                       /* never took off */
+    else if (pp->pln_effic < PLANE_MINEFF) {
+       ;                       /* destroyed */
+    } else if (pp->pln_ship >= 0) {
+       /* It is landing on a carrier */
+       getship(pp->pln_ship, &ship);
+       /* We should do more, like make sure it's really
+          a carrier, etc. but for now just make sure it's
+          not sunk. */
+       if (ship.shp_effic < SHIP_MINEFF) {
+           mpr(pp->pln_own,
+               "Ship #%d has been sunk, plane #%d has nowhere to land, and\n"
+               "splashes into the sea.\n",
+               pp->pln_ship, pp->pln_uid);
+           pp->pln_effic = 0;
        }
-       esc = esc_list->q_forw;
-       bombp = (struct plist *)bomb;
-       while (esc != esc_list) {
-           escp = (struct plist *)esc;
-           if (escp->plane.pln_uid == bombp->plane.pln_uid) {
-               emp_remque(esc);
-               free(esc);
-               esc = esc_list;
-           } else
-               esc = esc->q_forw;
+    } else {
+       /* Presume we are landing back in a sector. */
+       getsect(pp->pln_x, pp->pln_y, &sect);
+       if (sect.sct_type == SCT_WATER || sect.sct_type == SCT_WASTE) {
+           mpr(pp->pln_own,
+               "Nowhere to land at %s, plane #%d crashes and burns...\n",
+               xyas(pp->pln_x, pp->pln_y, pp->pln_own), pp->pln_uid);
+           pp->pln_effic = 0;
        }
-       bomb = bomb->q_forw;
     }
+    pp->pln_flags &= ~PLN_LAUNCHED;
+    putplane(pp->pln_uid, pp);
+    emp_remque(&plp->queue);
+    free(plp);
 }
 
+/*
+ * Can PP be loaded on a ship of SP's type?
+ * Assume that it already carries N planes, of which NCH are choppers,
+ * NXL xlight, NMSL light missiles, and the rest are light fixed-wing
+ * planes.
+ */
 int
-put_plane_on_ship(struct plnstr *plane, struct shpstr *ship)
+could_be_on_ship(struct plnstr *pp, struct shpstr *sp,
+                int n, int nch, int nxl, int nmsl)
 {
-    struct plchrstr *pcp;
-    struct mchrstr *mcp;
-
-    pcp = &plchr[(int)plane->pln_type];
-    mcp = &mchr[(int)ship->shp_type];
-
-    if (((int)plane->pln_ship) == ((int)ship->shp_uid))
-       return 1;               /* Already on ship */
-
-    /* Try to put on ship as a chopper plane */
-    if ((pcp->pl_flags & P_K) &&
-       (mcp->m_flags & M_CHOPPER) &&
-       (ship->shp_nchoppers < mcp->m_nchoppers)) {
-
-       ship->shp_nchoppers++;
-       plane->pln_x = ship->shp_x;
-       plane->pln_y = ship->shp_y;
-       plane->pln_ship = ship->shp_uid;
-       putship(ship->shp_uid, ship);
-       putplane(plane->pln_uid, plane);
-       return 1;
-    }
+    struct plchrstr *pcp = &plchr[pp->pln_type];
+    struct mchrstr *mcp = &mchr[sp->shp_type];
+    int nfw;
 
-    /* Try to put on ship as an xlight plane */
-    if ((pcp->pl_flags & P_E) &&
-       (mcp->m_flags & M_XLIGHT) &&
-       (ship->shp_nxlight < mcp->m_nxlight)) {
-
-       ship->shp_nxlight++;
-       plane->pln_x = ship->shp_x;
-       plane->pln_y = ship->shp_y;
-       plane->pln_ship = ship->shp_uid;
-       putship(ship->shp_uid, ship);
-       putplane(plane->pln_uid, plane);
-       return 1;
-    }
-
-    /* Try to put on ship as a normal plane */
-    if ((((pcp->pl_flags & P_L) && (mcp->m_flags & M_FLY)) ||
-        ((pcp->pl_flags & P_M) && (pcp->pl_flags & P_L) &&
-         (mcp->m_flags & M_MSL))) &&
-       (ship->shp_nplane < mcp->m_nplanes)) {
-
-       ship->shp_nplane++;
-       plane->pln_x = ship->shp_x;
-       plane->pln_y = ship->shp_y;
-       plane->pln_ship = ship->shp_uid;
-       putship(ship->shp_uid, ship);
-       putplane(plane->pln_uid, plane);
-       return 1;
-    }
-
-    /* We have failed */
-    return 0;
+    if (pcp->pl_flags & P_K)
+       nch++;
+    else if (pcp->pl_flags & P_E)
+       nxl++;
+    else if (!(pcp->pl_flags & P_L))
+       return 0;
+    else if (pcp->pl_flags & P_M)
+       nmsl++;
+    n++;
+    nfw = n - nch - nxl - nmsl;
+
+    if (nch > mcp->m_nchoppers) /* overflow into fixed-wing slots */
+       nfw += nch - mcp->m_nchoppers;
+    if (nxl > mcp->m_nxlight)  /* overflow into missile slots */
+       nmsl += nxl - mcp->m_nxlight;
+    if (nmsl && !(mcp->m_flags & (M_MSL | M_FLY)))
+       return 0;               /* missile slots wanted */
+    if (nfw && !(mcp->m_flags & M_FLY))
+       return 0;               /* fixed-wing slots wanted */
+    return nfw + nmsl <= mcp->m_nplanes;
 }
 
-int
-take_plane_off_ship(struct plnstr *plane, struct shpstr *ship)
+/*
+ * Fit a plane of PP's type on ship SP.
+ * Updating the plane accordingly is the caller's job.
+ * Return whether it fits.
+ */
+static int
+fit_plane_on_ship(struct plnstr *pp, struct shpstr *sp)
 {
-    struct plchrstr *pcp;
-    struct mchrstr *mcp;
-
-    pcp = &plchr[(int)plane->pln_type];
-    mcp = &mchr[(int)ship->shp_type];
-
-    /* Try to take off ship as a chopper plane */
-    if ((pcp->pl_flags & P_K) &&
-       (mcp->m_flags & M_CHOPPER) && (ship->shp_nchoppers)) {
-
-       ship->shp_nchoppers--;
-       plane->pln_ship = -1;
-       putship(ship->shp_uid, ship);
-       putplane(plane->pln_uid, plane);
-       return 1;
-    }
-
-    /* Try to take off ship as an xlight plane */
-    if ((pcp->pl_flags & P_E) &&
-       (mcp->m_flags & M_XLIGHT) && (ship->shp_nxlight)) {
+    int n, nch, nxl, nmsl;
 
-       ship->shp_nxlight--;
-       plane->pln_ship = -1;
-       putship(ship->shp_uid, ship);
-       putplane(plane->pln_uid, plane);
-       return 1;
-    }
-
-    /* Try to take off ship as a normal plane */
-    if ((((pcp->pl_flags & P_L) && (mcp->m_flags & M_FLY)) ||
-        ((pcp->pl_flags & P_M) && (pcp->pl_flags & P_L) &&
-         (mcp->m_flags & M_MSL))) && (ship->shp_nplane)) {
-
-       ship->shp_nplane--;
-       plane->pln_ship = -1;
-       putship(ship->shp_uid, ship);
-       putplane(plane->pln_uid, plane);
-       return 1;
-    }
-
-    /* We have failed */
-    return 0;
+    n = shp_nplane(sp, &nch, &nxl, &nmsl);
+    return could_be_on_ship(pp, sp, n, nch, nxl, nmsl);
 }
 
 int
-take_plane_off_land(struct plnstr *plane, struct lndstr *land)
+put_plane_on_ship(struct plnstr *plane, struct shpstr *ship)
 {
-    struct plchrstr *pcp;
+    if (plane->pln_ship == ship->shp_uid)
+       return 1;               /* Already on ship */
 
-    pcp = &plchr[(int)plane->pln_type];
+    if (!fit_plane_on_ship(plane, ship))
+       return 0;
 
-    /* Try to take off ship as an xlight plane */
-    if ((pcp->pl_flags & P_E) && land->lnd_nxlight) {
+    plane->pln_x = ship->shp_x;
+    plane->pln_y = ship->shp_y;
+    plane->pln_ship = ship->shp_uid;
+    putplane(plane->pln_uid, plane);
+    return 1;
+}
 
-       land->lnd_nxlight--;
-       plane->pln_land = -1;
-       putland(land->lnd_uid, land);
-       putplane(plane->pln_uid, plane);
-       return 1;
-    }
+/*
+ * Fit a plane of PP's type on land unit LP.
+ * Updating the plane accordingly is the caller's job.
+ * Return whether it fits.
+ */
+static int
+fit_plane_on_land(struct plnstr *pp, struct lndstr *lp)
+{
+    struct plchrstr *pcp = plchr + pp->pln_type;
+    struct lchrstr *lcp = lchr + lp->lnd_type;
 
-    /* We have failed */
-    return 0;
+    return (pcp->pl_flags & P_E) && lnd_nxlight(lp) < lcp->l_nxlight;
 }
 
 int
-can_be_on_ship(int p, int s)
+put_plane_on_land(struct plnstr *plane, struct lndstr *land)
 {
-    struct plnstr plane;
-    struct shpstr ship;
-    struct plchrstr *pcp;
-    struct mchrstr *mcp;
-
-    getplane(p, &plane);
-    getship(s, &ship);
-
-    pcp = &plchr[(int)plane.pln_type];
-    mcp = &mchr[(int)ship.shp_type];
-
-    if (pcp->pl_flags & P_L)
-       if (mcp->m_flags & M_FLY)
-           return 1;
-
-    if (pcp->pl_flags & P_K)
-       if (mcp->m_flags & M_CHOPPER)
-           return 1;
-
-    if (pcp->pl_flags & P_M)
-       if (mcp->m_flags & M_MSL)
-           return 1;
+    if (plane->pln_land == land->lnd_uid)
+       return 1;               /* Already on unit */
 
-    if (pcp->pl_flags & P_E)
-       if (mcp->m_flags & M_XLIGHT)
-           return 1;
+    if (!fit_plane_on_land(plane, land))
+       return 0;
 
-    return 0;
+    plane->pln_x = land->lnd_x;
+    plane->pln_y = land->lnd_y;
+    plane->pln_land = land->lnd_uid;
+    putplane(plane->pln_uid, plane);
+    return 1;
 }
 
 void
@@ -993,7 +927,7 @@ plane_sweep(struct emp_qelem *plane_list, coord x, coord y)
     if (mines_there == 0)
        return;
 
-    if ((sect.sct_type != SCT_WATER) && (sect.sct_type != SCT_HARBR))
+    if (sect.sct_type != SCT_WATER)
        return;
 
     for (qp = plane_list->q_forw; ((qp != plane_list) && (mines_there));
@@ -1005,9 +939,9 @@ plane_sweep(struct emp_qelem *plane_list, coord x, coord y)
        if (!(pcp->pl_flags & P_SWEEP)) /* if it isn't an sweep plane */
            continue;
 
-       if (chance((100.0 - pp->pln_acc) / 100.0)) {
+       if (chance((100.0 - pln_acc(pp)) / 100.0)) {
            pr("Sweep! in %s\n",
-              xyas(sect.sct_x, sect.sct_y, pp->pln_own));
+              xyas(sect.sct_x, sect.sct_y, player->cnum));
            mines_there--;
            found = 1;
        }
@@ -1019,123 +953,13 @@ plane_sweep(struct emp_qelem *plane_list, coord x, coord y)
     putsect(&sect);
 }
 
-void
-count_planes(struct shpstr *sp)
-{
-    struct nstr_item ni;
-    struct plnstr plane;
-    struct plchrstr *pcp;
-    struct mchrstr *mcp;
-    int nplane = 0;
-    int nchoppers = 0;
-    int nxlight = 0;
-
-    if (sp->shp_effic < SHIP_MINEFF)
-       return;
-
-    mcp = &mchr[(int)sp->shp_type];
-    snxtitem_xy(&ni, EF_PLANE, sp->shp_x, sp->shp_y);
-    while (nxtitem(&ni, &plane)) {
-       if (plane.pln_own == 0)
-           continue;
-       if (plane.pln_ship == sp->shp_uid) {
-           pcp = &plchr[(int)plane.pln_type];
-           if ((pcp->pl_flags & P_K) && (nchoppers < mcp->m_nchoppers))
-               nchoppers++;
-           else if ((pcp->pl_flags & P_E) && (nxlight < mcp->m_nxlight))
-               nxlight++;
-           else if ((pcp->pl_flags & P_L) || (pcp->pl_flags & P_M))
-               nplane++;
-       }
-    }
-
-    if (nplane != sp->shp_nplane ||
-       nxlight != sp->shp_nxlight || nchoppers != sp->shp_nchoppers) {
-       sp->shp_nplane = nplane;
-       sp->shp_nxlight = nxlight;
-       sp->shp_nchoppers = nchoppers;
-       putship(sp->shp_uid, sp);
-    }
-}
-
-void
-count_land_planes(struct lndstr *lp)
-{
-    struct nstr_item ni;
-    struct plnstr plane;
-    int nplane = 0;
-
-    if (lp->lnd_effic < LAND_MINEFF)
-       return;
-
-    snxtitem_all(&ni, EF_PLANE);
-    while (nxtitem(&ni, &plane)) {
-       if (plane.pln_own == 0)
-           continue;
-       if (plane.pln_land == lp->lnd_uid)
-           nplane++;
-    }
-
-    if (lp->lnd_nxlight != nplane) {
-       lp->lnd_nxlight = nplane;
-       putland(lp->lnd_uid, lp);
-    }
-}
-
-int
-count_sect_planes(struct sctstr *sp)
-{
-    int count = 0;
-    struct nstr_item ni;
-    struct plnstr plane;
-
-    snxtitem_all(&ni, EF_PLANE);
-    while (nxtitem(&ni, &plane)) {
-       if (!plane.pln_own)
-           continue;
-       if (plane.pln_flags & PLN_LAUNCHED)
-           continue;
-       if (plane.pln_x == sp->sct_x && plane.pln_y == sp->sct_y)
-           ++count;
-    }
-
-    return count;
-}
-
-int
-put_plane_on_land(struct plnstr *plane, struct lndstr *land)
-{
-    struct plchrstr *pcp;
-
-    pcp = &plchr[(int)plane->pln_type];
-
-    if (((int)plane->pln_land) == ((int)land->lnd_uid))
-       return 1;               /* Already on unit */
-
-    /* Try to put on unit as an xlight plane */
-    if ((pcp->pl_flags & P_E) &&
-       (land->lnd_nxlight < land->lnd_maxlight)) {
-
-       land->lnd_nxlight++;
-       plane->pln_x = land->lnd_x;
-       plane->pln_y = land->lnd_y;
-       plane->pln_land = land->lnd_uid;
-       putland(land->lnd_uid, land);
-       putplane(plane->pln_uid, plane);
-       return 1;
-    }
-
-    /* We have failed */
-    return 0;
-}
-
 int
 pln_hitchance(struct plnstr *pp, int hardtarget, int type)
 {
     struct plchrstr *pcp = plchr + pp->pln_type;
     double tfact = (double)(pp->pln_tech - pcp->pl_tech) /
        (pp->pln_tech - pcp->pl_tech / 2);
-    int acc = pp->pln_acc;
+    int acc = pln_acc(pp);
     int hitchance;
 
     if (type == EF_SHIP) {
@@ -1155,40 +979,26 @@ pln_hitchance(struct plnstr *pp, int hardtarget, int type)
     return hitchance;
 }
 
-/* return 0 if there was a nuclear detonation */
-
 int
-pln_damage(struct plnstr *pp, coord x, coord y, char type, int *nukedamp,
-          int noisy)
+pln_damage(struct plnstr *pp, char type, int noisy)
 {
-    struct nukstr nuke;
     struct plchrstr *pcp = plchr + pp->pln_type;
-    int i;
-    int hitroll;
+    int load, i, hitroll, aim, len;
     int dam = 0;
-    int aim;
     int effective = 1;
     int pinbomber = 0;
+    char buf[80];
 
-    if (pp->pln_nuketype != -1) {
-       if (nuk_on_plane(&nuke, pp->pln_uid) >= 0) {
-           mpr(pp->pln_own, "Releasing RV's for %s detonation...\n",
-               pp->pln_flags & PLN_AIRBURST ? "airburst" : "groundburst");
-           pp->pln_nuketype = -1;
-           *nukedamp = detonate(&nuke, x, y,
-                                pp->pln_flags & PLN_AIRBURST);
-           return 0;
-       }
-       CANT_REACH();
-    }
-    *nukedamp = 0;
+    if (CANT_HAPPEN(nuk_on_plane(pp) >= 0))
+       return 0;
 
-    if (!pp->pln_load)         /* e.g. ab, blowing up on launch pad */
+    load = pln_load(pp);
+    if (!load)                /* e.g. ab, blowing up on launch pad */
        return 0;
 
-    i = roll(pp->pln_load) + 1;
-    if (i > pp->pln_load)
-       i = pp->pln_load;
+    i = roll(load) + 1;
+    if (i > load)
+       i = load;
 
     if (pcp->pl_flags & P_M) {
        if (pcp->pl_flags & P_MAR)
@@ -1196,39 +1006,43 @@ pln_damage(struct plnstr *pp, coord x, coord y, char type, int *nukedamp,
     } else if (pcp->pl_flags & P_T)
        pinbomber = 1;
 
-    aim = 100 - pp->pln_acc;
+    aim = pln_acc(pp);
     if (type == 's') {
-       if (pinbomber) {
-           aim = pp->pln_acc;
-           effective = 0;
-       }
-       aim += 30;
+       effective = !pinbomber;
+       aim = 30 + (pinbomber ? aim : 100 - aim);
     } else {
-       if (!pinbomber) {
-           effective = 0;
-       }
+       effective = pinbomber;
+       aim = 100 - aim;
     }
+
+    len = 0;
     while (i--) {
        dam += roll(6);
        hitroll = roll(100);
        if (hitroll >= 90) {
            dam += 8;
            if (noisy)
-               mpr(pp->pln_own, "BLAM");
+               len += sprintf(buf + len, "BLAM");
        } else if (hitroll < aim) {
            dam += 5;
            if (noisy)
-               mpr(pp->pln_own, "Blam");
+               len += sprintf(buf + len, "Blam");
        } else {
            dam += 1;
            if (noisy)
-               mpr(pp->pln_own, "blam");
+               len += sprintf(buf + len, "blam");
+       }
+       if (noisy) {
+           if (len > 75) {
+               mpr(pp->pln_own, "%s\n", buf);
+               len = 0;
+           }
+           if (i)
+               len += sprintf(buf + len, "-");
        }
-       if (i && noisy)
-           mpr(pp->pln_own, "-");
     }
-    if (noisy)
-       mpr(pp->pln_own, "\n");
+    if (noisy && len)
+       mpr(pp->pln_own, "%s\n", buf);
     if (effective)
        dam *= 2;
     return dam;
@@ -1243,15 +1057,22 @@ pln_identchance(struct plnstr *pp, int hardtarget, int type)
 }
 
 int
-pln_mobcost(int dist, struct plnstr *pp, int flags)
+pln_mobcost(int dist, struct plnstr *pp, char mission)
 {
     double cost;
 
     cost = 20.0 / (pp->pln_effic / 100.0);
-    if ((flags & P_F) || (flags & P_ESC))
-       cost /= 2;
+    if (mission == 'e' || mission == 0)
+       cost /= 2;              /* escort or intercept */
 
-    return ldround(cost * dist / pp->pln_range_max + 5, 1);
+    return ldround(cost * dist / pln_range_max(pp) + 5, 1);
+}
+
+int
+pln_is_in_orbit(struct plnstr *pp)
+{
+    return (plchr[pp->pln_type].pl_flags & (P_M | P_O)) == P_O
+       && (pp->pln_flags & PLN_LAUNCHED);
 }
 
 /*
@@ -1261,21 +1082,14 @@ void
 pln_set_tech(struct plnstr *pp, int tlev)
 {
     struct plchrstr *pcp = plchr + pp->pln_type;
-    int tech_diff = tlev - pcp->pl_tech;
-    int limited_range = pp->pln_range < pp->pln_range_max;
-
-    if (CANT_HAPPEN(tech_diff < 0)) {
-      tlev -= tech_diff;
-      tech_diff = 0;
-    }
+    int limited_range = pp->pln_range < pln_range_max(pp);
+    int range_max;
 
+    if (CANT_HAPPEN(tlev < pcp->pl_tech))
+       tlev = pcp->pl_tech;
     pp->pln_tech = tlev;
-    pp->pln_att = PLN_ATTDEF(pcp->pl_att, tech_diff);
-    pp->pln_def = PLN_ATTDEF(pcp->pl_def, tech_diff);
-    pp->pln_acc = PLN_ACC(pcp->pl_acc, tech_diff);
-    pp->pln_range_max = PLN_RAN(pcp->pl_range, tech_diff);
-    pp->pln_load = PLN_LOAD(pcp->pl_load, tech_diff);
-
-    if (!limited_range || pp->pln_range > pp->pln_range_max)
-       pp->pln_range = pp->pln_range_max;
+
+    range_max = pln_range_max(pp);
+    if (!limited_range || pp->pln_range > range_max)
+       pp->pln_range = range_max;
 }