]> git.pond.sub.org Git - empserver/blobdiff - src/lib/commands/mine.c
Make sure lmine can't lay seamines even when sector changes
[empserver] / src / lib / commands / mine.c
index 156206aca1ad227d4c0f1e91d7169500dfd42c9b..21937453dbf1dd9fb4431a1e59796cb6531a4d14 100644 (file)
@@ -1,6 +1,6 @@
 /*
  *  Empire - A multi-player, client/server Internet based war game.
- *  Copyright (C) 1986-2000, Dave Pare, Jeff Bailey, Thomas Ruschak,
+ *  Copyright (C) 1986-2009, Dave Pare, Jeff Bailey, Thomas Ruschak,
  *                           Ken Stevens, Steve McClure
  *
  *  This program is free software; you can redistribute it and/or modify
  *
  *  ---
  *
- *  See the "LEGAL", "LICENSE", "CREDITS" and "README" files for all the
- *  related information and legal notices. It is expected that any future
- *  projects/authors will amend these files as needed.
+ *  See files README, COPYING and CREDITS in the root of the source
+ *  tree for related information and legal notices.  It is expected
+ *  that future projects/authors will amend these files as needed.
  *
  *  ---
  *
  *  mine.c: Lay mines from ships or units
- * 
+ *
  *  Known contributors to this file:
- *     
+ *
  */
 
-#include "misc.h"
-#include "player.h"
-#include "var.h"
-#include "ship.h"
-#include "land.h"
-#include "sect.h"
-#include "nat.h"
-#include "xy.h"
-#include "nsc.h"
-#include "file.h"
+#include <config.h>
+
 #include "commands.h"
