]> git.cameronkatri.com Git - bsdgames-darwin.git/blobdiff - sail/pl_7.c
Fix sign-compare problem that crept in due to working from an old tree.
[bsdgames-darwin.git] / sail / pl_7.c
index c1525b4c3a4270453895dd03c1616da53c95228e..fe8c5f29f7e7a3ec75bd5d469f4d5a8467b0bfad 100644 (file)
@@ -1,4 +1,4 @@
-/*     $NetBSD: pl_7.c,v 1.7 1997/10/13 19:45:39 christos Exp $        */
+/*     $NetBSD: pl_7.c,v 1.36 2009/03/15 20:11:24 dholland Exp $       */
 
 /*
  * Copyright (c) 1983, 1993
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
- * 3. All advertising materials mentioning features or use of this software
- *    must display the following acknowledgement:
- *     This product includes software developed by the University of
- *     California, Berkeley and its contributors.
- * 4. Neither the name of the University nor the names of its contributors
+ * 3. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
  *
 #if 0
 static char sccsid[] = "@(#)pl_7.c     8.1 (Berkeley) 5/31/93";
 #else
-__RCSID("$NetBSD: pl_7.c,v 1.7 1997/10/13 19:45:39 christos Exp $");
+__RCSID("$NetBSD: pl_7.c,v 1.36 2009/03/15 20:11:24 dholland Exp $");
 #endif
 #endif /* not lint */
 
-#include <sys/ttydefaults.h>
-#include "player.h"
-#ifdef __STDC__
+#include <curses.h>
+#include <err.h>
+#include <errno.h>
+#include <signal.h>
 #include <stdarg.h>
-#else
-#include <varargs.h>
-#endif
-#include <unistd.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include "array.h"
+#include "extern.h"
+#include "player.h"
+#include "display.h"
+
+/*
+ * Use values above KEY_MAX for custom keycodes. (blymn@ says this is ok)
+ */
+#define KEY_ESC(ch) (KEY_MAX+10+ch)
 
 
 /*
  * Display interface
  */
 
-static char sc_hasprompt;
-static char *sc_prompt;
-static char *sc_buf;
-static int sc_line;
+static void draw_view(void);
+static void draw_turn(void);
+static void draw_stat(void);
+static void draw_slot(void);
+static void draw_board(void);
+
+static struct stringarray *sc_lines;
+static unsigned sc_scrollup;
+static bool sc_hasprompt;
+static bool sc_hideprompt;
+static const char *sc_prompt;
+static const char *sc_buf;
+
+static WINDOW *view_w;
+static WINDOW *turn_w;
+static WINDOW *stat_w;
+static WINDOW *slot_w;
+static WINDOW *scroll_w;
+
+static bool obp[3];
+static bool dbp[3];
+
+int done_curses;
+int loaded, fired, changed, repaired;
+int dont_adjust;
+int viewrow, viewcol;
+char movebuf[sizeof SHIP(0)->file->movebuf];
+int player;
+struct ship *ms;               /* memorial structure, &cc->ship[player] */
+struct File *mf;               /* ms->file */
+struct shipspecs *mc;          /* ms->specs */
+
+////////////////////////////////////////////////////////////
+// overall initialization
+
+static
+void
+define_esc_key(int ch)
+{
+       char seq[3] = { '\x1b', ch, 0 };
+
+       define_key(seq, KEY_ESC(ch));
+}
 
 void
