]> git.pond.sub.org Git - empserver/blobdiff - src/lib/commands/head.c
Clean up dead stores
[empserver] / src / lib / commands / head.c
index 295f292be1aed18cb39244210cff5e472e955b7b..5f7ab0310f772cdf2bb7d7a215a6394bc6087155 100644 (file)
@@ -1,6 +1,6 @@
 /*
  *  Empire - A multi-player, client/server Internet based war game.
- *  Copyright (C) 1986-2005, Dave Pare, Jeff Bailey, Thomas Ruschak,
+ *  Copyright (C) 1986-2010, 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.
  *
  *  ---
  *
  *  head.c: Print headlines of the Empire News
- * 
+ *
  *  Known contributors to this file:
- *     
+ *
  */
 
 #include <config.h>
 
-#include "misc.h"
-#include "player.h"
-#include "news.h"
-#include "nat.h"
-#include "file.h"
-#include "xy.h"
-#include "nsc.h"
 #include "commands.h"
+#include "news.h"
 
 struct histstr {
     int h_past;
     int h_recent;
 };
 
-static s_char *head_meanwhile(int val);
-static s_char *head_describe(struct histstr *hp, int what);
+static char *head_meanwhile(int val);
+static void head_describe(struct histstr *, int, char *, char *);
 static int head_printscoop(struct histstr (*hist)[MAXNOC], natid ano,
                           natid vno);
 static int head_findscoop(struct histstr (*hist)[MAXNOC],
-                         register natid maxcnum, natid *ano, natid *vno);
+                         natid maxcnum, natid *ano, natid *vno);
 
 int
 head(void)
 {
-    register int i;
-    register struct histstr *hp;
-    register natid maxcnum;
+    struct histstr *hp;
+    natid maxcnum;
     time_t now;
     int severity;
     int scoop;
@@ -71,20 +64,15 @@ head(void)
     natid actor;
     natid victim;
     struct nstr_item nstr;
-    int n;
+    int i, n;
 
     (void)time(&now);
     natp = getnatp(player->cnum);
-    if (player->argp[1] != 0 && *player->argp[1] != 0) {
+    if (player->argp[1] && *player->argp[1]) {
        news_per = days(atoi(player->argp[1]));
        if (news_per > days(3))
            news_per = days(3);
     } else
-       /* replaced the following line with the one under it to
-        * fix headlines which never change.
-        * 3/3/90 bailey@math-cs.kent.edu
-        */
-/*             news_per = natp->nat_newstim; */
        news_per = now - natp->nat_newstim;
     pr("\n        -=[  EMPIRE NEWS  ]=-\n");
     pr("::::::::::::::::::::::::::::::::::::::::::::::::::\n");
@@ -96,6 +84,8 @@ head(void)
     snxtitem_all(&nstr, EF_NEWS);
     maxcnum = 0;
     while (nxtitem(&nstr, &news)) {
+       if (!news.nws_vrb || CANT_HAPPEN(news.nws_vrb > N_MAX_VERB))
+           continue;
        news_age = now - news.nws_when;
        if (news_age > news_per)
            continue;
@@ -103,7 +93,7 @@ head(void)
            continue;
        if ((i = rpt[(int)news.nws_vrb].r_good_will) == 0)
            continue;
-       if (news_age > (news_per / 2))
+       if (news_age > news_per / 2)
            hist[news.nws_ano][news.nws_vno].h_past += i;
        else
            hist[news.nws_ano][news.nws_vno].h_recent += i;
@@ -115,7 +105,7 @@ head(void)
     for (n = 0; n < 5; n++) {
        if ((scoop = head_findscoop(hist, maxcnum, &actor, &victim)) < 10)
            break;
-       severity = head_printscoop(hist, actor, victim);
+       head_printscoop(hist, actor, victim);
        hp = &hist[actor][victim];
        severity = hp->h_recent - hp->h_past;
        if (severity <= -scoop / 2 || severity >= scoop / 2) {
@@ -131,21 +121,21 @@ head(void)
 static int
 head_printscoop(struct histstr (*hist)[MAXNOC], natid ano, natid vno)
 {
-    register struct histstr *hp;
+    struct histstr *hp;
     int severity;
 
     hp = &hist[ano][vno];
     severity = abs(hp->h_past) > abs(hp->h_recent) ? 1 : 0;
     severity += (hp->h_past >= 0) ? 2 : 0;
     severity += (hp->h_recent >= 0) ? 4 : 0;
-    pr(head_describe(hp, severity), cname(ano), cname(vno));
+    head_describe(hp, severity, cname(ano), cname(vno));
     pr("\n");
     hp->h_past = 0;
     hp->h_recent = 0;
     return severity;
 }
 
-static s_char *
+static char *
 head_meanwhile(int val)
 {
     switch (val & 03) {
@@ -162,59 +152,62 @@ head_meanwhile(int val)
     return "";
 }
 
-static s_char *
-head_describe(struct histstr *hp, int what)
+static void
+head_describe(struct histstr *hp, int what, char *aname, char *vname)
 {
-    s_char *cp;
-
-    cp = 0;
     switch (what) {
     case 0:
        if (hp->h_recent > hp->h_past / 2)
-           cp = "Bad relations between %s and %s worsen!";
+           pr("Bad relations between %s and %s worsen!", aname, vname);
        else
-           cp = "Carnage wrought by %s on %s continues unabated!";
+           pr("Carnage wrought by %s on %s continues unabated!",
+              aname, vname);
        break;
     case 1:
        if (hp->h_recent < -16)
-           cp = "%s agression against %s has lessened slightly";
+           pr("%s agression against %s has lessened slightly",
+              aname, vname);
        else
-           cp = "Peace talks may occur between %s & %s";
+           pr("Peace talks may occur between %s & %s", aname, vname);
        break;
     case 2:
        if (hp->h_recent < -16) {
            if (hp->h_past > 0)
-               cp = " ! WAR !  Reversal of prior %s -- %s relations";
+               pr(" ! WAR !  Reversal of prior %s -- %s relations",
+                  aname, vname);
            else if (hp->h_recent >= -25)
-               cp = "VIOLENCE ERUPTS! -- %s wages war on %s";
+               pr("VIOLENCE ERUPTS! -- %s wages war on %s", aname, vname);
            else
-               cp = "%s wreaks havoc on %s!";
+               pr("%s wreaks havoc on %s!", aname, vname);
        } else
-           cp = "Breakdown in communication between %s & %s";
+           pr("Breakdown in communication between %s & %s", aname, vname);
        break;
     case 3:
-       cp = "FLASH!    %s turns on former ally, %s!";
+       pr("FLASH!    %s turns on former ally, %s!", aname, vname);
        break;
     case 4:
-       cp = "%s \"makes friends\" with %s";
+       pr("%s \"makes friends\" with %s", aname, vname);
        break;
     case 5:
        if (hp->h_past >= -25)
-           cp = "%s seems to have forgotten earlier disagreement with %s";
+           pr("%s seems to have forgotten earlier disagreement with %s",
+              aname, vname);
        else
-           cp = "Tensions ease as %s attacks on %s seem at an end";
+           pr("Tensions ease as %s attacks on %s seem at an end",
+              aname, vname);
        break;
     case 6:
-       cp = "%s good deeds further growing alliance with %s";
+       pr("%s good deeds further growing alliance with %s", aname, vname);
        break;
     case 7:
        if (hp->h_recent - hp->h_past < 12)
-           cp = "Honeymoon appears to be over between %s & %s";
+           pr("Honeymoon appears to be over between %s & %s",
+              aname, vname);
        else
-           cp = "Friendly relations between %s & %s have cooled";
+           pr("Friendly relations between %s & %s have cooled",
+              aname, vname);
        break;
     }
-    return cp;
 }
 
 /*
@@ -222,16 +215,12 @@ head_describe(struct histstr *hp, int what)
  * Pretty strange.
  */
 static int
-head_findscoop(struct histstr (*hist)[MAXNOC], register natid maxcnum,
+head_findscoop(struct histstr (*hist)[MAXNOC], natid maxcnum,
               natid *ano, natid *vno)
 {
-    register struct histstr *hp;
-    register int i;
-    register int j;
-    register int k;
-    int scoop;
-    natid actor;
-    natid victim;
+    struct histstr *hp;
+    int i, j, k, scoop;
+    natid actor, victim;
 
     scoop = 9;
     actor = 0;