+#include "land.h"
+#include "map.h"
+#include "ship.h"
 
 /*
  * format: mine <SHIPS> <NUMBER MINES>
@@ -56,32 +51,29 @@ mine(void)
     int mines;
     int shells;
     int mines_avail;
-    int mines_there;
 
-    if (!snxtitem(&ni, EF_SHIP, player->argp[1]))
+    if (!snxtitem(&ni, EF_SHIP, player->argp[1], NULL))
        return RET_SYN;
-    mines =
-       onearg(player->argp[2], "Drop how many mines from each ship?  ");
+    mines = onearg(player->argp[2],
+                  "Drop how many mines from each ship? ");
     if (mines <= 0)
        return RET_SYN;
-    while (nxtitem(&ni, (s_char *)&ship)) {
+    while (nxtitem(&ni, &ship)) {
        if (!player->owner)
            continue;
        mp = &mchr[(int)ship.shp_type];
        if ((mp->m_flags & M_MINE) == 0)
            continue;
-       if ((shells = getvar(V_SHELL, (s_char *)&ship, EF_SHIP)) == 0)
+       if ((shells = ship.shp_item[I_SHELL]) == 0)
            continue;
-       mines_avail = min(shells, mines);
+       mines_avail = MIN(shells, mines);
        if (getsect(ship.shp_x, ship.shp_y, &sect) == 0 ||
            (sect.sct_type != SCT_WATER && sect.sct_type != SCT_BSPAN)) {
            pr("You can't lay mines there!!\n");
            continue;
        }
-       mines_there = getvar(V_MINE, (s_char *)&sect, EF_SECTOR);
-       putvar(V_SHELL, shells - mines_avail, (s_char *)&ship, EF_SHIP);
-       putvar(V_MINE, mines_avail + mines_there, (s_char *)&sect,
-              EF_SECTOR);
+       sect.sct_mines = MIN(sect.sct_mines + mines_avail, MINES_MAX);
+       ship.shp_item[I_SHELL] = shells - mines_avail;
        putsect(&sect);
        ship.shp_mission = 0;
        putship(ship.shp_uid, &ship);
@@ -104,66 +96,80 @@ landmine(void)
     struct lchrstr *lp;
     struct nstr_item ni;
     int shells;
-    int mines_there;
     int mines_wanted;
     int mines_laid;
     int total_mines_laid;
-    s_char prompt[128];
+    char prompt[128];
 
-    if (!snxtitem(&ni, EF_LAND, player->argp[1]))
+    if (!snxtitem(&ni, EF_LAND, player->argp[1], NULL))
        return RET_SYN;
-    while (nxtitem(&ni, (s_char *)&land)) {
+    while (nxtitem(&ni, &land)) {
        if (!player->owner)
            continue;
        lp = &lchr[(int)land.lnd_type];
        if (!(lp->l_flags & L_ENGINEER))
            continue;
+       if (land.lnd_ship >= 0 || land.lnd_land >= 0) {
+           pr("%s is on a %s\n", prland(&land),
+              land. lnd_ship >= 0 ? "ship" : "land unit");
+           continue;
+       }
        if (land.lnd_mobil < 1) {
-           pr("Unit %d is out of mobility\n", land.lnd_uid);
+           pr("%s is out of mobility\n", prland(&land));
            continue;
        }
        resupply_commod(&land, I_SHELL);
-       if (!(shells = getvar(V_SHELL, (s_char *)&land, EF_LAND)))
+       putland(land.lnd_uid, &land);
+       if (!(shells = land.lnd_item[I_SHELL]))
            continue;
-       shells = min(shells, land.lnd_mobil);
+       shells = MIN(shells, land.lnd_mobil);
        if (!getsect(land.lnd_x, land.lnd_y, &sect) ||
            sect.sct_type == SCT_WATER || sect.sct_type == SCT_BSPAN) {
            pr("You can't lay mines there!!\n");
            continue;
        }
-       mines_there = getvar(V_MINE, (s_char *)&sect, EF_SECTOR);
        if (sect.sct_own == sect.sct_oldown)
            pr("There are currently %d mines in %s\n",
-              mines_there, xyas(sect.sct_x, sect.sct_y, player->cnum));
-       sprintf(prompt, "Drop how many mines from %s?  ", prland(&land));
+              sect.sct_mines, xyas(sect.sct_x, sect.sct_y, player->cnum));
+       sprintf(prompt, "Drop how many mines from %s? ", prland(&land));
        mines_wanted = onearg(player->argp[2], prompt);
-       if (!check_land_ok(&land))
+       if (mines_wanted < 0)
+           return RET_FAIL;
+       if (mines_wanted == 0)
            continue;
-       if (mines_wanted <= 0)
+       if (!check_land_ok(&land) || !check_sect_ok(&sect))
            continue;
        land.lnd_mission = 0;
        total_mines_laid = 0;
        while (shells > 0 && total_mines_laid < mines_wanted) {
-           mines_laid = min(shells, mines_wanted - total_mines_laid);
-           putvar(V_SHELL, shells - mines_laid, (s_char *)&land, EF_LAND);
+           mines_laid = MIN(shells, mines_wanted - total_mines_laid);
+           land.lnd_item[I_SHELL] = shells - mines_laid;
            land.lnd_mobil -= mines_laid;
            putland(land.lnd_uid, &land);
-           resupply_commod(&land, I_SHELL);
+           resupply_commod(&land, I_SHELL);    /* Get more shells */
            putland(land.lnd_uid, &land);
            total_mines_laid += mines_laid;
-           shells = getvar(V_SHELL, (s_char *)&land, EF_LAND);
-           shells = min(shells, land.lnd_mobil);
+           shells = land.lnd_item[I_SHELL];
+           shells = MIN(shells, land.lnd_mobil);
        }
        getsect(sect.sct_x, sect.sct_y, &sect);
-       putvar(V_MINE, total_mines_laid + mines_there, (s_char *)&sect,
-              EF_SECTOR);
+       sect.sct_mines = MIN(sect.sct_mines + total_mines_laid, MINES_MAX);
        putsect(&sect);
-       if (shells)
-           pr("%s laid a total of %d mines in %s\n",
+       if (total_mines_laid == mines_wanted) {
+           pr("%s laid a total of %d mines in %s",
               prland(&land), total_mines_laid,
               xyas(sect.sct_x, sect.sct_y, land.lnd_own));
-       else
-           pr("%s ran out of %s before it could finish the job\nOnly %d mines were laid in %s\n", prland(&land), land.lnd_mobil > 0 ? "supply" : "mobility", total_mines_laid, xyas(sect.sct_x, sect.sct_y, land.lnd_own));
+           if (!shells)
+               pr(" but is now out of supply\n");
+           else
+               pr("\n");
+       } else
+           pr("%s ran out of %s before it could finish the job\n"
+              "Only %d mines were laid in %s\n",
+              prland(&land),
+              land.lnd_mobil > 0 ? "supply" : "mobility",
+              total_mines_laid,
+              xyas(sect.sct_x, sect.sct_y, land.lnd_own));
     }
     return RET_OK;
 }