-initscreen()
+initscreen(void)
 {
-       /* initscr() already done in SCREENTEST() */
+       int ch;
+
+       sc_lines = stringarray_create();
+       if (sc_lines == NULL) {
+               err(1, "malloc");
+       }
+
+       if (signal(SIGTSTP, SIG_DFL) == SIG_ERR) {
+               err(1, "signal(SIGTSTP)");
+       }
+
+       if (initscr() == NULL) {
+               errx(1, "Can't sail on this terminal.");
+       }
+       if (STAT_R >= COLS || SCROLL_Y <= 0) {
+               errx(1, "Window/terminal not large enough.");
+       }
+
        view_w = newwin(VIEW_Y, VIEW_X, VIEW_T, VIEW_L);
        slot_w = newwin(SLOT_Y, SLOT_X, SLOT_T, SLOT_L);
        scroll_w = newwin(SCROLL_Y, SCROLL_X, SCROLL_T, SCROLL_L);
        stat_w = newwin(STAT_Y, STAT_X, STAT_T, STAT_L);
        turn_w = newwin(TURN_Y, TURN_X, TURN_T, TURN_L);
-       done_curses++;
-       (void) leaveok(view_w, 1);
-       (void) leaveok(slot_w, 1);
-       (void) leaveok(stat_w, 1);
-       (void) leaveok(turn_w, 1);
+
+       if (view_w == NULL ||
+           slot_w == NULL ||
+           scroll_w == NULL ||
+           stat_w == NULL ||
+           turn_w == NULL) {
+               endwin();
+               errx(1, "Curses initialization failed.");
+       }
+
+       leaveok(view_w, 1);
+       leaveok(slot_w, 1);
+       leaveok(stat_w, 1);
+       leaveok(turn_w, 1);
        noecho();
-       crmode();
+       cbreak();
+
+       /*
+        * Define esc-x keys
+        */
+       for (ch = 0; ch < 127; ch++) {
+               define_esc_key(ch);
+       }
+
+       done_curses++;
 }
 
 void
-cleanupscreen()
+cleanupscreen(void)
 {
        /* alarm already turned off */
        if (done_curses) {
-               (void) wmove(scroll_w, SCROLL_Y - 1, 0);
-               (void) wclrtoeol(scroll_w);
-               draw_screen();
+               wmove(scroll_w, SCROLL_Y - 1, 0);
+               wclrtoeol(scroll_w);
+               display_redraw();
                endwin();
        }
 }
 
-/*ARGSUSED*/
-void
-newturn(n)
-       int n;
+////////////////////////////////////////////////////////////
+// scrolling message area
+
+static void
+scrollarea_add(const char *text)
 {
-       repaired = loaded = fired = changed = 0;
-       movebuf[0] = '\0';
-
-       (void) alarm(0);
-       if (mf->readyL & R_LOADING)
-               if (mf->readyL & R_DOUBLE)
-                       mf->readyL = R_LOADING;
-               else
-                       mf->readyL = R_LOADED;
-       if (mf->readyR & R_LOADING)
-               if (mf->readyR & R_DOUBLE)
-                       mf->readyR = R_LOADING;
-               else
-                       mf->readyR = R_LOADED;
-       if (!hasdriver)
-               Write(W_DDEAD, SHIP(0), 0, 0, 0, 0, 0);
-
-       if (sc_hasprompt) {
-               (void) wmove(scroll_w, sc_line, 0);
-               (void) wclrtoeol(scroll_w);
+       char *copy;
+       int errsave;
+
+       copy = strdup(text);
+       if (copy == NULL) {
+               goto nomem;
+       }
+       if (stringarray_add(sc_lines, copy, NULL)) {
+               goto nomem;
+       }
+       return;
+
+nomem:
+       /*
+        * XXX this should use leave(), but that won't
+        * currently work right.
+        */
+       errsave = errno;
+#if 0
+       leave(LEAVE_MALLOC);
+#else
+       cleanupscreen();
+       sync_close(!hasdriver);
+       errno = errsave;
+       err(1, "malloc");
+#endif
+}
+
+static void
+draw_scroll(void)
+{
+       unsigned total_lines;
+       unsigned visible_lines;
+       unsigned index_of_top;
+       unsigned index_of_y;
+       unsigned y;
+       unsigned cursorx;
+
+       werase(scroll_w);
+
+       visible_lines = SCROLL_Y - 1;
+
+       total_lines = stringarray_num(sc_lines);
+       if (total_lines > visible_lines) {
+               index_of_top = total_lines - visible_lines;
+       } else {
+               index_of_top = 0;
+       }
+       if (index_of_top < sc_scrollup) {
+               index_of_top = 0;
+       } else {
+               index_of_top -= sc_scrollup;
+       }
+
+       /* XXX: SCROLL_Y and whatnot should be unsigned too */
+       for (y = 0; y < (unsigned) (SCROLL_Y - 1); y++) {
+               index_of_y = index_of_top + y;
+               if (index_of_y >= total_lines) {
+                       break;
+               }
+               wmove(scroll_w, y, 0);
+               waddstr(scroll_w, stringarray_get(sc_lines, index_of_y));
+       }
+       if (sc_hasprompt && !sc_hideprompt) {
+               wmove(scroll_w, SCROLL_Y-1, 0);
+               waddstr(scroll_w, sc_prompt);
+               waddstr(scroll_w, sc_buf);
+               cursorx = strlen(sc_prompt) + strlen(sc_buf);
+               wmove(scroll_w, SCROLL_Y-1, cursorx);
+       }
+       else {
+               wmove(scroll_w, SCROLL_Y-1, 0);
        }
-       if (Sync() < 0)
-               leave(LEAVE_SYNC);
-       if (!hasdriver)
-               leave(LEAVE_DRIVER);
-       if (sc_hasprompt)
-               (void) wprintw(scroll_w, "%s%s", sc_prompt, sc_buf);
-
-       if (turn % 50 == 0)
-               Write(W_ALIVE, SHIP(0), 0, 0, 0, 0, 0);
-       if (mf->FS && (!mc->rig1 || windspeed == 6))
-               Write(W_FS, ms, 0, 0, 0, 0, 0);
-       if (mf->FS == 1)
-               Write(W_FS, ms, 0, 2, 0, 0, 0);
-
-       if (mf->struck)
-               leave(LEAVE_QUIT);
-       if (mf->captured != 0)
-               leave(LEAVE_CAPTURED);
-       if (windspeed == 7)
-               leave(LEAVE_HURRICAN);
-
-       adjustview();
-       draw_screen();
-
-       (void) signal(SIGALRM, newturn);
-       (void) alarm(7);
 }
 
 /*VARARGS2*/
 void
-#ifdef __STDC__
 Signal(const char *fmt, struct ship *ship, ...)
-#else
-Signal(va_alist)
-       va_dcl
-#endif
 {
        va_list ap;
        char format[BUFSIZ];
-#ifndef __STDC__
-       const char *fmt;
-       struct ship *ship;
+       char buf[BUFSIZ];
 
-       va_start(ap);
-       fmt = va_arg(ap, const char *);
-       ship = va_arg(ap, struct ship *);
-#else
-       va_start(ap, ship);
-#endif
        if (!done_curses)
                return;
-       if (*fmt == '\7')
-               putchar(*fmt++);
+       va_start(ap, ship);
+       if (*fmt == '\a') {
+               beep();
+               fmt++;
+       }
        fmtship(format, sizeof(format), fmt, ship);
-       (void) vwprintw(scroll_w, format, ap);
+       vsnprintf(buf, sizeof(buf), format, ap);
        va_end(ap);
-       Scroll();
+       scrollarea_add(buf);
 }
 
 /*VARARGS2*/
 void
-#ifdef __STDC__
 Msg(const char *fmt, ...)
-#else
-Msg(va_alist)
-       va_dcl
-#endif
 {
        va_list ap;
-#ifndef __STDC__
-       const char *fmt;
-
-       va_start(ap);
-       fmt = va_arg(ap, const char *);
-#else
-       va_start(ap, fmt);
-#endif
+       char buf[BUFSIZ];
 
        if (!done_curses)
                return;
-       if (*fmt == '\7')
-               putchar(*fmt++);
-       (void) vwprintw(scroll_w, fmt, ap);
+       va_start(ap, fmt);
+       if (*fmt == '\a') {
+               beep();
+               fmt++;
+       }
+       vsnprintf(buf, sizeof(buf), fmt, ap);
        va_end(ap);
-       Scroll();
+       scrollarea_add(buf);
 }
 
 void
-Scroll()
+prompt(const char *p, struct ship *ship)
 {
-       if (++sc_line >= SCROLL_Y)
-               sc_line = 0;
-       (void) wmove(scroll_w, sc_line, 0);
-       (void) wclrtoeol(scroll_w);
+       static char buf[BUFSIZ];
+
+       fmtship(buf, sizeof(buf), p, ship);
+       sc_prompt = buf;
+       sc_buf = "";
+       sc_hasprompt = true;
 }
 
-void
-prompt(p, ship)
-char *p;
-struct ship *ship;
+static void
+endprompt(void)
 {
-       static char buf[60];
+       sc_prompt = NULL;
+       sc_buf = NULL;
+       sc_hasprompt = false;
+}
 
-       if (ship != 0) {
-               (void)sprintf(buf, p, ship->shipname, colours(ship),
-                       sterncolour(ship));
-               p = buf;
-       }
-       sc_prompt = p;
-       sc_buf = "";
-       sc_hasprompt = 1;
-       (void) waddstr(scroll_w, p);
+/*
+ * Next two functions called from newturn() to poke display. Shouldn't
+ * exist... XXX
+ */
+
+void
+display_hide_prompt(void)
+{
+       sc_hideprompt = true;
+       draw_scroll();
+       wrefresh(scroll_w);
 }
 
 void
-endprompt(flag)
-char flag;
+display_reshow_prompt(void)
 {
-       sc_hasprompt = 0;
-       if (flag)
-               Scroll();
+       sc_hideprompt = false;
+       draw_scroll();
+       wrefresh(scroll_w);
 }
 
+
 int
-sgetch(p, ship, flag)
-char *p;
-struct ship *ship;
-char flag;
+sgetch(const char *p, struct ship *ship, int flag)
 {
        int c;
+       char input[2];
 
        prompt(p, ship);
+       input[0] = '\0';
+       input[1] = '\0';
+       sc_buf = input;
        blockalarm();
-       (void) wrefresh(scroll_w);
+       draw_scroll();
+       wrefresh(scroll_w);
+       fflush(stdout);
        unblockalarm();
        while ((c = wgetch(scroll_w)) == EOF)
                ;
-       if (flag && c >= ' ' && c < 0x7f)
-               (void) waddch(scroll_w, c);
-       endprompt(flag);
+       if (flag && c >= ' ' && c < 0x7f) {
+               blockalarm();
+               input[0] = c;
+               draw_scroll();
+               wrefresh(scroll_w);
+               fflush(stdout);
+               unblockalarm();
+       }
+       endprompt();
        return c;
 }
 
 void
-sgetstr(pr, buf, n)
-char *pr;
-char *buf;
-int n;
+sgetstr(const char *pr, char *buf, int n)
 {
        int c;
        char *p = buf;
@@ -274,259 +372,331 @@ int n;
        for (;;) {
                *p = 0;
                blockalarm();
-               (void) wrefresh(scroll_w);
+               draw_scroll();
+               wrefresh(scroll_w);
+               fflush(stdout);
                unblockalarm();
                while ((c = wgetch(scroll_w)) == EOF)
                        ;
                switch (c) {
                case '\n':
                case '\r':
-                       endprompt(1);
+                       endprompt();
                        return;
                case '\b':
                        if (p > buf) {
-                               (void) waddstr(scroll_w, "\b \b");
+                               /*waddstr(scroll_w, "\b \b");*/
                                p--;
                        }
                        break;
                default:
                        if (c >= ' ' && c < 0x7f && p < buf + n - 1) {
                                *p++ = c;
-                               (void) waddch(scroll_w, c);
+                               /*waddch(scroll_w, c);*/
                        } else
-                               (void) putchar('\a');
+                               beep();
                }
        }
 }
 
+////////////////////////////////////////////////////////////
+// drawing of other panes
+
+void
+display_force_full_redraw(void)
+{
+       clear();
+}
+
 void
-draw_screen()
+display_redraw(void)
 {
+       draw_board();
        draw_view();
        draw_turn();
        draw_stat();
        draw_slot();
-       (void) wrefresh(scroll_w);              /* move the cursor */
+       draw_scroll();
+       /* move the cursor */
+       wrefresh(scroll_w);
+       /* paranoia */
+       fflush(stdout);
 }
 
-void
-draw_view()
+static void
+draw_view(void)
 {
        struct ship *sp;
 
-       (void) werase(view_w);
+       werase(view_w);
        foreachship(sp) {
                if (sp->file->dir
                    && sp->file->row > viewrow
                    && sp->file->row < viewrow + VIEW_Y
                    && sp->file->col > viewcol
                    && sp->file->col < viewcol + VIEW_X) {
-                       (void) wmove(view_w, sp->file->row - viewrow,
+                       wmove(view_w, sp->file->row - viewrow,
                                sp->file->col - viewcol);
-                       (void) waddch(view_w, colours(sp));
-                       (void) wmove(view_w,
+                       waddch(view_w, colours(sp));
+                       wmove(view_w,
                                sternrow(sp) - viewrow,
                                sterncol(sp) - viewcol);
-                       (void) waddch(view_w, sterncolour(sp));
+                       waddch(view_w, sterncolour(sp));
                }
        }
-       (void) wrefresh(view_w);
+       wrefresh(view_w);
 }
 
-void
-draw_turn()
+static void
+draw_turn(void)
 {
-       (void) wmove(turn_w, 0, 0);
-       (void) wprintw(turn_w, "%cTurn %d", dont_adjust?'*':'-', turn);
-       (void) wrefresh(turn_w);
+       wmove(turn_w, 0, 0);
+       wprintw(turn_w, "%cTurn %d", dont_adjust?'*':'-', turn);
+       wrefresh(turn_w);
 }
 
-void
-draw_stat()
+static void
+draw_stat(void)
 {
-       (void) wmove(stat_w, STAT_1, 0);
-       (void) wprintw(stat_w, "Points  %3d\n", mf->points);
-       (void) wprintw(stat_w, "Fouls    %2d\n", fouled(ms));
-       (void) wprintw(stat_w, "Grapples %2d\n", grappled(ms));
+       wmove(stat_w, STAT_1, 0);
+       wprintw(stat_w, "Points  %3d\n", mf->points);
+       wprintw(stat_w, "Fouls    %2d\n", fouled(ms));
+       wprintw(stat_w, "Grapples %2d\n", grappled(ms));
 
-       (void) wmove(stat_w, STAT_2, 0);
-       (void) wprintw(stat_w, "    0 %c(%c)\n",
+       wmove(stat_w, STAT_2, 0);
+       wprintw(stat_w, "    0 %c(%c)\n",
                maxmove(ms, winddir + 3, -1) + '0',
                maxmove(ms, winddir + 3, 1) + '0');
-       (void) waddstr(stat_w, "   \\|/\n");
-       (void) wprintw(stat_w, "   -^-%c(%c)\n",
+       waddstr(stat_w, "   \\|/\n");
+       wprintw(stat_w, "   -^-%c(%c)\n",
                maxmove(ms, winddir + 2, -1) + '0',
                maxmove(ms, winddir + 2, 1) + '0');
-       (void) waddstr(stat_w, "   /|\\\n");
-       (void) wprintw(stat_w, "    | %c(%c)\n",
+       waddstr(stat_w, "   /|\\\n");
+       wprintw(stat_w, "    | %c(%c)\n",
                maxmove(ms, winddir + 1, -1) + '0',
                maxmove(ms, winddir + 1, 1) + '0');
-       (void) wprintw(stat_w, "   %c(%c)\n",
+       wprintw(stat_w, "   %c(%c)\n",
                maxmove(ms, winddir, -1) + '0',
                maxmove(ms, winddir, 1) + '0');
 
-       (void) wmove(stat_w, STAT_3, 0);
-       (void) wprintw(stat_w, "Load  %c%c %c%c\n",
+       wmove(stat_w, STAT_3, 0);
+       wprintw(stat_w, "Load  %c%c %c%c\n",
                loadname[mf->loadL], readyname(mf->readyL),
                loadname[mf->loadR], readyname(mf->readyR));
-       (void) wprintw(stat_w, "Hull %2d\n", mc->hull);
-       (void) wprintw(stat_w, "Crew %2d %2d %2d\n",
+       wprintw(stat_w, "Hull %2d\n", mc->hull);
+       wprintw(stat_w, "Crew %2d %2d %2d\n",
                mc->crew1, mc->crew2, mc->crew3);
-       (void) wprintw(stat_w, "Guns %2d %2d\n", mc->gunL, mc->gunR);
-       (void) wprintw(stat_w, "Carr %2d %2d\n", mc->carL, mc->carR);
-       (void) wprintw(stat_w, "Rigg %d %d %d ", mc->rig1, mc->rig2, mc->rig3);
+       wprintw(stat_w, "Guns %2d %2d\n", mc->gunL, mc->gunR);
+       wprintw(stat_w, "Carr %2d %2d\n", mc->carL, mc->carR);
+       wprintw(stat_w, "Rigg %d %d %d ", mc->rig1, mc->rig2, mc->rig3);
        if (mc->rig4 < 0)
-               (void) waddch(stat_w, '-');
+               waddch(stat_w, '-');
        else
-               (void) wprintw(stat_w, "%d", mc->rig4);
-       (void) wrefresh(stat_w);
+               wprintw(stat_w, "%d", mc->rig4);
+       wrefresh(stat_w);
 }
 
 void
-draw_slot()
+draw_slot(void)
 {
+       int i;
+
        if (!boarding(ms, 0)) {
-               (void) mvwaddstr(slot_w, 0, 0, "   ");
-               (void) mvwaddstr(slot_w, 1, 0, "   ");
-       } else
-               (void) mvwaddstr(slot_w, 1, 0, "OBP");
+               mvwaddstr(slot_w, 0, 0, "   ");
+               mvwaddstr(slot_w, 1, 0, "   ");
+       } else {
+               wmove(slot_w, 0, 0);
+               for (i = 0; i < 3; i++) {
+                       waddch(slot_w, obp[i] ? '1'+i : ' ');
+               }
+               mvwaddstr(slot_w, 1, 0, "OBP");
+       }
        if (!boarding(ms, 1)) {
-               (void) mvwaddstr(slot_w, 2, 0, "   ");
-               (void) mvwaddstr(slot_w, 3, 0, "   ");
-       } else
-               (void) mvwaddstr(slot_w, 3, 0, "DBP");
+               mvwaddstr(slot_w, 2, 0, "   ");
+               mvwaddstr(slot_w, 3, 0, "   ");
+       } else {
+               wmove(slot_w, 2, 0);
+               for (i = 0; i < 3; i++) {
+                       waddch(slot_w, dbp[i] ? '1'+i : ' ');
+               }
+               mvwaddstr(slot_w, 3, 0, "DBP");
+       }
 
-       (void) wmove(slot_w, SLOT_Y-4, 0);
+       wmove(slot_w, SLOT_Y-4, 0);
        if (mf->RH)
-               (void) wprintw(slot_w, "%dRH", mf->RH);
+               wprintw(slot_w, "%dRH", mf->RH);
        else
-               (void) waddstr(slot_w, "   ");
-       (void) wmove(slot_w, SLOT_Y-3, 0);
+               waddstr(slot_w, "   ");
+       wmove(slot_w, SLOT_Y-3, 0);
        if (mf->RG)
-               (void) wprintw(slot_w, "%dRG", mf->RG);
+               wprintw(slot_w, "%dRG", mf->RG);
        else
-               (void) waddstr(slot_w, "   ");
-       (void) wmove(slot_w, SLOT_Y-2, 0);
+               waddstr(slot_w, "   ");
+       wmove(slot_w, SLOT_Y-2, 0);
        if (mf->RR)
-               (void) wprintw(slot_w, "%dRR", mf->RR);
+               wprintw(slot_w, "%dRR", mf->RR);
        else
-               (void) waddstr(slot_w, "   ");
+               waddstr(slot_w, "   ");
 
 #define Y      (SLOT_Y/2)
-       (void) wmove(slot_w, 7, 1);
-       (void) wprintw(slot_w,"%d", windspeed);
-       (void) mvwaddch(slot_w, Y, 0, ' ');
-       (void) mvwaddch(slot_w, Y, 2, ' ');
-       (void) mvwaddch(slot_w, Y-1, 0, ' ');
-       (void) mvwaddch(slot_w, Y-1, 1, ' ');
-       (void) mvwaddch(slot_w, Y-1, 2, ' ');
-       (void) mvwaddch(slot_w, Y+1, 0, ' ');
-       (void) mvwaddch(slot_w, Y+1, 1, ' ');
-       (void) mvwaddch(slot_w, Y+1, 2, ' ');
-       (void) wmove(slot_w, Y - dr[winddir], 1 - dc[winddir]);
+       wmove(slot_w, 7, 1);
+       wprintw(slot_w,"%d", windspeed);
+       mvwaddch(slot_w, Y, 0, ' ');
+       mvwaddch(slot_w, Y, 2, ' ');
+       mvwaddch(slot_w, Y-1, 0, ' ');
+       mvwaddch(slot_w, Y-1, 1, ' ');
+       mvwaddch(slot_w, Y-1, 2, ' ');
+       mvwaddch(slot_w, Y+1, 0, ' ');
+       mvwaddch(slot_w, Y+1, 1, ' ');
+       mvwaddch(slot_w, Y+1, 2, ' ');
+       wmove(slot_w, Y - dr[winddir], 1 - dc[winddir]);
        switch (winddir) {
        case 1:
        case 5:
-               (void) waddch(slot_w, '|');
+               waddch(slot_w, '|');
                break;
        case 2:
        case 6:
-               (void) waddch(slot_w, '/');
+               waddch(slot_w, '/');
                break;
        case 3:
        case 7:
-               (void) waddch(slot_w, '-');
+               waddch(slot_w, '-');
                break;
        case 4:
        case 8:
-               (void) waddch(slot_w, '\\');
+               waddch(slot_w, '\\');
                break;
        }
-       (void) mvwaddch(slot_w, Y + dr[winddir], 1 + dc[winddir], '+');
-       (void) wrefresh(slot_w);
+       mvwaddch(slot_w, Y + dr[winddir], 1 + dc[winddir], '+');
+       wrefresh(slot_w);
 }
 
 void
-draw_board()
+draw_board(void)
 {
        int n;
 
-       (void) clear();
-       (void) werase(view_w);
-       (void) werase(slot_w);
-       (void) werase(scroll_w);
-       (void) werase(stat_w);
-       (void) werase(turn_w);
-
-       sc_line = 0;
+       erase();
+       werase(view_w);
+       werase(slot_w);
+       werase(scroll_w);
+       werase(stat_w);
+       werase(turn_w);
 
-       (void) move(BOX_T, BOX_L);
+       move(BOX_T, BOX_L);
        for (n = 0; n < BOX_X; n++)
-               (void) addch('-');
-       (void) move(BOX_B, BOX_L);
+               addch('-');
+       move(BOX_B, BOX_L);
        for (n = 0; n < BOX_X; n++)
-               (void) addch('-');
+               addch('-');
        for (n = BOX_T+1; n < BOX_B; n++) {
-               (void) mvaddch(n, BOX_L, '|');
-               (void) mvaddch(n, BOX_R, '|');
+               mvaddch(n, BOX_L, '|');
+               mvaddch(n, BOX_R, '|');
        }
-       (void) mvaddch(BOX_T, BOX_L, '+');
-       (void) mvaddch(BOX_T, BOX_R, '+');
-       (void) mvaddch(BOX_B, BOX_L, '+');
-       (void) mvaddch(BOX_B, BOX_R, '+');
-       (void) refresh();
+       mvaddch(BOX_T, BOX_L, '+');
+       mvaddch(BOX_T, BOX_R, '+');
+       mvaddch(BOX_B, BOX_L, '+');
+       mvaddch(BOX_B, BOX_R, '+');
+       refresh();
 
+#if 0
 #define WSaIM "Wooden Ships & Iron Men"
-       (void) wmove(view_w, 2, (VIEW_X - sizeof WSaIM - 1) / 2);
-       (void) waddstr(view_w, WSaIM);
-       (void) wmove(view_w, 4, (VIEW_X - strlen(cc->name)) / 2);
-       (void) waddstr(view_w, cc->name);
-       (void) wrefresh(view_w);
-
-       (void) move(LINE_T, LINE_L);
-       (void) printw("Class %d %s (%d guns) '%s' (%c%c)",
+       wmove(view_w, 2, (VIEW_X - sizeof WSaIM - 1) / 2);
+       waddstr(view_w, WSaIM);
+       wmove(view_w, 4, (VIEW_X - strlen(cc->name)) / 2);
+       waddstr(view_w, cc->name);
+       wrefresh(view_w);
+#endif
+
+       move(LINE_T, LINE_L);
+       printw("Class %d %s (%d guns) '%s' (%c%c)",
                mc->class,
                classname[mc->class],
                mc->guns,
                ms->shipname,
                colours(ms),
                sterncolour(ms));
-       (void) refresh();
+       refresh();
+}
+
+void
+display_set_obp(int which, bool show)
+{
+       obp[which] = show;
+}
+
+void
+display_set_dbp(int which, bool show)
+{
+       dbp[which] = show;
+}
+
+////////////////////////////////////////////////////////////
+// external actions on the display
+
+void
+display_scroll_pageup(void)
+{
+       unsigned total_lines, visible_lines, limit;
+       unsigned pagesize = SCROLL_Y - 2;
+
+       total_lines = stringarray_num(sc_lines);
+       visible_lines = SCROLL_Y - 1;
+       limit = total_lines - visible_lines;
+
+       sc_scrollup += pagesize;
+       if (sc_scrollup > limit) {
+               sc_scrollup = limit;
+       }
+}
+
+void
+display_scroll_pagedown(void)
+{
+       unsigned pagesize = SCROLL_Y - 2;
+
+       if (sc_scrollup < pagesize) {
+               sc_scrollup = 0;
+       } else {
+               sc_scrollup -= pagesize;
+       }
 }
 
 void
-centerview()
+centerview(void)
 {
        viewrow = mf->row - VIEW_Y / 2;
        viewcol = mf->col - VIEW_X / 2;
 }
 
 void
-upview()
+upview(void)
 {
        viewrow -= VIEW_Y / 3;
 }
 
 void
-downview()
+downview(void)
 {
        viewrow += VIEW_Y / 3;
 }
 
 void
-leftview()
+leftview(void)
 {
        viewcol -= VIEW_X / 5;
 }
 
 void
-rightview()
+rightview(void)
 {
        viewcol += VIEW_X / 5;
 }
 
+/* Called from newturn()... rename? */
 void
-adjustview()
+display_adjust_view(void)
 {
        if (dont_adjust)
                return;